Fixed bug where refresh neccessary after sign in.

[syanase-calpoly]

Developer: Shuya Yanase

Closes #123

Pull Request Summary

• I fixed a bug where after sign in, it would require a refresh to see that you actually signed in on the navbar

Modifications

• I modified the sign in page so that it would require refresh when sent to home page since navbar is server component so cant be modified without a refresh

Testing Considerations

Pull Request Checklist

• [x ] Code is neat, readable, and works
• [x ] Comments are appropriate
• [x ] The commit messages follows our guidelines
• [x ] The developer name is specified
• [x ] The summary is completed
• [x ] Assign reviewers

Screenshots/Screencast

Untitled_.May.12.2024.11_02.PM.webm

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
slo-beaver-brigade	❌ Failed (Inspect)			May 20, 2024 3:19am

[javalosr2004]

Hey Shuya! I love the effort with figuring out ways to revalidate the data, I'm wondering if we can revalidate the route, using revalidatePath("/", "layout"), I have gone ahead and made this change with dashboard, and it seems to have fixed the issue with loading signed in events!

[javalosr2004]

Ok I see that you fixed the navbar issue, don't worry about the refresh, I revalidated the path for /dashboard, I think one issue that remains with your implementation is, it doesn't reflect admins role correctly.

[javalosr2004]

Ok so I went ahead and made those changes, the thing now is we need some way of letting the user know that they are being redirected, can you make the login / verifcation button disabled while they are being redirected? Also, Aubdrey brought up an issue about forget password, it seemed that she was unable to change it.