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

feat(auth): enhance workspace handling and error feedback #9118

Merged
merged 2 commits into from
Dec 18, 2024

Conversation

AMoreaux
Copy link
Contributor

Add support for setting a user's default workspace during sign-in if a target workspace subdomain exists. Enhance error feedback by displaying authentication error messages using a Snackbar in the front-end and improving redirect logic for workspace-specific errors.

Add support for setting a user's default workspace during sign-in if a target workspace subdomain exists. Enhance error feedback by displaying authentication error messages using a Snackbar in the front-end and improving redirect logic for workspace-specific errors.
@AMoreaux AMoreaux self-assigned this Dec 18, 2024
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Summary

Enhanced workspace-specific authentication handling and error feedback by adding default workspace selection during sign-in and improved error message display.

  • Added saveDefaultWorkspaceIfUserHasAccessOrThrow in sign-in-up.service.ts to set user's default workspace when target subdomain is specified
  • Enhanced google-auth.controller.ts to find and use active Google-auth-enabled workspaces when target is default subdomain
  • Added SnackBar error message display in VerifyEffect.tsx for authentication failures
  • Improved workspace validation with proper error handling for workspace existence and access rights
  • Updated redirect logic to use target workspace subdomain for error redirects instead of default subdomain

3 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings | Greptile

@charlesBochet charlesBochet merged commit 550756c into main Dec 18, 2024
22 checks passed
@charlesBochet charlesBochet deleted the fix/google-login-by-workspace branch December 18, 2024 15:46
Copy link

Thanks @AMoreaux for your contribution!
This marks your 31st PR on the repo. You're top 2% of all our contributors 🎉
See contributor page - Share on LinkedIn - Share on Twitter

Contributions

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.

2 participants