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.

Sign up for GitHub

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

Jump to bottom

feat(feedback): add POST endpoint for user feedback ingest #55735

Merged
merged 42 commits into from
Sep 12, 2023
Merged

feat(feedback): add POST endpoint for user feedback ingest #55735

merged 42 commits into from
Sep 12, 2023

Conversation

michellewzhang
Copy link
Member

@michellewzhang michellewzhang commented Sep 5, 2023 •
edited
Loading

Based on #55728

Will implement the project_feedback_details endpoint in a future PR.

@michellewzhang michellewzhang requested a review from a team September 5, 2023 23:26
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Sep 5, 2023
@codecov
Copy link

codecov bot commented Sep 5, 2023 •
edited
Loading

Codecov Report

Merging #55735 (d6053e8) into master (1851287) will increase coverage by 0.02%.
Report is 25 commits behind head on master.
The diff coverage is 88.50%.

@@            Coverage Diff             @@
##           master   #55735      +/-   ##
==========================================
+ Coverage   79.95%   79.98%   +0.02%     
==========================================
  Files        5060     5061       +1     
  Lines      217590   217745     +155     
  Branches    36838    36861      +23     
==========================================
+ Hits       173976   174163     +187     
+ Misses      38287    38235      -52     
- Partials     5327     5347      +20
Files Changed Coverage
src/sentry/conf/server.py ø
src/sentry/feedback/endpoints/feedback_ingest.py 88.23%
src/sentry/api/api_owners.py 100.00%
src/sentry/api/urls.py 100.00%

cmanallen
cmanallen requested changes Sep 6, 2023
View reviewed changes
src/sentry/userfeedback/serializers.py Outdated Show resolved Hide resolved
src/sentry/userfeedback/serializers.py Outdated Show resolved Hide resolved
src/sentry/userfeedback/endpoints/project_userfeedback_details.py Outdated Show resolved Hide resolved
src/sentry/userfeedback/endpoints/project_userfeedback_details.py Outdated Show resolved Hide resolved
@michellewzhang michellewzhang changed the title feat(userfeedback): add endpoints for user feedback details and index feat(user_feedback): add endpoints for user feedback details and index Sep 6, 2023
@michellewzhang michellewzhang changed the title feat(user_feedback): add endpoints for user feedback details and index feat(feedback): add endpoints for user feedback details and index Sep 6, 2023
@michellewzhang michellewzhang changed the title feat(feedback): add endpoints for user feedback index feat(feedback): add POST endpoint for user feedback ingest Sep 8, 2023
@ryan953 ryan953 requested review from cmanallen and removed request for cmanallen September 11, 2023 23:01
@michellewzhang michellewzhang merged commit 6b22ef0 into master Sep 12, 2023
57 of 58 checks passed
@michellewzhang michellewzhang deleted the mz/endpoints branch September 12, 2023 00:15
@github-actions github-actions bot locked and limited conversation to collaborators Sep 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@evanpurkhiser evanpurkhiser evanpurkhiser left review comments

@cmanallen cmanallen cmanallen approved these changes

@JoshFerge JoshFerge JoshFerge approved these changes

@billyvg billyvg Awaiting requested review from billyvg

Assignees
No one assigned
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@michellewzhang @billyvg @evanpurkhiser @JoshFerge @cmanallen