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

courses page stuck on loading page when logged out #558

Merged
merged 2 commits into from
Aug 12, 2023

Conversation

Zahra-8d
Copy link
Contributor

@Zahra-8d Zahra-8d commented Aug 2, 2023

  • added documentation
  • removed check for firebase update applied as its not necessary and when a user is not logged in and lands on an authed page, this will never be true

@Zahra-8d
Copy link
Contributor Author

Zahra-8d commented Aug 2, 2023

This fix isnt quite right. Needs rethinking

@eleanorreem
Copy link
Contributor

@Zahra-8d - I'd like your thoughts on this PR (even though ive merged it). I think the user.loading flag in the _app.tsx page does all the work. I think we can delete the piece of state .firebaseUpdateApplied as the user.token seems to indicate the same thing anyways. So confusing

@eleanorreem eleanorreem merged commit 2e754e3 into develop Aug 12, 2023
4 checks passed
@eleanorreem eleanorreem deleted the course-loading-bug branch August 12, 2023 07:54
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