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(issue-views): Update banner and tooltip copy, make banner dismissable, feedback -> read docs #77878

Merged
merged 4 commits into from
Sep 25, 2024

Conversation

MichaelSun48
Copy link
Member

@MichaelSun48 MichaelSun48 commented Sep 20, 2024

Makes a couple changes to the add view page for EA readiness:

  1. Updates the banner copy to call out: views are user scoped, views are reorderable, and that custom searches are persisted temporarily (only appears if you actually have any custom searches)
  2. Adds a dismiss button to the banner
  3. Switches the "Give Feedback" button for a "Read Docs" button
  4. Adds a "Learn More" link to docs within the saved searches tooltip

IMPORTANT: Issue views docs have not been merged in yet, so the docs links are broken. I will not merge this until the docs links resolve. Docs PR if you're curious about it.

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Sep 20, 2024
Copy link

codecov bot commented Sep 20, 2024

Codecov Report

Attention: Patch coverage is 23.07692% with 10 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
static/app/views/issueList/addViewPage.tsx 23.07% 10 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master   #77878      +/-   ##
==========================================
- Coverage   78.12%   78.09%   -0.03%     
==========================================
  Files        6991     7007      +16     
  Lines      310002   310474     +472     
  Branches    50724    50795      +71     
==========================================
+ Hits       242182   242464     +282     
- Misses      56101    56282     +181     
- Partials    11719    11728       +9     

<Title>
{t('Welcome to the new Issue Views experience (Early Adopter only)')}
<DismissButton
analyticsEventKey="issue_views.add_view.banner_dismissed"
Copy link
Member

Choose a reason for hiding this comment

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

if the button is the only place you are using this analytics event key, you don't need to define it in issueAnalyticsEvents

@MichaelSun48 MichaelSun48 requested a review from a team September 24, 2024 17:25
Copy link

codecov bot commented Sep 24, 2024

Bundle Report

Changes will increase total bundle size by 2.67kB (0.01%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
app-webpack-bundle-array-push 29.52MB 2.67kB (0.01%) ⬆️

0Calories pushed a commit that referenced this pull request Sep 25, 2024
Adds the prompt for the add view banner 


[related PR](#77878)
@MichaelSun48 MichaelSun48 force-pushed the msun/issueViews/addViewEAReadiness branch from 11225b5 to 6378047 Compare September 25, 2024 18:02
@MichaelSun48 MichaelSun48 merged commit 5701f09 into master Sep 25, 2024
42 of 43 checks passed
@MichaelSun48 MichaelSun48 deleted the msun/issueViews/addViewEAReadiness branch September 25, 2024 20:54
@github-actions github-actions bot locked and limited conversation to collaborators Oct 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Frontend Automatically applied to PRs that change frontend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants