Skip to content

ref(issues): Move multiple endpoints and tests to issues folder#97428

Merged
JoshFerge merged 5 commits intomasterfrom
jferg/move-multiple-endpoints
Aug 11, 2025
Merged

ref(issues): Move multiple endpoints and tests to issues folder#97428
JoshFerge merged 5 commits intomasterfrom
jferg/move-multiple-endpoints

Conversation

@JoshFerge
Copy link
Member

@JoshFerge JoshFerge commented Aug 7, 2025

Summary

Move multiple issue-related endpoints and the issue_search module from the API package to the issues package to better organize the codebase.

Files Moved:

Endpoints:

  • group_attachments.py
  • team_all_unresolved_issues.py
  • team_issue_breakdown.py
  • organization_event_details.py
  • organization_issues_resolved_in_release.py
  • issue_search.py (module)

Tests:

  • test_group_attachments.py
  • test_team_all_unresolved_issues.py
  • test_team_issue_breakdown.py
  • test_issue_search.py

Changes:

  • Updated all import references in src/sentry/api/urls.py
  • Updated imports in affected source files and tests
  • Maintained all existing functionality
  • Added proper type annotations to moved test files

refs https://linear.app/getsentry/issue/RTC-1110/move-issues-endpoints-into-issues-app

@linear
Copy link

linear bot commented Aug 7, 2025

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Aug 7, 2025
@codecov
Copy link

codecov bot commented Aug 7, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #97428      +/-   ##
==========================================
+ Coverage   79.13%   80.64%   +1.50%     
==========================================
  Files        8564     8564              
  Lines      377171   377170       -1     
  Branches    24562    24562              
==========================================
+ Hits       298488   304180    +5692     
+ Misses      78317    72624    -5693     
  Partials      366      366              

- Move group_attachments, team_all_unresolved_issues, team_issue_breakdown, organization_event_details, organization_issues_resolved_in_release endpoints to issues/endpoints/
- Move issue_search module from api/ to issues/
- Move corresponding test files to tests/sentry/issues/endpoints/
- Update all import references in api/urls.py and other affected files

refs https://linear.app/getsentry/issue/RTC-1110/move-issues-endpoints-into-issues-app
@JoshFerge JoshFerge force-pushed the jferg/move-multiple-endpoints branch from 4648630 to 987ecaf Compare August 11, 2025 20:41
@JoshFerge JoshFerge marked this pull request as ready for review August 11, 2025 21:23
@JoshFerge JoshFerge requested review from a team as code owners August 11, 2025 21:23
@JoshFerge JoshFerge requested a review from a team August 11, 2025 21:23
@JoshFerge JoshFerge requested a review from a team as a code owner August 11, 2025 21:23
@JoshFerge JoshFerge merged commit 27053fe into master Aug 11, 2025
67 checks passed
@JoshFerge JoshFerge deleted the jferg/move-multiple-endpoints branch August 11, 2025 21:29
andrewshie-sentry pushed a commit that referenced this pull request Aug 12, 2025
## Summary
Move multiple issue-related endpoints and the `issue_search` module from
the API package to the issues package to better organize the codebase.

### Files Moved:
**Endpoints:**
- `group_attachments.py`
- `team_all_unresolved_issues.py` 
- `team_issue_breakdown.py`
- `organization_event_details.py`
- `organization_issues_resolved_in_release.py`
- `issue_search.py` (module)

**Tests:**
- `test_group_attachments.py`
- `test_team_all_unresolved_issues.py`
- `test_team_issue_breakdown.py` 
- `test_issue_search.py`

### Changes:
- Updated all import references in `src/sentry/api/urls.py`
- Updated imports in affected source files and tests
- Maintained all existing functionality
- Added proper type annotations to moved test files

refs
https://linear.app/getsentry/issue/RTC-1110/move-issues-endpoints-into-issues-app
priscilawebdev pushed a commit that referenced this pull request Aug 25, 2025
## Summary
Move multiple issue-related endpoints and the `issue_search` module from
the API package to the issues package to better organize the codebase.

### Files Moved:
**Endpoints:**
- `group_attachments.py`
- `team_all_unresolved_issues.py` 
- `team_issue_breakdown.py`
- `organization_event_details.py`
- `organization_issues_resolved_in_release.py`
- `issue_search.py` (module)

**Tests:**
- `test_group_attachments.py`
- `test_team_all_unresolved_issues.py`
- `test_team_issue_breakdown.py` 
- `test_issue_search.py`

### Changes:
- Updated all import references in `src/sentry/api/urls.py`
- Updated imports in affected source files and tests
- Maintained all existing functionality
- Added proper type annotations to moved test files

refs
https://linear.app/getsentry/issue/RTC-1110/move-issues-endpoints-into-issues-app
@github-actions github-actions bot locked and limited conversation to collaborators Aug 27, 2025
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