Skip to content
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.

Sign up for GitHub

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

Jump to bottom

bug: Duplicated users on scatter chart #372

Closed
Tracked by #399
bdougie opened this issue Sep 15, 2022 · 6 comments · Fixed by #529
Closed
Tracked by #399

bug: Duplicated users on scatter chart #372

bdougie opened this issue Sep 15, 2022 · 6 comments · Fixed by #529
Labels
🐛 bug Something isn't working high-priority Work on these issues first low-priority 👀 needs triage released on @beta released
Milestone
Self serve SaaS -...

Comments

@bdougie
Copy link
Member

bdougie commented Sep 15, 2022

Screen Shot 2022-09-15 at 12 49 44 PM

It might make sense to grab unique users and group to PRs together in size. The idea is to see how contribution is distributed and combing them would be ideal
@0-vortex
Copy link
Contributor

Suppose this issue originating in the backend, not sure I can group by github username just yet but can look into it! 🍕

@chadstewart
Copy link
Contributor

Suppose this issue originating in the backend, not sure I can group by github username just yet but can look into it! pizza

Would be very much appreciated cause I'm having a hard time trying to fix it on my end honestly.

@bdougie bdougie mentioned this issue Sep 20, 2022
Closed
3 tasks
@eryc-cc eryc-cc changed the title Duplicated users on scatter chart bug: Duplicated users on scatter chart Sep 20, 2022
@eryc-cc eryc-cc added the 🐛 bug Something isn't working label Sep 20, 2022
@bdougie
Copy link
Member Author

bdougie commented Oct 4, 2022 •
edited
Loading

Update

When viewing the scatter chart I expect to see those same contributors listed on the contributor tab.

Steps to reproduce

Describe how to reproduce the behavior.

  1. View the dashboard
  2. Choose the recent filter
  3. Click contributors
  4. Note none of the avatars are not the same despite being the most recent contributions/prs.

Can we apply the same filter for contributions to present some overlap in the data presented?

FYI @brandonroberts - we talked about this in the Sprint Planning.

This was referenced Oct 10, 2022
Closed
Merged
brandonroberts added a commit that referenced this issue Oct 12, 2022
@brandonroberts
fix: remove duplicate contributors from the scatter chart (#529)
9b4b2c8 
Closes #372
github-actions bot pushed a commit that referenced this issue Oct 12, 2022
@brandonroberts @open-sauced
chore(patch): release 1.12.0-beta.4 on beta channel [skip ci]
2953641 
## [1.12.0-beta.4](v1.12.0-beta.3...v1.12.0-beta.4) (2022-10-12)

### 🐛 Bug Fixes

* remove duplicate contributors from the scatter chart ([#529](#529)) ([9b4b2c8](9b4b2c8)), closes [#372](#372)
@github-actions
Copy link

🎉 This issue has been resolved in version 1.12.0-beta.4 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

github-actions bot pushed a commit that referenced this issue Oct 13, 2022
@brandonroberts @open-sauced
chore(minor): release 1.13.0-beta.1 on beta channel [skip ci]
09218b8 
## [1.13.0-beta.1](v1.12.0...v1.13.0-beta.1) (2022-10-13)

### 🐛 Bug Fixes

* remove duplicate contributors from the scatter chart ([#529](#529)) ([9b4b2c8](9b4b2c8)), closes [#372](#372)

### 🍕 Features

* implement InsightPageCard component ([#519](#519)) ([0dd5057](0dd5057)), closes [#509](#509)
* re-enable toggling of bots on scatterchart ([#530](#530)) ([bcd3428](bcd3428)), closes [#476](#476)
@github-actions
Copy link

🎉 This issue has been resolved in version 1.13.0-beta.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

github-actions bot pushed a commit that referenced this issue Oct 14, 2022
@0-vortex @open-sauced
chore(minor): release 1.13.0 [skip ci]
f6fe464 
## [1.13.0](v1.12.0...v1.13.0) (2022-10-14)

### 🐛 Bug Fixes

* remove duplicate contributors from the scatter chart ([#529](#529)) ([9b4b2c8](9b4b2c8)), closes [#372](#372)

### 🍕 Features

* implement InsightPageCard component ([#519](#519)) ([0dd5057](0dd5057)), closes [#509](#509)
* re-enable toggling of bots on scatterchart ([#530](#530)) ([bcd3428](bcd3428)), closes [#476](#476)
@github-actions
Copy link

🎉 This issue has been resolved in version 1.13.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

ElpisHelle added a commit to ElpisHelle/next.js-tailwindcss that referenced this issue Aug 17, 2023
@ElpisHelle
chore(patch): release 1.12.0-beta.4 on beta channel [skip ci]
e2b5c09 
## [1.12.0-beta.4](open-sauced/app@v1.12.0-beta.3...v1.12.0-beta.4) (2022-10-12)

### 🐛 Bug Fixes

* remove duplicate contributors from the scatter chart ([#529](open-sauced/app#529)) ([9b4b2c8](open-sauced/app@9b4b2c8)), closes [#372](open-sauced/app#372)
ElpisHelle added a commit to ElpisHelle/next.js-tailwindcss that referenced this issue Aug 17, 2023
@ElpisHelle
chore(minor): release 1.13.0-beta.1 on beta channel [skip ci]
fb21f8e 
## [1.13.0-beta.1](open-sauced/app@v1.12.0...v1.13.0-beta.1) (2022-10-13)

### 🐛 Bug Fixes

* remove duplicate contributors from the scatter chart ([#529](open-sauced/app#529)) ([9b4b2c8](open-sauced/app@9b4b2c8)), closes [#372](open-sauced/app#372)

### 🍕 Features

* implement InsightPageCard component ([#519](open-sauced/app#519)) ([0dd5057](open-sauced/app@0dd5057)), closes [#509](open-sauced/app#509)
* re-enable toggling of bots on scatterchart ([#530](open-sauced/app#530)) ([bcd3428](open-sauced/app@bcd3428)), closes [#476](open-sauced/app#476)
ElpisHelle added a commit to ElpisHelle/next.js-tailwindcss that referenced this issue Aug 17, 2023
@ElpisHelle
chore(minor): release 1.13.0 [skip ci]
c1f1ba0 
## [1.13.0](open-sauced/app@v1.12.0...v1.13.0) (2022-10-14)

### 🐛 Bug Fixes

* remove duplicate contributors from the scatter chart ([#529](open-sauced/app#529)) ([9b4b2c8](open-sauced/app@9b4b2c8)), closes [#372](open-sauced/app#372)

### 🍕 Features

* implement InsightPageCard component ([#519](open-sauced/app#519)) ([0dd5057](open-sauced/app@0dd5057)), closes [#509](open-sauced/app#509)
* re-enable toggling of bots on scatterchart ([#530](open-sauced/app#530)) ([bcd3428](open-sauced/app@bcd3428)), closes [#476](open-sauced/app#476)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
🐛 bug Something isn't working high-priority Work on these issues first low-priority 👀 needs triage released on @beta released
Projects
No open projects
OpenSauced Operational Roadmap
Status: Done
Milestone
Self serve SaaS - Week 44 (continued ...
Development

Successfully merging a pull request may close this issue.

fix: remove duplicate contributors from the scatter chart open-sauced/app
5 participants
@0-vortex @bdougie @chadstewart @OgDev-01 @eryc-cc