Taariq: Fix filterColor persistence issue on change and refresh for Individual as well as bulk Selected filterColors Backend.#1667
Conversation
sourabhbagde
left a comment
There was a problem hiding this comment.
Blocking to Test
Error: ReferenceError: dashboardMetrics is not defined. at runtime.
Root Cause: dashboardMetrics is referenced in src/startup/routes.js (passed into bmDashboardPrototypeRouter) even though the both src/models/bmdashboard/dashboardMetrics.js and src/routes/bmdashboard/bmDashboardPrototypeRouter.js were removed in this PR and are not defined.
Attaching screenshots - please refer and let me know if I am missing anything here.
akshith312
left a comment
There was a problem hiding this comment.
Provided Feedback in the corresponding FE PR 3916
RitzzzZ2021
left a comment
There was a problem hiding this comment.
The update in backend doesn't seem to work. See screenshots and more details in the corresponding frontend PR #3916
|
Tested the related frontend PR-3916 for this branch. The changes work as expected. |
|
Update does not seem to be functioning as expected. I have provided details in the corresponding #3916 comments. |
|
Tested the related frontend PR-3916 and its working as expected. |
|
Tested with frontend PR #3916 , backend changes work correctly for filter color updates and persistence. |
|
Hi Taariq, |
Sriamshreddy000
left a comment
There was a problem hiding this comment.
Tested this PR along with the related frontend PR #3916.
Verified via browser console that the filterColor updates are correctly sent to and persisted by the backend.
Everything is working as expected, left detailed notes and screenshots in the frontend PR.
Anusha-Gali
left a comment
There was a problem hiding this comment.
Hi Taariq,
I tested the PR locally and provided the feedback in the FrontEnd PR: OneCommunityGlobal/HighestGoodNetworkApp#3916 (review)
…r it to be merged with the development branch
…ting a validation check
|
Thank you all, merging! |
Description
This PR addresses issues in the filterColor handling and bulk updates within user profiles. Previously, filterColor updates were not consistently applied across bulk operations, leading to mismatches between frontend UI state and backend data.
Related PRS (if any):
This backend PR is related to the frontend PR 3916. Please checkout both the PR's while testing it on your local.
…
Main changes explained:
How to test:
npm installandnpm run devto start the backend server.Screenshots or videos of changes:
Note:
Only the above files were changed by me, rest other files were fetched from the origin as I was behind several commits from the development branch.