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(shared-views): Create GET group-search-view/starred endpoint #88398

Merged
merged 5 commits into from
Apr 2, 2025

Conversation

MichaelSun48
Copy link
Member

This PR creates the GET group-search-views/starred endpoint, which returns the user's starred views.

This logic is directly copied over from the GET /group-search-views/ endpoint, and is intended to replace it in certain instances.

I still need to refactor the frontend to switch over to this new endpoint before I can remove this logic from the old endpoint.

@MichaelSun48 MichaelSun48 requested review from a team as code owners March 31, 2025 21:53
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Mar 31, 2025
Copy link

codecov bot commented Mar 31, 2025

Codecov Report

Attention: Patch coverage is 90.00000% with 8 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...dpoints/organization_group_search_views_starred.py 81.39% 8 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master   #88398      +/-   ##
==========================================
+ Coverage   87.65%   87.73%   +0.07%     
==========================================
  Files       10000    10002       +2     
  Lines      566485   566565      +80     
  Branches    22255    22255              
==========================================
+ Hits       496578   497051     +473     
+ Misses      69489    69096     -393     
  Partials      418      418              

@MichaelSun48 MichaelSun48 merged commit be96119 into master Apr 2, 2025
48 checks passed
@MichaelSun48 MichaelSun48 deleted the msun/sharedViews/createGetStarredViewsEndpoint branch April 2, 2025 16:53
jan-auer added a commit that referenced this pull request Apr 2, 2025
* master: (96 commits)
  fix(autofix): fix highlight popup behavior (#88552)
  🔧 chore: introduce workflow engine ui links ff (#88569)
  feat(pipeline): Add CoveragePageWrapper component and tests for it (#88519)
  feat(taskworker):Make sdk tasks taskworker compatible (#88488)
  chore(flamegraph): Updating wording to trace (#88516)
  feat(shared-views): Create `GET` `group-search-view/starred` endpoint (#88398)
  DI-612: print flaky pytest errors to log (v2) (#88512)
  chore(nav): Update banner copy (#88566)
  feat(sentryapps): add RPC method to get all installation component contents (#88179)
  chore(issue summary): Add limit to query (#88563)
  fix(dashboards): Add limit suggestion to validation (#88436)
  feat(checkout): show starting PAYG prices (#88510)
  ✨ feat(aci): add workflow_id when creating an ephemeral rule in noa (#88520)
  fix(billing): hide pay now button for self serve partners (#88504)
  chore(HC): Re-adds logging with a low sample rate for cache hits/misses on options (#88464)
  fix(logs): Upgrade sentry log integration to fix dogfooding issues (#88561)
  chore(issue summary): Remove dividers from AI summary alert (#88554)
  feat(insights): Enable bubbles in full-screen mode (#88445)
  fix(explore): Update search bar query on filter change (#88473)
  chore(dependencies): Upgrade drf-spectacular (#88459)
  ...
andrewshie-sentry pushed a commit that referenced this pull request Apr 8, 2025
…#88398)

This PR creates the `GET` `group-search-views/starred` endpoint, which
returns the user's starred views.

This logic is directly copied over from the `GET` `/group-search-views/`
endpoint, and is intended to replace it in certain instances.

I still need to refactor the frontend to switch over to this new
endpoint before I can remove this logic from the old endpoint.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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