Skip to content

Commit

Permalink
Merge pull request #594 from eason-pan-bc/23971-allow-editable-certif…
Browse files Browse the repository at this point in the history
…y-legal-name

23971 - allow editable certify legal name for corps as regular users
  • Loading branch information
eason-pan-bc authored Nov 2, 2024
2 parents eb99042 + e14e83f commit bc543b2
Show file tree
Hide file tree
Showing 6 changed files with 7 additions and 9 deletions.
4 changes: 2 additions & 2 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "business-edit-ui",
"version": "4.11.1",
"version": "4.11.2",
"private": true,
"appName": "Edit UI",
"sbcName": "SBC Common Components",
Expand Down
1 change: 0 additions & 1 deletion src/views/Alteration.vue
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,6 @@
class="mt-10"
:sectionNumber="showTransactionalFolioNumber ? '3.' : '2.'"
:validate="getAppValidate"
:disableEdit="!isRoleStaff"
/>

<!-- STAFF ONLY: Court Order/Plan of Arrangement and Staff Payment -->
Expand Down
1 change: 0 additions & 1 deletion src/views/SpecialResolution.vue
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,6 @@
class="mt-10"
:sectionNumber="showTransactionalFolioNumber ? '4.' : '3.'"
:validate="getAppValidate"
:disableEdit="!isRoleStaff"
/>

<template v-if="isRoleStaff">
Expand Down
4 changes: 2 additions & 2 deletions tests/unit/Alteration.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -413,7 +413,7 @@ describe('Alteration component', () => {
expect(store.stateModel.filingData[0].filingTypeCode).toBe(FilingCodes.ALTERATION_BC_TO_ULC)
})

it('certify text is not prefilled/editable for staff user', async () => {
it('certify text is not prefilled for staff user', async () => {
store.stateModel.tombstone.keycloakRoles = ['staff']
store.stateModel.tombstone.userInfo = {
firstname: 'Jon',
Expand All @@ -425,7 +425,7 @@ describe('Alteration component', () => {
expect(store.stateModel.certifyState.certifiedBy).toBe('undefined undefined')
})

it('certify text is prefilled/uneditable for non-staff user', async () => {
it('certify text is prefilled for non-staff user', async () => {
store.stateModel.tombstone.keycloakRoles = []
store.stateModel.tombstone.userInfo = {
firstname: 'Jon',
Expand Down
4 changes: 2 additions & 2 deletions tests/unit/SpecialResolution.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -278,7 +278,7 @@ describe('Special Resolution component', () => {
expect(store.stateModel.currentFees[0].filingFees).toBe(70)
})

it('certify text is not prefilled/editable for staff user', async () => {
it('certify text is not prefilled for staff user', async () => {
store.stateModel.tombstone.keycloakRoles = ['staff']
store.stateModel.tombstone.userInfo = {
firstname: 'Jon',
Expand All @@ -290,7 +290,7 @@ describe('Special Resolution component', () => {
expect(store.stateModel.certifyState.certifiedBy).toBe('undefined undefined')
})

it('certify text is prefilled/uneditable for non-staff user', async () => {
it('certify text is prefilled for non-staff user', async () => {
store.stateModel.tombstone.keycloakRoles = []
store.stateModel.tombstone.userInfo = {
firstname: 'Jon',
Expand Down

0 comments on commit bc543b2

Please sign in to comment.