-
Notifications
You must be signed in to change notification settings - Fork 69
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
24576 - Updates to NSF messaging #3207
Conversation
@State(useOrgStore) private currentOrganization!: Organization | ||
@State(useCodesStore) private suspensionReasonCodes!: Code[] | ||
private formatDate = CommonUtils.formatDisplayDate | ||
export default defineComponent({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you looks good
auth-web/src/components/auth/account-freeze/AccountOverviewNSF.vue
Outdated
Show resolved
Hide resolved
auth-web/src/util/common-util.ts
Outdated
@@ -148,6 +148,11 @@ export default class CommonUtils { | |||
static formatDatePickerDate (date?: Date) { | |||
return moment(date || new Date()).format('YYYY-MM-DD') | |||
} | |||
|
|||
static formatDateToHumanReadable (date: Date | string | moment.Moment) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
type DateLike = string | Date | moment.Moment
date: DateLike
can replace line 141 too like this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -21,7 +21,7 @@ | |||
Account Suspended | |||
</div> | |||
<div> | |||
Account has been suspended for {{ accountSuspendReason }}. | |||
This acount has been suspended. Returned system error message: {{ reason }}. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
acount
@@ -85,16 +85,9 @@ | |||
{{ currentOrganization.name }} | |||
</h1> | |||
<p | |||
v-if="isPremiumAccount" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aye good stuff
Quality Gate passedIssues Measures |
Issue #:
bcgov/entity#24576
Description of changes:
Added reason to client view NSF when account is suspended.
Added reason to admin view when viewing a suspended account.
Updated NSF copy and icons.
Cleaned up date formats to be more human-readable.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the sbc-auth license (Apache 2.0).