-
Notifications
You must be signed in to change notification settings - Fork 14
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
#3258 - CAS Supplier Retry operation #4053
Conversation
sources/packages/backend/apps/api/src/services/cas-supplier/cas-supplier.service.ts
Outdated
Show resolved
Hide resolved
sources/packages/backend/apps/api/src/services/cas-supplier/cas-supplier.service.ts
Outdated
Show resolved
Hide resolved
sources/packages/backend/apps/api/src/services/cas-supplier/cas-supplier.service.ts
Outdated
Show resolved
Hide resolved
sources/packages/backend/apps/api/src/services/cas-supplier/cas-supplier.service.ts
Outdated
Show resolved
Hide resolved
sources/packages/backend/apps/api/src/services/cas-supplier/cas-supplier.service.ts
Outdated
Show resolved
Hide resolved
sources/packages/backend/apps/api/src/services/cas-supplier/cas-supplier.service.ts
Outdated
Show resolved
Hide resolved
sources/packages/backend/apps/api/src/services/cas-supplier/cas-supplier.service.ts
Show resolved
Hide resolved
sources/packages/backend/apps/api/src/services/cas-supplier/cas-supplier.service.ts
Outdated
Show resolved
Hide resolved
sources/packages/backend/apps/api/src/services/cas-supplier/cas-supplier.service.ts
Outdated
Show resolved
Hide resolved
sources/packages/web/src/views/aest/student/CASSupplierInformation.vue
Outdated
Show resolved
Hide resolved
sources/packages/web/src/views/aest/student/CASSupplierInformation.vue
Outdated
Show resolved
Hide resolved
sources/packages/web/src/views/aest/student/CASSupplierInformation.vue
Outdated
Show resolved
Hide resolved
...e-controllers/cas-supplier/_tests_/cas-supplier.aest.controller.retryCASSupplier.e2e-spec.ts
Show resolved
Hide resolved
...e-controllers/cas-supplier/_tests_/cas-supplier.aest.controller.retryCASSupplier.e2e-spec.ts
Outdated
Show resolved
Hide resolved
...e-controllers/cas-supplier/_tests_/cas-supplier.aest.controller.retryCASSupplier.e2e-spec.ts
Outdated
Show resolved
Hide resolved
...e-controllers/cas-supplier/_tests_/cas-supplier.aest.controller.retryCASSupplier.e2e-spec.ts
Outdated
Show resolved
Hide resolved
...e-controllers/cas-supplier/_tests_/cas-supplier.aest.controller.retryCASSupplier.e2e-spec.ts
Outdated
Show resolved
Hide resolved
...e-controllers/cas-supplier/_tests_/cas-supplier.aest.controller.retryCASSupplier.e2e-spec.ts
Outdated
Show resolved
Hide resolved
/** | ||
* Creates a new CAS Supplier for a student. | ||
* @param studentId student id. | ||
* @param auditUser user. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rename to auditUserId
.
* Creates a new CAS Supplier for a student. | ||
* @param studentId student id. | ||
* @param auditUser user. | ||
* @param entityManager optional entity manager. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Adjust to options.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for making the changes, look good 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work 👍 Thanks for making the changes.
Retry button is disabled, if the student already has a supplier in Pending supplier verification.

Retry enabled when student does not have a Pending supplier verification.

Retry will create a new record in cas_suppliers table with status as PendingSupplierVerification.