-
Notifications
You must be signed in to change notification settings - Fork 24
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
PSP-7930 : Generate Document Hyperlink located within L/L File Summar… #4091
Conversation
…y Pane - H1005 - Database updates
✅ No secrets were detected in the code. |
✅ No secrets were detected in the code. |
# Conflicts: # source/database/mssql/scripts/dbscripts/PSP_PIMS_LATEST/Build/123_DML_PIMS_FORM_TYPE.sql # source/database/mssql/scripts/dbscripts/PSP_PIMS_S82_00/Alter Up/123_DML_PIMS_FORM_TYPE_Alter_Up.sql # source/database/mssql/scripts/dbscripts/PSP_PIMS_S82_00/Build/123_DML_PIMS_FORM_TYPE.sql
✅ No secrets were detected in the code. |
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.
When you merge this, you will need to run the alter down in dev, and then the alter up. Feel free to ask for help if you aren't confident with this.
|
||
IF @@ROWCOUNT = 1 | ||
BEGIN | ||
DELETE |
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.
Hmm, I feel like this will crash if a document has been uploaded with the H1005 form type code - probably need to clear that first before executing this.
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.
the script file is duplicated I'll delete it.
|
||
{hasClaim(Claims.LEASE_VIEW) && | ||
index === 0 && | ||
leaseTypeCode === ApiGen_CodeTypes_LeaseLicenceTypes.LIPPUBHWY && ( |
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.
no toString here but there is above?
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.
I could make the update to match, either way is fin and do cast to string.
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.
This story cannot be merged until a valid template is merged to https://confluence.th.gov.bc.ca/display/PIMS/Release+5.4
Page updated with template. |
✅ No secrets were detected in the code. |
✅ No secrets were detected in the code. |
✅ No secrets were detected in the code. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #4091 +/- ##
===========================================
- Coverage 85.40% 77.01% -8.40%
===========================================
Files 1621 493 -1128
Lines 122460 18052 -104408
Branches 9386 1238 -8148
===========================================
- Hits 104593 13903 -90690
+ Misses 17561 3843 -13718
Partials 306 306
Flags with carried forward coverage won't be shown. Click here to find out more.
|
…y Pane - H1005