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.

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

Bug: Add explainer tootlip to "active", "alum", and "new" highlight cards on list pages #2251

Closed
2 tasks done
jpmcb opened this issue Dec 4, 2023 · 6 comments · Fixed by #2453
Closed
2 tasks done
Assignees
Labels
🐛 bug Something isn't working core team work Work that the OpenSauced core team takes on released on @beta released
Milestone

Comments

@jpmcb
Copy link
Member

jpmcb commented Dec 4, 2023

Describe the bug

When looking at "new", "alum", and "active" user list contribution charts, it's not obvious what these terms mean and how they relate to the current time/date range.

Screenshot 2023-12-04 at 8 52 06 AM

This is an example of one of my charts where I've selected alum contributors but nothing else much shows up.

For clarification:

  • "All" are all contributors in the current and previous timeblocks.
  • "New" are contributors who've contributed in the current timeblock but not the last timeblock.
  • "Alum" are contributors who've contributed in the previous timeblock but not the current one.
  • "Active" are contributors who've contributed in both the current and previous timeblocks.

This is a sort of sliding window algorithm. So, for example, with a timeblock of 30 days, a new contributor is someone who's contributed in the last 30 days but not 30-60 days ago.

Cc @isabensusan - not sure how to approach this from the product perspective.

Steps to reproduce

  1. Go to a list chart
  2. Notice there's no explanation on what our terms mean.

Browsers

Firefox

Additional context (Is this in dev or production?)

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs
@jpmcb jpmcb added 🐛 bug Something isn't working core team work Work that the OpenSauced core team takes on labels Dec 4, 2023
Copy link
Contributor

github-actions bot commented Dec 4, 2023

Thanks for the issue, our team will look into it as soon as possible! If you would like to work on this issue, please wait for us to decide if it's ready. The issue will be ready to work on once we remove the "needs triage" label.

To claim an issue that does not have the "needs triage" label, please leave a comment that says ".take". If you have any questions, please reach out to us on Discord or follow up on the issue itself.

For full info on how to contribute, please check out our contributors guide.

@BekahHW
Copy link
Member

BekahHW commented Dec 8, 2023

@jpmcb I thought this was going to be part of the tooltip issue. It's mentioned in here: #1806 (comment). @OgDev-01 do you know?

@jpmcb
Copy link
Member Author

jpmcb commented Dec 8, 2023

Ah nice: yeah I think that issue covers it - this is a duplicate, I didn't see that issue when I initially searched for it. Thanks for the heads up!

@jpmcb jpmcb closed this as completed Dec 8, 2023
@isabensusan
Copy link
Member

isabensusan commented Jan 8, 2024

Hey! Even if we had scoped this on the quoted issue and PR, the active/alum/new didn't get added so I'm re-opening this issue.

To add more context:

Scope of work

In PR #2309 we implemented explainer tooltips to the metrics cards on Insight pages that give users a brief description of what the metric refers to:
image

We want to replicate the same change to the metric cards in lists pages:
image

Using the following copy by @BekahHW :
Active Contributors - a contributor who has been active in the last 30 days.
New Contributors - a contributor who has recently made their first contribution to a project.
Alumni Contributors - a contributor who has not been active on a project in the last 30 days.

@isabensusan isabensusan reopened this Jan 8, 2024
@isabensusan isabensusan changed the title Bug: List chart "active", "alum", and "new" are not explained for users Bug: Add explainer tootlip to "active", "alum", and "new" highlight cards on list pages Jan 8, 2024
@isabensusan isabensusan added this to the Papercuts milestone Jan 8, 2024
@OgDev-01 OgDev-01 self-assigned this Jan 8, 2024
Copy link
Contributor

open-sauced bot commented Jan 12, 2024

🎉 This issue has been resolved in version 1.83.0-beta.5 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Copy link
Contributor

open-sauced bot commented Jan 17, 2024

🎉 This issue has been resolved in version 1.83.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@open-sauced open-sauced bot added the released label Jan 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working core team work Work that the OpenSauced core team takes on released on @beta released
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants