-
Notifications
You must be signed in to change notification settings - Fork 320
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enhanced dashboard group by options without any duplicates #8624
base: main
Are you sure you want to change the base?
Conversation
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Summary
- DashboardsContext.tsx: Enhanced state management to prevent duplicate dashboards.
- CreateDashboardModal.tsx: Introduced a new Select component to handle unique 'Group By Options'.
- Dashboard.tsx: Added a filter to ensure unique
groupByKey
values in dashboard graphs. - DashboardOverview.tsx: Implemented a mechanism to filter out duplicate visualizations, ensuring unique entries.
Notes
- These changes improve user experience by preventing duplicate entries across various dashboard components.
Comments
frontend/src/pages/Dashboards/DashboardsContext/DashboardsContext.tsx
- Line 32: The
DashboardsContextProvider
is redefined here, which might cause a conflict with the existing export. Consider renaming the new provider to avoid confusion. - Line 45: The
upsertDashboardMutation
function is used here but not imported. Ensure it is imported correctly to avoid runtime errors. - Line 52: The
uniqueDashboards
filter logic is incorrect. It should filter based on thenewDashboard.id
instead ofdashboard.id
to ensure uniqueness.
frontend/src/pages/Graphing/Dashboard.tsx
- Line 7: The
uniqueGroupByKey
filtering logic is correct, but it could be optimized using a Set for better performance with large arrays.
frontend/src/pages/Graphing/DashboardOverview.tsx
- Line 303: Update the type of 'data' prop in 'DashboardRows' to reflect the new structure.
💵 To receive payouts, sign up on Algora, link your Github account and connect with Stripe/Alipay. |
@greptile review the changes. I have incorporated all your suggestions |
@Vadman97 PTAL. I think it's ready for your final review and also can please approve the workflows. |
@asr2003 looks like there are some issues with the build - see https://github.com/highlight/highlight/actions/runs/9153049403/job/25193368143?pr=8624 |
Fixing them |
@Vadman97 I have fixed the errors of build emitted here: |
@Vadman97 I have also addressed the build errors emitted recently. PTAL |
@asr2003 can you also run |
@Vadman97 I will fix the bugs. Currently I am troubleshooting my laptop with some repairs |
any update @asr2003 ? |
Yes, done with the setup. Hopefully will fix it soon. Thanks for guiding
and understanding
…On Fri, 31 May, 2024, 9:07 pm Vadim Korolik, ***@***.***> wrote:
any update @asr2003 <https://github.com/asr2003> ?
—
Reply to this email directly, view it on GitHub
<#8624 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BGXZB6DPVEBPSG3E4FAAVO3ZFCKKLAVCNFSM6AAAAABH65US46VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBSGUYTOMRUGE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Summary
This PR addresses the issue of duplicate group by options across various dashboard components. The focus has been on implementing specific code changes to ensure the uniqueness of group by options.
How did you test this change?
Will be tested on deployment
Are there any deployment considerations?
N/A
Does this work require review from our design team?
N/A