-
Notifications
You must be signed in to change notification settings - Fork 2
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
ORV2 2299 LOA #1486
ORV2 2299 LOA #1486
Conversation
…outebc into ORV2-2299-loa-db-setup
# Conflicts: # database/mssql/scripts/versions/revert/v_31_ddl_revert.sql # database/mssql/scripts/versions/v_31_ddl.sql # database/mssql/test/versions/v_31_1_test.sql # database/mssql/test/versions/v_31_test.sh
…se when download mode is not proxy
… into ORV2-2299-loa-db-setup
|
||
@LogAsyncMethodExecution() | ||
async deleteLoaDocument(companyId: number, loaId: number): Promise<number> { | ||
const { affected } = await this.loaDetailRepository.update( |
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.
Please send the correct response message with success or failure instead of the affected row numbers
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.
@praju-aot Also the same should be done for async delete()
as well? Sending success/failure rather than affected row numbers
vehicles/src/modules/loa/dto/request/queryParam/get-loa.query-params.dto.ts
Outdated
Show resolved
Hide resolved
Stale image |
Quality Gate failed for 'onroutebc dops'Failed conditions |
Quality Gate failed for 'onroutebc vehicles'Failed conditions |
Description
This PR is for LOA DB set up and CRUD endpoints.
Fixes # (issue)
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist
Further comments
Thanks for the PR!
Deployments, as required, will be available below:
Please create PRs in draft mode. Mark as ready to enable:
After merge, new images are promoted to: