codeforcauseorg / click-short

3 stars 7 forks source link

created AppBar and added AuthGuard #11

Closed Rathoresourabh closed 3 years ago

Rathoresourabh commented 3 years ago

Created AppBar and added AuthGuard @Abhishek-kumar09

Rathoresourabh commented 3 years ago

@Abhishek-kumar09 Fixed AppBar , and Created the Footer as mentioned in the Figma Design.

Rathoresourabh commented 3 years ago

@Abhishek-kumar09 do review my Pr once and Let me know if any changes is required.

Abhishek-kumar09 commented 3 years ago

I will look into it :)

Abhishek-kumar09 commented 3 years ago

please provide screenshots.

Rathoresourabh commented 3 years ago

Screenshot 2021-07-29 at 00-31-02 React App @Abhishek-kumar09

Rathoresourabh commented 3 years ago

Screen Shot 2021-07-29 at 00 35 00 Screen Shot 2021-07-29 at 00 36 41

Screen Shot 2021-07-29 at 01 02 33

@Abhishek-kumar09 Mobile View With Logout Button

Abhishek-kumar09 commented 3 years ago

Hey @Rathoresourabh Are you getting any problem in implementing these?

Rathoresourabh commented 3 years ago

Hi @Abhishek-kumar09 I was saying if I use local storage to get the user data to check if the user is present or not, even I tried using localstorage but it still says user is undefined whenever I reload the page, but using contextAPI it works fine .

Abhishek-kumar09 commented 3 years ago

Okay, So is this code implements the user context successfully?

Rathoresourabh commented 3 years ago

@Abhishek-kumar09 yes

Abhishek-kumar09 commented 3 years ago

Thanks @Rathoresourabh