Skip to content

32321 - change_of_liquidators filing processor - fix missing offices#4047

Merged
deetz99 merged 1 commit intobcgov:mainfrom
deetz99:32321-liquidators-missing-offices-fix
Feb 4, 2026
Merged

32321 - change_of_liquidators filing processor - fix missing offices#4047
deetz99 merged 1 commit intobcgov:mainfrom
deetz99:32321-liquidators-missing-offices-fix

Conversation

@deetz99
Copy link
Collaborator

@deetz99 deetz99 commented Feb 4, 2026

Issue #: /bcgov/entity#32321

update_or_create_offices expects a dict for the offices.

update_or_create_offices(business: Business, offices: dict[str, dict[str, dict[str, str]]])

During a changeAddressLiquidator, the user may only change a party address and not the liquidationRecordsOffice, in this case there would be no offices submitted in the filing payload.

Description of changes:

  • add default value to offices to prevent attribute error
  • update test

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

Copy link
Collaborator

@kialj876 kialj876 left a comment

Choose a reason for hiding this comment

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

Thanks!

@deetz99 deetz99 merged commit fb9d48d into bcgov:main Feb 4, 2026
5 of 6 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.

4 participants