Skip to content

APPT-664 Add site type and cancellation reason field in booking extract#1529

Draft
pchoi-al wants to merge 6 commits intomainfrom
APPT-664/add-site-type-field-in-booking-extract
Draft

APPT-664 Add site type and cancellation reason field in booking extract#1529
pchoi-al wants to merge 6 commits intomainfrom
APPT-664/add-site-type-field-in-booking-extract

Conversation

@pchoi-al
Copy link
Contributor

@pchoi-al pchoi-al commented Mar 18, 2026

Description

  • Add Site Type field in Booking Extract
  • Add Cancellation Reason to Booking Extract
  • Created unit tests for Site Type and Cancellation Reason

Fixes # (issue)

Checklist:

  • My work is behind a feature toggle (if appropriate)
  • If my work is behind a feature toggle, I've added a full suite of tests for both the ON and OFF state
  • The ticket number is in the Pull Request title, with format "APPT-XXX: My Title Here"
  • I have ran npm tsc / lint (in the future these will be ran automatically)
  • My code generates no new .NET warnings (in the future these will be treated as errors)
  • If I've added a new Function, it is disabled in all but one of the terraform groups (e.g. http_functions)
  • If I've added a new Function, it has both unit and integration tests. Any request body validators have unit tests also
  • If I've made UI changes, I've added appropriate Playwright and Jest tests
  • If I've added/updated an end-point, I've added the appropriate annotations and tested the Swagger documentation reflects the change

@pchoi-al pchoi-al requested a review from pata9 March 18, 2026 15:53
…ingsReporting. Added sad path tests for CancellationReason and SiteType
… SiteTypeBookingsReporting from FeatureToggleCollection.cs
@sonarqubecloud
Copy link

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