Skip to content

Conversation

@chenhongjing
Copy link
Collaborator

@chenhongjing chenhongjing commented Dec 2, 2025

Issue #: /bcgov/entity#31394

Description of changes:

  • Fix bug when a PPR amendment is submitted to update trust indenture, addTrustIndenture and removeTrustIndenture will incorrectly return in response. The ticket only describes one scenario, but this fix is to resolve all the scenarios.
  • Scenarios for trust indenture changes
Before Amendment After Amendment Current Response Expected Response
N N add + remove remove
N Y add + remove add
Y Y add + remove add
Y N add + remove remove
  • Update unit tests

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the PPR license (Apache 2.0).

…ndenture

Signed-off-by: Hongjing Chen <Hongjing.Chen@gov.bc.ca>
Signed-off-by: Hongjing Chen <Hongjing.Chen@gov.bc.ca>
@chenhongjing chenhongjing self-assigned this Dec 2, 2025
@chenhongjing chenhongjing marked this pull request as ready for review December 2, 2025 22:36
Copy link
Collaborator

@doug-lovett doug-lovett left a comment

Choose a reason for hiding this comment

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

Looks good, could you also increase the version in pyproject.toml.

Signed-off-by: Hongjing Chen <Hongjing.Chen@gov.bc.ca>
Copy link
Collaborator

@cameron-eyds cameron-eyds left a comment

Choose a reason for hiding this comment

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

Looks great!

@chenhongjing chenhongjing merged commit 03d6d72 into bcgov:main Dec 3, 2025
3 of 4 checks passed
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.

3 participants