fix: `RossChart` order changing on filter

open-sauced / app

🍕 Insights into your entire open source ecosystem.

https://pizza.new

Apache License 2.0

430 stars 230 forks source link

fix: `RossChart` order changing on filter #4018

Closed zeucapua closed 3 months ago

zeucapua commented 3 months ago

Description

Uses toReversed() function to create a reversed copy of stats.contributors instead of using reversed() in place, which causes the data to flip on every filter change

Related Tickets & Documents

Closes #3993

Mobile & Desktop Screenshots/Recordings

Before

https://github.com/user-attachments/assets/c2bc652b-16c2-4b70-9a9e-88aee0f5cff7

After

https://github.com/user-attachments/assets/f9c5ce67-a018-430e-b3b4-e6fbde6fb7da

Steps to QA

Tier (staff will fill in)

[ ] Tier 1
[ ] Tier 2
[ ] Tier 3
[x] Tier 4

[optional] What gif best describes this PR or how it makes you feel?

netlify[bot] commented 3 months ago

Deploy Preview for oss-insights ready!

Name	Link
Latest commit	d4efa367e1efe384edea6955d58a203bba9f57bd
Latest deploy log	https://app.netlify.com/sites/oss-insights/deploys/66c8eb9d9d884c0008186f55
Deploy Preview	https://deploy-preview-4018--oss-insights.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.