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(prompts): support a visible status for un-hiding components #74228

Merged
merged 6 commits into from
Jul 15, 2024

Conversation

aliu39
Copy link
Member

@aliu39 aliu39 commented Jul 12, 2024

Needed for #74209. We're using the prompt backend to persist a user's decision to hide/unhide user feedback in the issue details page. The goal is to be able to toggle between promptIsDismissed = true/false.

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Jul 12, 2024
@aliu39 aliu39 requested review from a team and ryan953 July 12, 2024 20:41
Copy link
Member

@evanpurkhiser evanpurkhiser left a comment

Choose a reason for hiding this comment

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

is there a test we can add?

Copy link
Member

@markstory markstory left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@aliu39 aliu39 enabled auto-merge (squash) July 12, 2024 21:02
Copy link

codecov bot commented Jul 12, 2024

Codecov Report

Attention: Patch coverage is 83.33333% with 1 line in your changes missing coverage. Please review.

Project coverage is 78.13%. Comparing base (597d259) to head (447c5c1).
Report is 4 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #74228      +/-   ##
==========================================
- Coverage   79.94%   78.13%   -1.82%     
==========================================
  Files        6836     6665     -171     
  Lines      307457   297923    -9534     
  Branches    51252    51281      +29     
==========================================
- Hits       245808   232772   -13036     
+ Misses      61225    58876    -2349     
- Partials      424     6275    +5851     
Files Coverage Δ
src/sentry/api/endpoints/prompts_activity.py 80.23% <83.33%> (-8.80%) ⬇️

... and 1753 files with indirect coverage changes

@aliu39 aliu39 disabled auto-merge July 15, 2024 03:50
@aliu39 aliu39 merged commit 5a9f327 into master Jul 15, 2024
49 of 50 checks passed
@aliu39 aliu39 deleted the aliu/prompt-api-visible branch July 15, 2024 16:15
aliu39 added a commit that referenced this pull request Jul 16, 2024
…74236)

Depends on #74228

Relates to #74209

Adds a button to the "User Feedback" section of issue details page.
Allows members to hide/unhide feedback from **all** issue details in a
project. Persists this decision across reloads using the prompt backend

Updated demo with chevron/drop-down style hiding:


https://github.com/user-attachments/assets/43a02a63-98c0-42bd-8651-f1baeeb22218



Updated loading indicator: (you can see for a split sec at :12)


https://github.com/user-attachments/assets/a2d57c51-e459-467b-9a9c-5e227050d9b7
@github-actions github-actions bot locked and limited conversation to collaborators Jul 31, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants