Aayush tools equipment tracking missing graph#4508
Aayush tools equipment tracking missing graph#4508aayushshetty12 wants to merge 26 commits intodevelopmentfrom
Conversation
✅ Deploy Preview for highestgoodnetwork-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
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 Aayush,
I have reviewed your PR locally and have identified the below issue:
- The calendar year value is getting trimmed off due to text-box length
- In dark mode we cannot see the Project we are selecting
- Please do mention the related backend PR else without it the chart is not loading
Anusha-Gali
left a comment
Anusha-Gali
left a comment
There was a problem hiding this comment.
Hi Aayush,
I tried to re-review the PR after you made changes but this time i was unable to access the webpage itself. Getting the below error.
|
Anusha-Gali
left a comment
Anusha-Gali
left a comment
There was a problem hiding this comment.
Hi Aayush,
I tried to re-review the PR, this time the page did load but the chart did not load, not was i able to select the dropdown to select project. Do have a look into it.
rohanrastogi311
left a comment
rohanrastogi311
left a comment
There was a problem hiding this comment.
Hi Aayush,
The graphs are not loading.
saitejakaasoju
left a comment
saitejakaasoju
left a comment
There was a problem hiding this comment.
Hi Aayush,
I reviewed the PR and noticed that while the page loads, the chart does not render.
| const [data, setData] = useState([]); | ||
| const emptyData = []; | ||
|
|
||
| useEffect(() => { |
There was a problem hiding this comment.
If user changes filters quickly - multiple overlapping API calls.
…://github.com/OneCommunityGlobal/HighestGoodNetworkApp into aayush_tools_equipment_tracking_missing_graph
Anusha-Gali
left a comment
Anusha-Gali
left a comment
There was a problem hiding this comment.
Hi Aayush,
As per the requirements mentioned in the description the chart is visible. However the calendar does not seem to be working as well as the description says Blue Red and Yellow - however the chart has green inplace of blue and the legend in light mode covers the whole bar at times, if you could reduce the opacity it would be great. Else if these are not in your requirements do let Jae know.
4508.mov
|
7 participants
Description
Related PRS (if any):
backend PR: #1550
…
Main changes explained:
How to test:
npm installand...to run this PR locallyScreenshots or videos of changes:
Recording.2025-12-05.180655.mp4