Skip to content

Bind database user.Id to useSession() object - #24

Merged
hkirat merged 3 commits into
code100x:mainfrom
Mahhheshh:bind-userid
Aug 30, 2024
Merged

Bind database user.Id to useSession() object#24
hkirat merged 3 commits into
code100x:mainfrom
Mahhheshh:bind-userid

Conversation

@Mahhheshh

Copy link
Copy Markdown
Contributor

This PR adds:

  • The session object also have an id (user database) parameter now.
  • Removed <Redirect \> component and added a new useRedirectHook (better practice).
  • Refactored the dashboard/page.tsx.
  • Fixed the issue where the user was logged out but wasn’t redirected to the homepage.

@MahhheshhMahhheshh changed the title Bind databse user.Id to useSession() objectBind database user.Id to useSession() objectAug 29, 2024
@hkirat
hkirat merged commit a6d65d6 into code100x:mainAug 30, 2024
@hkirat

Copy link
Copy Markdown
Contributor

/bounty $25

@Bot-Rakshit

Copy link
Copy Markdown
Contributor

@hkirat the page(landing) is still using the redirect component but its removed from this commit, if i'm not wrong

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

since this file is removed , we'll have to change the imports in the landing page i guess

@SujithThirumalaisamy

Copy link
Copy Markdown
Contributor

@Mahhheshh#39

Can you please follow this up sir?

@Mahhheshh

Copy link
Copy Markdown
ContributorAuthor

Can you please follow this up sir?
Looks likes #41, resolved it..

@Mahhheshh
Mahhheshh deleted the bind-userid branch August 31, 2024 13:38
Sign up for freeto 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.

5 participants

@Mahhheshh@hkirat@Bot-Rakshit@SujithThirumalaisamy@dev13-suthar