Skip to content

feat(feedback): add BE prompt for hiding feedback from issue details #74219

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

Merged
merged 1 commit into from
Jul 12, 2024

Conversation

aliu39
Copy link
Member

@aliu39 aliu39 commented Jul 12, 2024

Relates to #74209

@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, michellewzhang and ryan953 July 12, 2024 19:29
@@ -19,6 +19,7 @@
"data_consent_banner": {"required_fields": ["organization_id"]},
"data_consent_priority": {"required_fields": ["organization_id"]},
"issue_replay_inline_onboarding": {"required_fields": ["organization_id", "project_id"]},
"issue_feedback_hidden": {"required_fields": ["organization_id", "project_id"]},
Copy link
Member

Choose a reason for hiding this comment

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

what if we renamed it to something like issue_user_feedback? the trend seems to be that the names should be more related to the component rather than the action

Copy link
Member Author

Choose a reason for hiding this comment

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

I think the _hidden makes it more clear what the component is using this prompt for

Copy link
Member

Choose a reason for hiding this comment

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

i think all prompts are used for hiding/dismissing something tho, it's implied but im not super adamant on it

Copy link

codecov bot commented Jul 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.13%. Comparing base (f9b7f3f) to head (3a04026).
Report is 2 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #74219       +/-   ##
===========================================
+ Coverage   57.27%   78.13%   +20.85%     
===========================================
  Files        6445     6665      +220     
  Lines      285309   297901    +12592     
  Branches    49067    51274     +2207     
===========================================
+ Hits       163400   232751    +69351     
+ Misses     117384    58875    -58509     
- Partials     4525     6275     +1750     
Files Coverage Δ
src/sentry/utils/prompts.py 68.00% <ø> (ø)

... and 2088 files with indirect coverage changes

@aliu39 aliu39 merged commit 9576387 into master Jul 12, 2024
51 checks passed
@aliu39 aliu39 deleted the aliu/prompt-config branch July 12, 2024 20:15
@github-actions github-actions bot locked and limited conversation to collaborators Jul 28, 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.

2 participants