braddf commented 4 months ago

Pull Request

Description

Add Auth0 integration and corresponding endpoints/auth loading state/logout page:

Fixes #504

N.B. also includes small fix for nowcasting-app as the SSR wasn't passing Cypress tests with date string formats being different between prerender on server and render on client-side, causing a React hydration error. Have forced the component to render client-side for now, but may need to keep an eye on this for date/time stuff.

How Has This Been Tested?

Locally and Vercel preview branch: https://quartz-4su29j2w3-openclimatefix.vercel.app/

Checklist:

[x] My code follows OCF's coding style guidelines
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have checked my code and corrected any misspellings

vercel[bot] commented 4 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
quartz-app	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	May 30, 2024 11:04am

1 Ignored Deployment

peterdudfield commented 4 months ago

Node CI error?

braddf commented 4 months ago

Node CI error?

@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

openclimatefix / quartz-frontend

Issue/504 add auth #518

Pull Request

Description

How Has This Been Tested?

Checklist: