17534 - New NSF Resource Implementation and NSF Invoice PDF Download #2658
Conversation
auth-web/src/components/auth/account-freeze/AccountOverview.vue
Outdated
Show resolved
Hide resolved
|
|
@rodrigo-barraza looks like we might need another version bump |
| const calculateFailedInvoices: any = orgStore.calculateFailedInvoices | ||
| const downloadNSFInvoicesPDF: any = orgStore.downloadNSFInvoicesPDF | ||
| const formatDate = CommonUtils.formatDisplayDate | ||
| const suspendedDate = (currentOrganization?.suspendedOn) ? formatDate(new Date(currentOrganization.suspendedOn)) : '' |
There was a problem hiding this comment.
You're going to run into date issues with timezones here, suggestion use Moment instead I think it's already included
There was a problem hiding this comment.
moment(currentOrganization.suspendedOn)
There was a problem hiding this comment.
formatDate is already a moment instance, so the dates should be fine. formatDate needs to be passed a date object, this is why it's new Date.
| emits: ['step-forward'], | ||
| setup (_, { emit }) { | ||
| const orgStore = useOrgStore() | ||
| const currentOrganization: Organization = orgStore.currentOrganization |
There was a problem hiding this comment.
are these reactive? or does it not matter?
There was a problem hiding this comment.
Moved currentOrganization and currentMembership to reactive
|
package.json and package-lock update, then we'll merge |
|
*Issue #:17534 *
bcgov/entity#17534
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 sbc-auth license (Apache 2.0).