Skip to content

Show a full page loader instead of briefly showing login screen while waiting for auth#1328

Merged
deep1401 merged 5 commits intomainfrom
fix/refresh-login
Feb 16, 2026
Merged

Show a full page loader instead of briefly showing login screen while waiting for auth#1328
deep1401 merged 5 commits intomainfrom
fix/refresh-login

Conversation

@deep1401
Copy link
Copy Markdown
Member

@deep1401 deep1401 commented Feb 10, 2026

@paragon-review
Copy link
Copy Markdown

Paragon Review Unavailable

Hi @deep1401! To enable Paragon reviews on this repository, please register at https://home.polarity.cc

Once registered, connect your GitHub account and Paragon will automatically review your pull requests.

Copy link
Copy Markdown
Member

@dadmobile dadmobile left a comment

Choose a reason for hiding this comment

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

I don't know what I could be doing wrong but I can't get the full page loader to show up for me? I still see the two phases of login (pre Google auth, then google auth).
I tried hard reload. I restarted web and API. Anything else I could hav emissed?

Copy link
Copy Markdown
Member Author

@deep1401 deep1401 left a comment

Choose a reason for hiding this comment

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

I don't know what I could be doing wrong but I can't get the full page loader to show up for me? I still see the two phases of login (pre Google auth, then google auth).
I tried hard reload. I restarted web and API. Anything else I could hav emissed?

@dadmobile It wont show up for the stage between pre-google auth and google auth.
That is just the App realizing the new env vars and starting with Google auth.

The full page loader shows up when you refresh and the login screen shows up momentarily in that flicker

@dadmobile
Copy link
Copy Markdown
Member

OK hmmm. I'd like to get to the root of htis so let's hold on to this for now.

@deep1401 deep1401 merged commit 4be1fd5 into main Feb 16, 2026
3 checks passed
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