Skip to content
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

20138 - Suspended EFT Account Views #2827

Merged
merged 9 commits into from
May 28, 2024

Conversation

rodrigo-barraza
Copy link
Collaborator

Issue #:
bcgov/entity#20138
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).

@seeker25
Copy link
Collaborator

seeker25 commented May 15, 2024

might be worth adding a unit test or two more.. Also can this be deployed to PROD today? or does it have breaking changes from the existing flows?

@@ -114,6 +114,7 @@ export enum AccountStatus {
PENDING_STAFF_REVIEW = 'PENDING_STAFF_REVIEW',
PENDING_ACTIVATION = 'PENDING_ACTIVATION',
NSF_SUSPENDED = 'NSF_SUSPENDED',
EFT_SUSPENDED = 'EFT_SUSPENDED',
Copy link
Collaborator

@seeker25 seeker25 May 27, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we're not adding in another code..

Probably just use SUSPENDED I think that's what @Jxio wanted to do?

I was thinking just use NSF_SUSPENDED and then fill in Overdue EFT and NSF PAD payments

image

But I think if we used NSF_SUSPENDED and SUSPENDED that would work too

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

talk to @Jxio about this one

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

Copy link
Collaborator

@seeker25 seeker25 May 27, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Jxio we'll need a status for PAD NSF

Nvm I see what you did now.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Jxio the problem with using suspended though, there's an entire flow in sbc-common-components that looks for nsf_suspended and not suspended?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we only used nsf_suspended for eft currently.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah maybe just use nsf_suspended then, we'll need another field in the suspension_reason_codes to differentiate between PAD NSF and EFT NSF?

Copy link
Collaborator

@seeker25 seeker25 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, just some small stuff to address

Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
2 Security Hotspots
3.7% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

@rodrigo-barraza rodrigo-barraza merged commit 281ff67 into bcgov:main May 28, 2024
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants