Skip to content

refactor: remove unused API endpoints not consumed by the UI#1534

Merged
oblodgett merged 4 commits intostagefrom
api_refactor
Mar 24, 2026
Merged

refactor: remove unused API endpoints not consumed by the UI#1534
oblodgett merged 4 commits intostagefrom
api_refactor

Conversation

@oblodgett
Copy link
Copy Markdown
Member

Remove 11 endpoints and their supporting code that are not called by agr_ui:

  • GET /api/gene/{id}/expression-summary, phenotype-summary
  • GET /api/variant/{id}/transcripts, alleles
  • GET /api/expression/{taxonID}
  • All /api/termName/* (PhenotypeRESTInterface)
  • GET /api/go/{id} (GoRESTInterface)
  • GET /api/entity/{id} (EntityRESTInterface)

Deleted 11 files (interfaces, controllers, services, DTOs) and cleaned up 8 remaining files. Updated tests accordingly.

Remove 11 endpoints and their supporting code that are not called by agr_ui:
- GET /api/gene/{id}/expression-summary, phenotype-summary
- GET /api/variant/{id}/transcripts, alleles
- GET /api/expression/{taxonID}
- All /api/termName/* (PhenotypeRESTInterface)
- GET /api/go/{id} (GoRESTInterface)
- GET /api/entity/{id} (EntityRESTInterface)

Deleted 11 files (interfaces, controllers, services, DTOs) and cleaned
up 8 remaining files. Updated tests accordingly.
@oblodgett oblodgett requested a review from a team as a code owner March 23, 2026 18:02
@claude
Copy link
Copy Markdown

claude bot commented Mar 23, 2026

Review

Clean removal of unused API endpoints. The changes are correct — endpoints, controllers, services, DTOs, REST interfaces, and corresponding tests are all removed consistently with no dangling references.

No bugs, no security concerns, no breaking changes to remaining functionality. LGTM.

@oblodgett oblodgett merged commit 4dc0fbc into stage Mar 24, 2026
5 checks passed
@oblodgett oblodgett deleted the api_refactor branch March 24, 2026 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants