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: Fetch session info without userDetails on local #360

Closed
wants to merge 1 commit into from

Conversation

amattu2
Copy link
Member

@amattu2 amattu2 commented May 7, 2024

Overview

This PR is a QoL update for local development until CRDCDH-591 is fixed. When going through the login process locally, a developer only needs to copy the connect.sid cookie from DEV/DEV2, then reload. You no longer need to copy the userDetails localStorage value.

Change Details (Specifics)

  • Refetch user status without userDetails localStorage value if running local dev server
  • Add NODE_ENV to AppEnv definition (no .env updates needed)
  • Add string constraints to REACT_APP_DEV_TIER

Related Ticket(s)

N/A

@amattu2 amattu2 added this to the 3.0.0 (PMVP-M1) milestone May 7, 2024
@amattu2 amattu2 added the 🚧 Do Not Merge This PR is not ready for merging label May 7, 2024
@amattu2 amattu2 closed this May 8, 2024
@amattu2 amattu2 deleted the amattu2/refetch-local-dev branch May 8, 2024 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚧 Do Not Merge This PR is not ready for merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant