Sharadha taking over for Manvitha - Phase 4 Student Daily Log Frontend#4225
Sharadha taking over for Manvitha - Phase 4 Student Daily Log Frontend#4225myeeli wants to merge 14 commits intodevelopmentfrom
Conversation
✅ Deploy Preview for highestgoodnetwork-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
myeeli
changed the title
myeeli
changed the title
one-community
added
the
High Priority - Please Review First
Anusha-Gali
left a comment
Anusha-Gali
left a comment
There was a problem hiding this comment.
Hi Manvitha,
I have tried to review your PR locally but i am unable to sign in nor open that link in an already signed in page. You can see this in the below video as well.
4225.mp4
VijayAnirudh
left a comment
VijayAnirudh
left a comment
There was a problem hiding this comment.
Hello!
I cannot view your page, its just white!
After that, opened in other browser and I cannot login!
|
Please verify the jwtdecode version added in the note |
Please verify the jwtdecode version added in the note |
rohanrastogi311
left a comment
rohanrastogi311
left a comment
There was a problem hiding this comment.
Hi Manvitha,
I installed jwt 4 like your notes say but still wasn't able to run the code. Please have a look and let me know to re-review.
SharadhaKasiviswanathan
changed the title
5 participants
Description
Related PRS (if any):
This frontend PR is related to the development backend PR.
…
Main changes explained:
…
How to test:
npm installand...to run this PR locallyScreenshots or videos of changes:
1.mp4
Note:
The data which is used here is mock data. API yet to be integrated
Jwt-Decode version should be 4.0.0 to do that do npm install jwt-decode@4