search

Together-100Devs / Together

Together is a group calendar application using the MERN stack intended to bring discord communities closer!
https://together.cyclic.app/
MIT License
167 stars 112 forks source link

Initial "Login to Discord" leads back to unrefreshed Together home page #430

Open macauleydev opened 1 year ago

macauleydev commented 1 year ago

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

Caleb-Cohen commented 1 year ago

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.

macauleydev commented 1 year ago

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.

Caleb-Cohen commented 1 year ago

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?

guel-codes commented 1 year ago

@Caleb-Cohen I think this is still happening. The redirect is failing for me and I'm getting a connection failed / callback error:

image

MingJae92 commented 10 months ago

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

intelagense commented 10 months ago

Let's go! @MingJae92

MingJae92 commented 10 months ago

What are the steps to view this issue? I am quite new tot this.

intelagense commented 10 months ago

@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 https://together.cyclic.app/calendar after a successful login instead of going back to the landing page and that is not working.