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

VIH-10899 Delete obsolete V1 operations and models #919

Open
wants to merge 21 commits into
base: master
Choose a base branch
from

Conversation

shaed-parkar
Copy link
Contributor

@shaed-parkar shaed-parkar commented Nov 21, 2024

Jira link

VIH-10899

Change description

Delete V1 operations and migrate those that still used obsolete models to V2 namespace

More API tests

  • Opting for API tests first. This will help identify which paths are executed and which parts of the application can be removed. Most classes which have no coverage except for historical DAL integration tests have no route via API operations or services are likely safe to removal

Package Updates:

  • Updated System.Text.Json package version from 8.0.0 to 8.0.5 in BookingsApi.Client.csproj and BookingsApi.Common.DotNet6.csproj files.
  • Updated Microsoft.Identity.Client package version from 4.60.3 to 4.60.4 in BookingsApi.Common.csproj and packages.lock.json files.

Code Removal:

  • Removed IVhLogger and VhLogger classes from BookingsApi.Common.
  • Removed several unused request and enum classes from BookingsApi.Contract/V1/Requests and BookingsApi.Contract/V1/Enums.

Refactoring:

  • Refactored constructors to use property initializers in AddJusticeUserRequest, GetHearingRequest, and similar classes.

File Renaming:

  • Renamed HearingRoles.cs from BookingsApi.DAL/Helper to BookingsApi.Common.Helpers and added a new constant.

@shaed-parkar shaed-parkar requested review from a team as code owners November 21, 2024 11:44
@shaed-parkar shaed-parkar force-pushed the feature/VIH-10899_remove_v1_and_obsolete branch from 7b4cdcf to 9b7d3e3 Compare November 25, 2024 14:55
@shaed-parkar shaed-parkar force-pushed the feature/VIH-10899_remove_v1_and_obsolete branch from 57f74ae to 39b90cc Compare December 2, 2024 13:59
Copy link

sonarcloud bot commented Dec 3, 2024

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