Skip to content

29073 - update specs match validation updates #232

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

Merged

Conversation

eason-pan-bc
Copy link
Contributor

@eason-pan-bc eason-pan-bc commented Jun 20, 2025

Issue #: /bcgov/entity#29073

Description of changes:
Updated the specs based on the latest updates on legal-api sandbox validation updates

Mismatch Type Field / Sub Field Affected Filings
Inconsistent field type & length check Contact info -> Phone number IA, Amalgamation, Cont. in, Alteration
extension IA, Amalgamation, Cont. in, Alteration
proof of authorization -> max number of files Cont. in
Inconsistent required fields recordsOffice IA, Amalgamation, Cont. in, AR
parties -> officer -> firstName IA, Amalgamation
directors -> mailingAddress & deliveryAddress CoD, AR

Phone number & Extension

  • Add description for maximum digits each field takes
    image

Max 5 files for Cont. in

image

recordsOffice is required

image

add firstName and lastName to Person schema

image

both mailingAddress and deliveryAddress of a director are required for CoD and AR

image

@argush3
Copy link
Collaborator

argush3 commented Jun 20, 2025

/gcbrun

@bcregistry-sre
Copy link
Collaborator

Temporary Url for review: https://developer-connect-dev--pr-232-ddf5zn5v.web.app

@eason-pan-bc
Copy link
Contributor Author

eason-pan-bc commented Jun 20, 2025

updated some screenshots in the PR description

@argush3
Copy link
Collaborator

argush3 commented Jun 20, 2025

/gcbrun

@bcregistry-sre
Copy link
Collaborator

Temporary Url for review: https://developer-connect-dev--pr-232-ddf5zn5v.web.app

@argush3
Copy link
Collaborator

argush3 commented Jun 20, 2025

/gcbrun

@bcregistry-sre
Copy link
Collaborator

Temporary Url for review: https://developer-connect-dev--pr-232-ddf5zn5v.web.app

@argush3 argush3 merged commit 0422926 into bcgov:main Jun 20, 2025
8 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