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

feat: Added explanation of what users are seeing on list activity graph #3328

Merged
merged 14 commits into from
Jul 30, 2024

Conversation

devharsh2k4
Copy link
Contributor

@devharsh2k4 devharsh2k4 commented May 7, 2024

Description

  • Added subtext below like we do for the Most Active Contributors chart

Related Tickets & Documents

Fixes #2270

Mobile & Desktop Screenshots/Recordings

Screenshot from 2024-05-07 23-31-09

Steps to QA

1.Go to Contributors TreeMap

Tier (staff will fill in)

  • Tier 1
  • Tier 2
  • Tier 3
  • Tier 4

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

Copy link

netlify bot commented May 7, 2024

👷 Deploy request for oss-insights pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 5018912

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Compliance Checks

Thank you for your Pull Request! We have run several checks on this pull request in order to make sure it's suitable for merging into this project. The results are listed in the following section.

Protected Branch

In order to be considered for merging, the pull request changes must not be implemented on the "beta" branch. This is described in our Contributing Guide. We would suggest that you close this PR and implement your changes as described in our Contributing Guide and open a new pull request.

Copy link

netlify bot commented May 7, 2024

Deploy Preview for design-insights ready!

Name Link
🔨 Latest commit 5018912
🔍 Latest deploy log https://app.netlify.com/sites/design-insights/deploys/66a951ac94ea6b0008aa0281
😎 Deploy Preview https://deploy-preview-3328--design-insights.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@BekahHW
Copy link
Member

BekahHW commented May 7, 2024

Hey @devharsh2k4, looks like you need to fix some things. Take a look at this for more info.

@devharsh2k4 devharsh2k4 changed the title Feature: added explanation of what users are seeing on list activity graph -feat : added explanation of what users are seeing on list activity graph May 7, 2024
@devharsh2k4 devharsh2k4 changed the title -feat : added explanation of what users are seeing on list activity graph - feat : added explanation of what users are seeing on list activity graph May 7, 2024
@devharsh2k4 devharsh2k4 changed the title - feat : added explanation of what users are seeing on list activity graph feat : added explanation of what users are seeing on list activity graph May 7, 2024
@devharsh2k4 devharsh2k4 changed the title feat : added explanation of what users are seeing on list activity graph feat: added explanation of what users are seeing on list activity graph May 7, 2024
@devharsh2k4
Copy link
Contributor Author

Hey am unable to fix the workflow errors

@devharsh2k4 devharsh2k4 changed the title feat: added explanation of what users are seeing on list activity graph feat: Added explanation of what users are seeing on list activity graph May 7, 2024
@adiati98
Copy link
Member

adiati98 commented May 14, 2024

Hey am unable to fix the workflow errors

Hey @devharsh2k4,

You need to first read the bot messages thoroughly and address them one by one. Then, once the bot is resolved, wait for a moment to see if all action workflows (as screenshot below) pass the checks.

github action checks

@BekahHW BekahHW changed the title feat: Added explanation of what users are seeing on list activity graph feat: Added explanation of what users are seeing on list activity graph May 17, 2024
@jpmcb
Copy link
Member

jpmcb commented Jul 30, 2024

cc @open-sauced/engineering let's have a look at this and clean up long standing PRs 🙏🏼

@jpmcb jpmcb requested a review from a team July 30, 2024 15:35
Copy link
Member

@nickytonline nickytonline left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR @devharsh2k4! 🚢

@nickytonline nickytonline merged commit b41dfdc into open-sauced:beta Jul 30, 2024
11 checks passed
open-sauced bot pushed a commit that referenced this pull request Jul 30, 2024
## [2.48.0-beta.7](v2.48.0-beta.6...v2.48.0-beta.7) (2024-07-30)

### 🍕 Features

* Added explanation of what users are seeing on list activity graph ([#3328](#3328)) ([b41dfdc](b41dfdc))
open-sauced bot pushed a commit that referenced this pull request Jul 30, 2024
## [2.48.0](v2.47.0...v2.48.0) (2024-07-30)

### 🍕 Features

* add new OG image for contributor highlights ([#3779](#3779)) ([a2ac727](a2ac727))
* add new OG image for repository insights ([#3800](#3800)) ([d6c1d63](d6c1d63))
* Added explanation of what users are seeing on list activity graph ([#3328](#3328)) ([b41dfdc](b41dfdc))
* added OSCR to avatar hover card ([#3803](#3803)) ([034cb1a](034cb1a))
* implement OSCR on devcards ([#3791](#3791)) ([9f2d7d2](9f2d7d2))
* MVP `OssfChart` component ([#3766](#3766)) ([d7827a3](d7827a3))
* updated one of the suggested prompts for workspaces ([#3786](#3786)) ([6dc225e](6dc225e))

### 🐛 Bug Fixes

* fixed the share button does not close when clicked outside it. ([#3617](#3617)) ([ddaa375](ddaa375))
* now pinned table columns only get pinned when embedded StarSearch is closed ([#3783](#3783)) ([c1535ac](c1535ac))
* round 'Other Contributors' to avoid overflow ([#3784](#3784)) ([62ac24d](62ac24d))
* Upgrades email validation regex to the more agnostic HTML5 regex ([#3801](#3801)) ([8643aef](8643aef))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature: add explanation of what users are seeing on list activity graph
5 participants