Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement Error States and Settings Link for GitHub Integration Panel #474

Open
toyamarinyon opened this issue Mar 18, 2025 · 2 comments
Open
Assignees
Labels
good first issue Good for newcomers help wanted Extra attention is needed

Comments

@toyamarinyon
Copy link
Contributor

toyamarinyon commented Mar 18, 2025

The current GitHub Integration panel in the Agentic-workflow builder app only displays the UI for a successfully authenticated and configured state. This issue tracks the work required to handle other possible states and improve the user experience.

Tasks:

  • Implement UI for unauthorized state: When the user has not authorized the application to access their GitHub account, display an appropriate message and a clear call to action to authorize.
  • Implement UI for invalid-credential state: When the user's GitHub credentials are invalid (e.g., token expired, revoked), display an error message and guide the user to re-authenticate or update their credentials.
  • Add a "Change Settings" link: Provide a clear and easily accessible link within the GitHub Integration panel that allows users to modify their GitHub settings (e.g., re-authenticate, change repositories).
    Context:

Currently, users encountering unauthorized or invalid-credential states may be confused or unable to resolve the issue. Providing clear UI and guidance will improve the user experience and reduce support requests.

Reference:

The following code snippet from another project may be helpful as a reference for implementing these states:

function GitHubIntegration() {
const { status } = useGitHubIntegration();
const content = () => {
switch (status) {
case "unauthorized":
return <RequireGitHubAuthorization />;
case "invalid-credential":
return <RequireGitHubReAuthorization />;
case "installed":
case "not-installed":
return <GitHubIntegrationForm />;
default:
throw new Error(status satisfies never);
}
};
return (
<ContentPanel>
<ContentPanelHeader>GitHub Integration</ContentPanelHeader>
{content()}
</ContentPanel>
);
}

@toyamarinyon toyamarinyon added help wanted Extra attention is needed good first issue Good for newcomers labels Mar 18, 2025
@prakha
Copy link

prakha commented Mar 23, 2025

look like a nice issue to work on, can i work on this issue??

@toyamarinyon
Copy link
Contributor Author

@prakha Thanks for your interest, Prakha! Yes, absolutely, please feel free to work on this issue. We appreciate your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants