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.

Sign up for GitHub

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

Jump to bottom

Additional AML Status options #3689

Closed
AshDeeming opened this issue Mar 9, 2021 · 1 comment · Fixed by #3922, #3930 or #3945
Closed

Additional AML Status options #3689

AshDeeming opened this issue Mar 9, 2021 · 1 comment · Fixed by #3922, #3930 or #3945
Assignees
@willmcvay
Labels
aml-checklist Relates to Anti-money laundering app feature New feature or request front-end Relates to front end issues

Comments

@AshDeeming
Copy link

AshDeeming commented Mar 9, 2021 •
edited by HollyJoyPhillips
Loading

Summary: Currently AML only supports 1 status update, to set the ID Check as 'Successful'. However, there are occasions that ID Checks maybe, in progress or have failed.

We should look to support additional status handling when updating the status of a record.

Task:

  • When clicking on 'Update Status', provide a check box list of available status' to select
  • Statuses:
    (pass/fail/pending/cancelled/warnings/unchecked)
    Should only be able to select one
  • Default on current status
  • Button should be 'SAVE'

Mock Up:
image

Ticket raised following FoundationsSupport email based on Support case:
https://reapit.atlassian.net/browse/RSD-110430
@AshDeeming AshDeeming added bug Something isn't working aml-checklist Relates to Anti-money laundering app front-end Relates to front end issues labels Mar 9, 2021
@AshDeeming AshDeeming added feature New feature or request and removed bug Something isn't working labels Mar 10, 2021
@HollyJoyPhillips HollyJoyPhillips changed the title Additional AML Status options missing Additional AML Status options Mar 10, 2021
@willmcvay willmcvay self-assigned this Apr 9, 2021
willmcvay added a commit that referenced this issue Apr 9, 2021
@willmcvay
feat: #3689 #3812 #3913 aml app fixes
3de83b2
@willmcvay willmcvay mentioned this issue Apr 9, 2021
Merged
willmcvay added a commit that referenced this issue Apr 9, 2021
@willmcvay
feat: #3689 #3812 #3913 aml app fixes (#3922)
44fdc5e
willmcvay added a commit that referenced this issue Apr 12, 2021
@willmcvay
fix: #3689 aml status update
802f895
@willmcvay willmcvay mentioned this issue Apr 12, 2021
Merged
willmcvay added a commit that referenced this issue Apr 12, 2021
@willmcvay
fix: #3689 aml status update (#3930)
e655734
@HollyJoyPhillips
Copy link
Contributor

@willmcvay Can you remove 'Warnings' as a status option please as it's used for custom config and requires additional backend support.

willmcvay added a commit that referenced this issue Apr 13, 2021
@willmcvay
fix: #3689 #3928 #3942 aml and dev portal fixes
0990681
@willmcvay willmcvay mentioned this issue Apr 13, 2021
Merged
willmcvay added a commit that referenced this issue Apr 13, 2021
@willmcvay
fix: #3689 #3928 #3942 aml and dev portal fixes (#3945)
ef91689
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@willmcvay willmcvay

Labels
aml-checklist Relates to Anti-money laundering app feature New feature or request front-end Relates to front end issues
Projects
None yet
Milestone
No milestone
3 participants
@willmcvay @HollyJoyPhillips @AshDeeming