-
Notifications
You must be signed in to change notification settings - Fork 111
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
Initial "Login to Discord" leads back to unrefreshed Together home page #430
Comments
Thanks for pointing this out! Is this happening to you in production or development? I'm unable to reproduce in either, but this would coincide with the routing update. |
It happened for me on https//together.cyclic.app. So far, it's not a big deal, but I figured I'd open the issue in case it happens for other new users. |
No errors or anything in the console? |
@Caleb-Cohen I think this is still happening. The redirect is failing for me and I'm getting a connection failed / callback error: |
Hi there, I would like to fix this issue. Please may I be assigned? I am still pretty new to OS, so please bare with me. Kind regards Ming Chi |
Let's go! @MingJae92 |
What are the steps to view this issue? I am quite new tot this. |
@MingJae92 The original issue was that when you logged in, it would take you back to the landing page and the login button would be in the wrong state. Right now that is working for me, but see if it breaks for you. I think the other part is that it should redirect to |
Describe the bug
On my first visit to the Together website, after clicking "Login to Discord" and logging in and granting access, the browser took me back to the Together homepage with both login buttons unchanged ("Login" and "Login to Discord").
To Reproduce
I have not yet been able to reproduce this bug.
Things I tried:
Expected behavior
The login buttons on the Together homepage should reflect login status (by changing to "Logout" after successful login). (This happened correctly on all login attempts after the first.)
Redirection to the calendar (and display of a welcome message) appears to be another intended result of successful login.
Screenshots
(none yet)
Which Branch should this fix be Pulled into?
(none yet)
Device Information
Desktop:
Additional context
(none yet)
If you're interested in working on this feature, please comment so you can be assigned
The text was updated successfully, but these errors were encountered: