dashboard group by options show duplicates about highlight HOT 15 CLOSED

from highlight.

HIG-4663 dashboard group by options show duplicates

from highlight.

To resolve the issue of duplicate group by options in the dashboard, follow these steps:

1. Dashboard.tsx: Ensure that the Graph component's props are correctly managed to avoid duplicates. Specifically, check the groupByKey prop.

2. DashboardOverview.tsx: Verify that the data fetched by useGetVisualizationsQuery does not contain duplicates. You may need to filter the results before setting the state.

3. CreateDashboardModal.tsx: When creating a new dashboard, ensure that the options for grouping are unique. This can be done by filtering the options before rendering them in the modal.

4. DashboardsContext.tsx: Ensure that the context provider does not introduce duplicates when managing the state of dashboards and their group by options.

Example code snippet to filter duplicates:

const uniqueOptions = options.filter((option, index, self) =>
  index === self.findIndex((o) => (
    o.value === option.value
  ))
);

Apply this filtering logic wherever the group by options are set or rendered.

References

/frontend/src/pages/Graphing/Dashboard.tsx
/frontend/src/pages/Graphing/DashboardOverview.tsx
/frontend/src/pages/Dashboards/components/CreateDashboardModal/CreateDashboardModal.tsx
/frontend/src/pages/Dashboards/DashboardsContext/DashboardsContext.tsx

Ask Greptile

from highlight.

/bounty $30

from highlight.

💎 $30 bounty • Highlight (YC W23)

Payment will be awarded to the first person to successfully merge a PR meeting all requirements. In the event that multiple PRs are submitted for the issue, we will award payment to the highest quality PR (the one that has cleanest code, best test coverage, most thorough, etc.).

Steps to solve:

1. Start working: Comment /attempt #8580 with your implementation plan
2. Submit work: Create a pull request including /claim #8580 in the PR body to claim the bounty
3. Receive payment: 100% of the bounty is received 2-5 days post-reward. Make sure you are eligible for payouts

Thank you for contributing to highlight/highlight!

Add a bounty • Share on socials

from highlight.

Assign me on this as I think I am close to a solution . Thanks

from highlight.

/attempt #8580

from highlight.

@Vadman97 can you please provide more context to help us better understand the issues at hand? A screenshot or recording would be greatly helpful.

from highlight.

/attempt #8580

from highlight.

/attempt #8580 hey @Vadman97 , it's been three weeks since the last pull request. Can I get assigned ?

from highlight.

from highlight.

Hey! I.have almost done with issue except the big fixes. I am crashing with pc memory issues.So made this late. Will complete it asap
…
On Fri, 7 Jun, 2024, 9:10 pm Hunain Ahmed, @.> wrote: /attempt #8580 <#8580> hey @Vadman97 https://github.com/Vadman97 , it's been three weeks since the last pull request. Can I get assigned ? Algora profile https://console.algora.io/@/hunxjunedo Completed bounties Tech Active attempts Options @hunxjunedo https://github.com/hunxjunedo 1 bounty from 1 project JavaScript Cancel attempt https://console.algora.io/api/bounties/clw8f7ky5000jky09i4p4tqe6/cancel-attempt — Reply to this email directly, view it on GitHub <#8580 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/BGXZB6C5CDNXHMBKGQWBBQDZGHH5ZAVCNFSM6AAAAABHYO42AOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJVGA4DQMZSG4 . You are receiving this because you were mentioned.Message ID: @.>

Hey, If you don't mind. can you share the recording of what the issue is? I was not able to replicate it. I could help.
I mean the ticket bug issue, not your internal crashes.

from highlight.

@Vadman97 If it is possible. can I get a screenshot or recording of the issue? I could not able to replicate it

from highlight.

💡 @rajeshj11 submitted a pull request that claims the bounty. You can visit your bounty board to reward.

from highlight.

🎉🎈 @rajeshj11 has been awarded $30! 🎈🎊

• Share on socials
• Give feedback

from highlight.