Make Certify block non-editable if non-staff user (AGM Filings) (#564)

Make certify block non-editable if non-staff user
bcgov · Nov 2, 2023 · b93ab40 · b93ab40
1 parent 9b3a4e8
commit b93ab40
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 7 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
  "name": "business-filings-ui",
- "version": "6.9.5",
+ "version": "6.9.6",
  "private": true,
  "appName": "Filings UI",
  "sbcName": "SBC Common Components",

diff --git a/src/views/AgmExtension.vue b/src/views/AgmExtension.vue
@@ -97,6 +97,7 @@
  :isCertified.sync="isCertified"
  :certifiedBy.sync="certifiedBy"
  :class="{ 'invalid-certify': !certifyFormValid && showErrors }"
+ :disableEdit="!isRoleStaff"
  :entityDisplay="displayName()"
  :message="certifyText(FilingCodes.AGM_EXTENSION)"
  @valid="certifyFormValid=$event"

diff --git a/src/views/AgmLocationChg.vue b/src/views/AgmLocationChg.vue
@@ -192,6 +192,7 @@
  :isCertified.sync="isCertified"
  :certifiedBy.sync="certifiedBy"
  :class="{ 'invalid-certify': !certifyFormValid && showErrors }"
+ :disableEdit="!isRoleStaff"
  :entityDisplay="displayName()"
  :message="certifyText(FilingCodes.AGM_LOCATION_CHANGE)"
  @valid="certifyFormValid=$event"
@@ -419,7 +420,7 @@ export default class AgmLocationChg extends Mixins(CommonMixin, DateMixin,
  mounted (): void {
  // Pre-populate the certified block with the logged in user's name (if not staff)
  if (!this.isRoleStaff && this.getUserInfo) {
- this.certifiedBy = this.getUserInfo.firstname + ' ' + this.getUserInfo.lastName
+ this.certifiedBy = this.getUserInfo.firstname + ' ' + this.getUserInfo.lastname
  }
 
  // always include agm location change code

diff --git a/src/views/ConsentContinuationOut.vue b/src/views/ConsentContinuationOut.vue
@@ -507,9 +507,7 @@ export default class ConsentContinuationOut extends Mixins(CommonMixin, DateMixi
 
  // Pre-populate the certified block with the logged in user's name (if not staff)
  if (!this.isRoleStaff && this.getUserInfo) {
- const firstName = this.getUserInfo?.firstname
- const lastName = this.getUserInfo?.lastname
- this.certifiedBy = firstName + ' ' + lastName
+ this.certifiedBy = this.getUserInfo.firstname + ' ' + this.getUserInfo.lastname
  }
 
  // always include consent continue out code