Skip to content

AM 1175 boat-charging: add cost-breakdown type - #236

Merged
jjbeekman merged 1 commit into
mainfrom
boat-charging/AM-1175-add-cost-breakdown-type
Aug 24, 2026
Merged

AM 1175 boat-charging: add cost-breakdown type#236
jjbeekman merged 1 commit into
mainfrom
boat-charging/AM-1175-add-cost-breakdown-type

Conversation

@jjbeekman

Copy link
Copy Markdown
Collaborator

Changes

Affected services

  • bridge

Definition of done

  • API is backwards compatible (make openapi-diff)
  • Swagger UI up-to-date & tested (make dev)
  • Dependencies updated (make requirements)
  • Infrastructure config updated (aapp_azure_infra)
  • Loadtests for relevant endpoints (aapp_testing_loadtests)

After PR created (and deployed on dev):

  • Manual check on dev (o) e.g. use feature via Swagger or admin panel
  • Sanity check on test app (check that dev environment is selected)

Other notes

GitHub Copilot was used in writing the code

@jjbeekman
jjbeekman requested review from a team and a lite review from Copilot August 24, 2026 08:15

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds support for STANDARD_FINE items in boat-charging cost breakdowns.

Changes:

  • Extends serializer choices with STANDARD_FINE.
  • Regression coverage for the new value is requested.

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread bridge/boat_charging/serializers/session_serializers.py
@jjbeekman
jjbeekman merged commit 21029be into main Aug 24, 2026
24 of 32 checks passed
@jjbeekman
jjbeekman deleted the boat-charging/AM-1175-add-cost-breakdown-type branch August 24, 2026 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants