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

DR | Remove old API endpoints & feature toggle #102316

Open
23 tasks
Mottie opened this issue Jan 31, 2025 · 0 comments
Open
23 tasks

DR | Remove old API endpoints & feature toggle #102316

Mottie opened this issue Jan 31, 2025 · 0 comments
Labels
Decision-Reviews-Team (Formerly squad-2) Label for issues being worked on by Decision Reviews on BMT & DR team frontend HLR Higher Level Review needs-refinement Identifies tickets that need to be refined NOD Notice of Disagreement SC Supplemental Claim (Decision Reviews)

Comments

@Mottie
Copy link
Contributor

Mottie commented Jan 31, 2025

Value Statement

As a developer on Decision Reviews
I want to remove old API endpoints and associated feature toggle code
So that we reduce tech debt


Background Context

Our backend was modularized and to ensure we didn't encounter any problems, we put the new API behind feature toggles. All feature toggles are now fully enabled. This ticket is to remove the older API endpoints and feature toggles:

  • decision_review_hlr_new_api
  • decision_review_nod_new_api
  • decision_review_sc_new_api

Tasks

  • Remove usage of feature toggles from HLR, NOD & SC
  • Remove old API endpoint constants
  • Remove feature toggle in platform code
  • Update unit tests
  • Update e2e tests

Acceptance Criteria

  • Feature toggles removed from HLR, NOD & SC
  • Old API code removed from HLR, NOD & SC
  • All tests passing

Definition of Ready

  • Clear value description
  • Testable acceptance criteria
  • Accessibility added to acceptance criteria
  • Approved designs attached
  • Sample data provided where appropriate
  • Estimated to fit within the sprint
  • Dependencies and blockers linked

Definition of Done

  • Meets acceptance criteria
  • Passed E2E testing (90% coverage)
  • Passed unit testing (90% coverage)
  • Passed integration testing (if applicable)
  • Code reviewed (internal)
  • Submitted to staging
  • Team approved production verification process
  • Reviewed and approved by product and/or design
@Mottie Mottie added Decision-Reviews-Team (Formerly squad-2) Label for issues being worked on by Decision Reviews on BMT & DR team frontend HLR Higher Level Review needs-refinement Identifies tickets that need to be refined NOD Notice of Disagreement SC Supplemental Claim (Decision Reviews) labels Jan 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Decision-Reviews-Team (Formerly squad-2) Label for issues being worked on by Decision Reviews on BMT & DR team frontend HLR Higher Level Review needs-refinement Identifies tickets that need to be refined NOD Notice of Disagreement SC Supplemental Claim (Decision Reviews)
Projects
None yet
Development

No branches or pull requests

1 participant