Skip to content

feat(ui): implement retry logic for contributors list in About screen#633

Closed
RajnishKMehta wants to merge 2 commits into
Arturo254:masterfrom
RajnishKMehta:implement-about-screen-retry
Closed

feat(ui): implement retry logic for contributors list in About screen#633
RajnishKMehta wants to merge 2 commits into
Arturo254:masterfrom
RajnishKMehta:implement-about-screen-retry

Conversation

@RajnishKMehta

Copy link
Copy Markdown
Contributor

This PR implements a retry mechanism for the contributors list in the About screen.

Key changes:

  1. Modified AboutViewModel to make the data fetching method fetchContributorsFromGitHub() public.
  2. Enhanced the ErrorCard component in AboutScreen with a 'Retry' button that triggers the data fetch.
  3. Updated the UI to handle the retry callback.

Technically, this uses the existing AboutViewModel state management and consistent Material 3 styling with FilledTonalButton and RoundedCornerShape(14.dp) as seen elsewhere in the screen.

- Made fetchContributorsFromGitHub() in AboutViewModel public to allow manual refresh.
- Updated ErrorCard in AboutScreen to include a "Retry" button.
- Connected the "Retry" button to the ViewModel's fetch method.
- Used consistent styling with existing UI components (FilledTonalButton with 14dp rounded corners).
@codesandbox

codesandbox Bot commented May 31, 2026

Copy link
Copy Markdown

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

@RajnishKMehta

Copy link
Copy Markdown
Contributor Author

@Arturo254

El flujo de trabajo está fallando con el error "403: Resource not accessible by integration" al intentar publicar un comentario en el PR.

Parece que GitHub Actions no tiene los permisos necesarios para crear o actualizar comentarios en pull requests.

Es posible que debas habilitar los permisos adecuados para los workflows:

  1. Ve a Settings → Actions
  2. En Workflow permissions, selecciona:
    • Read and write permissions
  3. Guarda los cambios.

Workflow Permissions

Sin permisos de escritura, el workflow no puede crear ni eliminar comentarios en los pull requests.

Translated into Spanish using ChatGPT.

@RajnishKMehta

Copy link
Copy Markdown
Contributor Author

Implemented 577194d

@RajnishKMehta RajnishKMehta deleted the implement-about-screen-retry branch June 1, 2026 00:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant