-
-
Notifications
You must be signed in to change notification settings - Fork 17
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
Issue/504 add auth #518
Issue/504 add auth #518
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
Node CI error? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks great, and app seems to be working
Ive just left a few comments, do you mind looking at them?
@peterdudfield Yes – this needs a bit of tinkering to get Cypress to run properly in the CI – I think I got most of the way there but need to revisit when I get a chance... is an expected error atm and it runs pre-commit successfully so should be okay in meantime |
Pull Request
Description
Add Auth0 integration and corresponding endpoints/auth loading state/logout page:
Fixes #504
How Has This Been Tested?
Locally and Vercel preview branch:
https://quartz-4su29j2w3-openclimatefix.vercel.app/
Checklist: