Sharadha taking over for Manvitha pr review team analytics dashboard#3861
Sharadha taking over for Manvitha pr review team analytics dashboard#3861myeeli wants to merge 14 commits into
Conversation
✅ Deploy Preview for highestgoodnetwork-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Namitha7070
left a comment
Namitha7070
left a comment
There was a problem hiding this comment.
The message "Reviewers Ranked by Requirement Satisfied" is being displayed but when selecting the dropdown for "All time", No chart, data, or visualization is being rendered under the heading.
It takes sometime to get the data, loader was not implemented as this UI is implemented only for testing. After sometime you can see the data. |
|
The notification "Reviewers Ranked by Requirement Satisfied" is currently visible; however, upon choosing the dropdown option for "All time," no chart, data, or visual representation is shown beneath the heading. I have waited sometime but still got the same output
|
|
I chose the dropdown option for "All time" nothing shown. 
|
Did you setup the environment variable as i have mentioned in the description,if not please follow the below step. To Test this API you need to get the GITHUB_TOKEN by following the below video and add it in the environment variables |
There was a problem hiding this comment.
Hi Manvitha,
I have reviewed your PR locally and have included the github token in the backend env file. In the frontend i have spotted a common issue across all the 4 duration's i.e the names are not visible for few reviewers with only bar graphs visible and we need to hover on them to see the name and also for the duration "Last month and All Time" why is there a scroll bar ? Shouldn't the graph take up the whole webpage rather than being a scrollable center element since when i scroll the entire webpage down it looks empty and weird.
Issue 1: Missing names
Issue 2: Scroll - empty space
|
one-community
changed the title
|
SharadhaKasiviswanathan
added
the
High Priority - Please Review First
6 participants
Description
Related PRS (if any):
This frontend PR is related to the #1648 backend PR.
Closed other version of the frontend: #3829
…
Main changes explained:
This UI is only for testing the backend
…
How to test:
npm installand...to run this PR locallyScreenshots or videos of changes:
Note:
Include the information the reviewers need to know.