Skip to content

API - Assign & Unassign - Allow on all statuses#681

Merged
shaangill025 merged 2 commits intobcgov:mainfrom
shaangill025:fix_26347_api
Apr 11, 2025
Merged

API - Assign & Unassign - Allow on all statuses#681
shaangill025 merged 2 commits intobcgov:mainfrom
shaangill025:fix_26347_api

Conversation

@shaangill025
Copy link
Copy Markdown
Contributor

Issue:

Description of changes:
Based on clarified requirement from @andyyanggov. Assign and Unassign need to allowed for other application statuses and not just FULL_REVIEW & PROVISIONAL_REVIEW.
#669 (comment)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the BC Registry and Digital Services BSD 3-Clause License

@shaangill025
Copy link
Copy Markdown
Contributor Author

@kris-daxiom This is ready and is a blocker for #669

Application.Status.FULL_REVIEW,
Application.Status.PROVISIONAL_REVIEW,
Application.Status.PROVISIONAL,
Application.Status.NOC_PENDING,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

For Payment Due, Paid applications the staff does not have any action and it should not be assigned to them when they cannot act on it

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

As shared in the teams chat, I think current UXA and business understanding is that all applications in the examiner list view should be assignable.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Payment due and Paid statuses removed

@kris-daxiom
Copy link
Copy Markdown
Contributor

Approving based on the comment bcgov/entity#26346 (comment)

…t draft

Signed-off-by: Shaanjot Gill <gill.shaanjots@gmail.com>
Signed-off-by: Shaanjot Gill <gill.shaanjots@gmail.com>
@sonarqubecloud
Copy link
Copy Markdown

@shaangill025 shaangill025 changed the title API - Assign & Unassign - Allow on all statuses execpt Draft API - Assign & Unassign - Allow on all statuses Apr 11, 2025
@shaangill025 shaangill025 merged commit 019d793 into bcgov:main Apr 11, 2025
10 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.

2 participants