18761, 18763 - EFT statement settings updates #2659
Conversation
|
Note, a small pay-api update is coming in another PR for this ticket is coming to automatically enable notifications and populate admin as recipient if there are no current recipients. |
| import { StatementListItem, StatementNotificationSettings, StatementRecipient, StatementSettings } from '@/models/statement' | ||
| import { mapActions, mapState } from 'pinia' | ||
| import CommonUtils from '@/util/common-util' | ||
| import LaunchDarklyService from 'sbc-common-components/src/services/launchdarkly.services' |
There was a problem hiding this comment.
We should probably move this component from class-api/options-api into composition-api
Similar to Statements.vue
There was a problem hiding this comment.
Will take a look thanks
There was a problem hiding this comment.
Looks good, might be worth adding another point to the tickets (extra work done here)
|
Example for EFT
Non EFT as an Admin (should be the same as previous) |
|
/gcbrun |
|
Temporary Url for review: https://bcregistry-account-dev--pr-2659-nc5eu9he.web.app |
|
Kudos, SonarCloud Quality Gate passed!
|
|
/gcbrun |
|
Temporary Url for review: https://bcregistry-account-dev--pr-2659-nc5eu9he.web.app |
Issue #:
bcgov/entity#18761
bcgov/entity#18763
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).