Refactor/auth architecture pt2 #277
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
| async function handleClick() { | ||
| const [res, error] = await serverSignOut(); | ||
| const authAdapter = resolve<AuthClientAdapter>(TYPES.AuthClientAdapter); |
There was a problem hiding this comment.
In my opinion this is not the proper way to use the classes inside the components, but if this is working now I will fix it in my next task
There was a problem hiding this comment.
Sure, I'd like to see what suggestion you have. Do you want to go ahead and merge this for now? This is how I did it with the login (which has been merged already)
There was a problem hiding this comment.
If you'd rather wait, then just do your fix in the sign in page (I forget the exact filename), and submit the pr with the suggested change (along with the other tasks) and then we can revisit this one
There was a problem hiding this comment.
The fix is straightforward and is related to the task on ClickUp. If you agree, I prefer to merge this pr and then do the fix. If things work I would like to complete the migration server >>> client components and then do the task on ClickUp
Description
Added the logout functionality to the modules folder
Expected behavior: Clicking on sign out in the drop down will redirect user to the sign in page and also clear cookies in the browser
Issue link
Fixes # (issue)
Type of change
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist: