Skip to content

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
@codecov
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
@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.

4 participants