ecohabit icon indicating copy to clipboard operation
ecohabit copied to clipboard

Published 20 hours ago • verified publisher icon lugenx

implement private route component

Open Owobilum opened this issue 1 year ago • 6 comments

What does this PR do?

  • Adds a private route component to resolve issue #230

Owobilum avatar Aug 08 '23 11:08 Owobilum

Hi @Owobilum, this is currently under review. could you please leave a comment under the issue? That way, I can assign it to you. Thanks!

lugenx avatar Aug 09 '23 21:08 lugenx

Hi @Owobilum, this is currently under review. could you please leave a comment under the issue? That way, I can assign it to you. Thanks!

Hello @lugenx, it's done

Owobilum avatar Aug 10 '23 08:08 Owobilum

Nice work @Owobilum! I noticed though that from the ResponsiveAppBar the user can click 'Home' and be directed to the Homepage still.

Hello @tsimian. Kindly check again

Owobilum avatar Aug 14 '23 13:08 Owobilum

I just have a few questions, it's looking great though! Coming to think of it, we will need additional protected routes but for users who are already signed in, that can't visit pages such as /login, since they are already logged in. This can be taken care of on a different issue!

rcamach7 avatar Aug 22 '23 23:08 rcamach7

hi @Owobilum , when you get a chance, could you please address @rcamach7 's comments above? thank you.

lugenx avatar Aug 28 '23 00:08 lugenx

Hi @Owobilum , any update?

lugenx avatar Oct 23 '23 21:10 lugenx