18238 Applied director/party middleInitial/middleName workaround to all filings #541
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- moved director workaround to LegalServices - moved parties workaround to LegalServices - removed workaround from Change - removed workaround from Conversion - updated unit tests
severinbeauvais
force-pushed
the
18238
branch
from
58e3525 to
e130ce8
Compare
severinbeauvais
commented
Nov 8, 2023
| { appointmentDate: '2022-04-01' }, | ||
| { appointmentDate: '2022-05-01' } | ||
| { appointmentDate: '2022-04-01', officer: {} }, | ||
| { appointmentDate: '2022-05-01', officer: {} } |
There was a problem hiding this comment.
Needed to be able to de-reference director.officer.
Ditto below for party.
severinbeauvais
commented
Nov 8, 2023
| if (middleInitial !== undefined) { | ||
| director.officer.middleName = middleInitial | ||
| delete director.officer['middleInitial'] | ||
| } |
There was a problem hiding this comment.
It turns out this workaround is needed for Alteration / LimitedRestorationExtension / LimitedRestorationToFull / SpecialResolution filings.
There was a problem hiding this comment.
Sample alteration:
severinbeauvais
commented
Nov 8, 2023
| delete party.officer['middleInitial'] | ||
| } | ||
| return party | ||
| }) |
There was a problem hiding this comment.
It turns out this workaround is needed for Change / Conversion / CoopCorrection / CorpCorrection / FirmCorrection filings.
It was already implemented for Change and Conversion so I removed it from there.
There was a problem hiding this comment.
Sample change:
There was a problem hiding this comment.
Sample conversion:
There was a problem hiding this comment.
Sample coop correction:
There was a problem hiding this comment.
Sample corp correction:
There was a problem hiding this comment.
Sample firm correction:
severinbeauvais
requested review from
seeker25,
jamespaologarcia,
PaulGarewal,
tshyun24 and
leodube-aot
|
/gcbrun |
|
Temporary Url for review: https://business-edit-dev--pr-541-pu6ujz6g.web.app |
severinbeauvais
changed the title
|
Ping! Could you guys please review this PR? Thanks! |
PaulGarewal
approved these changes
Nov 9, 2023
JazzarKarim
pushed a commit
to JazzarKarim/business-edit-ui
that referenced
this pull request
Jan 26, 2024
- moved director workaround to LegalServices - moved parties workaround to LegalServices - removed workaround from Change - removed workaround from Conversion - updated unit tests
JazzarKarim
added a commit
that referenced
this pull request
Jan 26, 2024
* Add small rendering delay for Firefox, that way the review and confirm scrolls to the top of the screen. (#535) * 17347 - Add in directors for corrections for CP. (#536) * Add in directors for corrections for CP. * Fix unit tests for director change for CP. * Add in COOP. * Add in appointmentDate for co-op corrections. (#537) * Bump postcss from 8.4.30 to 8.4.31 (#533) Bumps [postcss](https://github.com/postcss/postcss) from 8.4.30 to 8.4.31. - [Release notes](https://github.com/postcss/postcss/releases) - [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md) - [Commits](postcss/postcss@8.4.30...8.4.31) --- updated-dependencies: - dependency-name: postcss dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * - app version = 4.7.5 (#541) - moved director workaround to LegalServices - moved parties workaround to LegalServices - removed workaround from Change - removed workaround from Conversion - updated unit tests * - app version = 4.7.6 (#542) - deleted obsolete "alteration-ui-enabled" FF - deleted obsolete "change-ui-enabled" FF - deleted obsolete "conversion-ui-enabled" FF - removed flag usage from Alteration.vue - removed flag usage from Change.vue - removed flag usage from Conversion.vue - removed flag usage from SpecialResolution.vue * - app version = 4.7.7 (#543) - uninstalled v-sanitize - installed vue-sanitize-directive - updated imports - added "basic" option (same as default with old package) * 17158 update fetchNameRequest (#540) * 17158 update fetchNameRequest * add debug code * add debug --wip * add debug --wip * update error handling * update NR status * update version * 18277 - Don't pre select no fee option for co-op correction. (#539) * Don't pre select no fee option for co-op correction. * Set to FAS, this way it will display $20. * Update package version. * Add in dropdown option for BEN when alterating from ULC (#545) * add in dropdown option for BEN when going from ULC * Name request check error update * Update comment and package versions * 18716 added phone number and email address to name request correction… (#547) * 18716 added phone number and email address to name request correction on file and pay * updated package json * removed debug code * 18728 - COOP Bug: Payment shows up as #3 when it should be #5 (#548) * fix csr index bug * 4.7.11 * 18735 - Fix rules showing up as being changed, when it's on paper (#549) * Fix rules being changed, when it's on paper * Fix bad code * add in extra || null * 18622 - Correction - Blank Special Resolution (#550) * 18495 - Coop Correction Changes when the Resolution is Absent (#551) * 18495 - Bug Fix - Show Resolution Section when not Correction Filing (#552) * 19069 - Special Resolution - Validation on blank association type (#553) * Bump vite from 4.3.9 to 4.5.2 (#555) Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 4.3.9 to 4.5.2. - [Release notes](https://github.com/vitejs/vite/releases) - [Changelog](https://github.com/vitejs/vite/blob/v4.5.2/packages/vite/CHANGELOG.md) - [Commits](https://github.com/vitejs/vite/commits/v4.5.2/packages/vite) --- updated-dependencies: - dependency-name: vite dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * major package version update --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Travis Semple <travis8814@gmail.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Séverin Beauvais <severinbeauvais@users.noreply.github.com> Co-authored-by: Kevin Zhang <54437031+kzdev420@users.noreply.github.com> Co-authored-by: Paul <144158015+PaulGarewal@users.noreply.github.com> Co-authored-by: Jia Xu <Jia.Xu@gov.bc.ca>
JazzarKarim
added a commit
to JazzarKarim/business-edit-ui
that referenced
this pull request
Jan 31, 2024
* Add small rendering delay for Firefox, that way the review and confirm scrolls to the top of the screen. (bcgov#535) * 17347 - Add in directors for corrections for CP. (bcgov#536) * Add in directors for corrections for CP. * Fix unit tests for director change for CP. * Add in COOP. * Add in appointmentDate for co-op corrections. (bcgov#537) * Bump postcss from 8.4.30 to 8.4.31 (bcgov#533) Bumps [postcss](https://github.com/postcss/postcss) from 8.4.30 to 8.4.31. - [Release notes](https://github.com/postcss/postcss/releases) - [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md) - [Commits](postcss/postcss@8.4.30...8.4.31) --- updated-dependencies: - dependency-name: postcss dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * - app version = 4.7.5 (bcgov#541) - moved director workaround to LegalServices - moved parties workaround to LegalServices - removed workaround from Change - removed workaround from Conversion - updated unit tests * - app version = 4.7.6 (bcgov#542) - deleted obsolete "alteration-ui-enabled" FF - deleted obsolete "change-ui-enabled" FF - deleted obsolete "conversion-ui-enabled" FF - removed flag usage from Alteration.vue - removed flag usage from Change.vue - removed flag usage from Conversion.vue - removed flag usage from SpecialResolution.vue * - app version = 4.7.7 (bcgov#543) - uninstalled v-sanitize - installed vue-sanitize-directive - updated imports - added "basic" option (same as default with old package) * 17158 update fetchNameRequest (bcgov#540) * 17158 update fetchNameRequest * add debug code * add debug --wip * add debug --wip * update error handling * update NR status * update version * 18277 - Don't pre select no fee option for co-op correction. (bcgov#539) * Don't pre select no fee option for co-op correction. * Set to FAS, this way it will display $20. * Update package version. * Add in dropdown option for BEN when alterating from ULC (bcgov#545) * add in dropdown option for BEN when going from ULC * Name request check error update * Update comment and package versions * 18716 added phone number and email address to name request correction… (bcgov#547) * 18716 added phone number and email address to name request correction on file and pay * updated package json * removed debug code * 18728 - COOP Bug: Payment shows up as bcgov#3 when it should be bcgov#5 (bcgov#548) * fix csr index bug * 4.7.11 * 18735 - Fix rules showing up as being changed, when it's on paper (bcgov#549) * Fix rules being changed, when it's on paper * Fix bad code * add in extra || null * 18622 - Correction - Blank Special Resolution (bcgov#550) * 18495 - Coop Correction Changes when the Resolution is Absent (bcgov#551) * 18495 - Bug Fix - Show Resolution Section when not Correction Filing (bcgov#552) * 19069 - Special Resolution - Validation on blank association type (bcgov#553) * Bump vite from 4.3.9 to 4.5.2 (bcgov#555) Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 4.3.9 to 4.5.2. - [Release notes](https://github.com/vitejs/vite/releases) - [Changelog](https://github.com/vitejs/vite/blob/v4.5.2/packages/vite/CHANGELOG.md) - [Commits](https://github.com/vitejs/vite/commits/v4.5.2/packages/vite) --- updated-dependencies: - dependency-name: vite dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * major package version update --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Travis Semple <travis8814@gmail.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Séverin Beauvais <severinbeauvais@users.noreply.github.com> Co-authored-by: Kevin Zhang <54437031+kzdev420@users.noreply.github.com> Co-authored-by: Paul <144158015+PaulGarewal@users.noreply.github.com> Co-authored-by: Jia Xu <Jia.Xu@gov.bc.ca>
JazzarKarim
added a commit
to JazzarKarim/business-edit-ui
that referenced
this pull request
Feb 1, 2024
* Add small rendering delay for Firefox, that way the review and confirm scrolls to the top of the screen. (bcgov#535) * 17347 - Add in directors for corrections for CP. (bcgov#536) * Add in directors for corrections for CP. * Fix unit tests for director change for CP. * Add in COOP. * Add in appointmentDate for co-op corrections. (bcgov#537) * Bump postcss from 8.4.30 to 8.4.31 (bcgov#533) Bumps [postcss](https://github.com/postcss/postcss) from 8.4.30 to 8.4.31. - [Release notes](https://github.com/postcss/postcss/releases) - [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md) - [Commits](postcss/postcss@8.4.30...8.4.31) --- updated-dependencies: - dependency-name: postcss dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * - app version = 4.7.5 (bcgov#541) - moved director workaround to LegalServices - moved parties workaround to LegalServices - removed workaround from Change - removed workaround from Conversion - updated unit tests * - app version = 4.7.6 (bcgov#542) - deleted obsolete "alteration-ui-enabled" FF - deleted obsolete "change-ui-enabled" FF - deleted obsolete "conversion-ui-enabled" FF - removed flag usage from Alteration.vue - removed flag usage from Change.vue - removed flag usage from Conversion.vue - removed flag usage from SpecialResolution.vue * - app version = 4.7.7 (bcgov#543) - uninstalled v-sanitize - installed vue-sanitize-directive - updated imports - added "basic" option (same as default with old package) * 17158 update fetchNameRequest (bcgov#540) * 17158 update fetchNameRequest * add debug code * add debug --wip * add debug --wip * update error handling * update NR status * update version * 18277 - Don't pre select no fee option for co-op correction. (bcgov#539) * Don't pre select no fee option for co-op correction. * Set to FAS, this way it will display $20. * Update package version. * Add in dropdown option for BEN when alterating from ULC (bcgov#545) * add in dropdown option for BEN when going from ULC * Name request check error update * Update comment and package versions * 18716 added phone number and email address to name request correction… (bcgov#547) * 18716 added phone number and email address to name request correction on file and pay * updated package json * removed debug code * 18728 - COOP Bug: Payment shows up as bcgov#3 when it should be bcgov#5 (bcgov#548) * fix csr index bug * 4.7.11 * 18735 - Fix rules showing up as being changed, when it's on paper (bcgov#549) * Fix rules being changed, when it's on paper * Fix bad code * add in extra || null * 18622 - Correction - Blank Special Resolution (bcgov#550) * 18495 - Coop Correction Changes when the Resolution is Absent (bcgov#551) * 18495 - Bug Fix - Show Resolution Section when not Correction Filing (bcgov#552) * 19069 - Special Resolution - Validation on blank association type (bcgov#553) * Bump vite from 4.3.9 to 4.5.2 (bcgov#555) Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 4.3.9 to 4.5.2. - [Release notes](https://github.com/vitejs/vite/releases) - [Changelog](https://github.com/vitejs/vite/blob/v4.5.2/packages/vite/CHANGELOG.md) - [Commits](https://github.com/vitejs/vite/commits/v4.5.2/packages/vite) --- updated-dependencies: - dependency-name: vite dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * major package version update --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Travis Semple <travis8814@gmail.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Séverin Beauvais <severinbeauvais@users.noreply.github.com> Co-authored-by: Kevin Zhang <54437031+kzdev420@users.noreply.github.com> Co-authored-by: Paul <144158015+PaulGarewal@users.noreply.github.com> Co-authored-by: Jia Xu <Jia.Xu@gov.bc.ca>
JazzarKarim
added a commit
to JazzarKarim/business-edit-ui
that referenced
this pull request
Feb 22, 2024
* Add small rendering delay for Firefox, that way the review and confirm scrolls to the top of the screen. (bcgov#535) * 17347 - Add in directors for corrections for CP. (bcgov#536) * Add in directors for corrections for CP. * Fix unit tests for director change for CP. * Add in COOP. * Add in appointmentDate for co-op corrections. (bcgov#537) * Bump postcss from 8.4.30 to 8.4.31 (bcgov#533) Bumps [postcss](https://github.com/postcss/postcss) from 8.4.30 to 8.4.31. - [Release notes](https://github.com/postcss/postcss/releases) - [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md) - [Commits](postcss/postcss@8.4.30...8.4.31) --- updated-dependencies: - dependency-name: postcss dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * - app version = 4.7.5 (bcgov#541) - moved director workaround to LegalServices - moved parties workaround to LegalServices - removed workaround from Change - removed workaround from Conversion - updated unit tests * - app version = 4.7.6 (bcgov#542) - deleted obsolete "alteration-ui-enabled" FF - deleted obsolete "change-ui-enabled" FF - deleted obsolete "conversion-ui-enabled" FF - removed flag usage from Alteration.vue - removed flag usage from Change.vue - removed flag usage from Conversion.vue - removed flag usage from SpecialResolution.vue * - app version = 4.7.7 (bcgov#543) - uninstalled v-sanitize - installed vue-sanitize-directive - updated imports - added "basic" option (same as default with old package) * 17158 update fetchNameRequest (bcgov#540) * 17158 update fetchNameRequest * add debug code * add debug --wip * add debug --wip * update error handling * update NR status * update version * 18277 - Don't pre select no fee option for co-op correction. (bcgov#539) * Don't pre select no fee option for co-op correction. * Set to FAS, this way it will display $20. * Update package version. * Add in dropdown option for BEN when alterating from ULC (bcgov#545) * add in dropdown option for BEN when going from ULC * Name request check error update * Update comment and package versions * 18716 added phone number and email address to name request correction… (bcgov#547) * 18716 added phone number and email address to name request correction on file and pay * updated package json * removed debug code * 18728 - COOP Bug: Payment shows up as bcgov#3 when it should be bcgov#5 (bcgov#548) * fix csr index bug * 4.7.11 * 18735 - Fix rules showing up as being changed, when it's on paper (bcgov#549) * Fix rules being changed, when it's on paper * Fix bad code * add in extra || null * 18622 - Correction - Blank Special Resolution (bcgov#550) * 18495 - Coop Correction Changes when the Resolution is Absent (bcgov#551) * 18495 - Bug Fix - Show Resolution Section when not Correction Filing (bcgov#552) * 19069 - Special Resolution - Validation on blank association type (bcgov#553) * Bump vite from 4.3.9 to 4.5.2 (bcgov#555) Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 4.3.9 to 4.5.2. - [Release notes](https://github.com/vitejs/vite/releases) - [Changelog](https://github.com/vitejs/vite/blob/v4.5.2/packages/vite/CHANGELOG.md) - [Commits](https://github.com/vitejs/vite/commits/v4.5.2/packages/vite) --- updated-dependencies: - dependency-name: vite dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * major package version update --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Travis Semple <travis8814@gmail.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Séverin Beauvais <severinbeauvais@users.noreply.github.com> Co-authored-by: Kevin Zhang <54437031+kzdev420@users.noreply.github.com> Co-authored-by: Paul <144158015+PaulGarewal@users.noreply.github.com> Co-authored-by: Jia Xu <Jia.Xu@gov.bc.ca>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue #: bcgov/entity#18238
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the bcrs-entities-create-ui license (Apache 2.0).