-
Notifications
You must be signed in to change notification settings - Fork 0
Updates for 24.11 #746
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
Closed
Closed
Updates for 24.11 #746
Changes from all commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
846e595
Update TeamCity installer URL for docker hub
csebranek a6266b8
Update labkey npm dependencies and rehyrdate package-lock.json
csebranek b6eb062
explicitly add children as a type
csebranek 50a6703
remove placeholder in purchasing module, update react for wnprc_ehr, …
csebranek 464332a
update to react 18 for virology module, update build script for docke…
csebranek 7952492
update react types, update deprecated ReactDOM.render
csebranek 1d52b83
add test for breeding page
csebranek 3804fd6
new upload format working
csebranek f95b7c4
fix missing middle_name issue
csebranek e708739
change areas to access_level, remove employee number, fix notes/displ…
csebranek a59dc9d
remove old card info from unidentified cards page, add new cols
csebranek 7107b42
removed unused code
csebranek d8b5ec6
Merge branch '24.11_fb_labkey_update' into 24.11_fb_barrier_access_up…
csebranek 104c8f9
Mark modules as PostgreSQL-only (#691)
labkey-adam cfbe297
Remove SupportedDatabases property now that default is PostgreSQL-onl…
labkey-adam 3a10dcc
Additional details for necropsy requests (#688)
LeviCameron1 6b6c0fc
remove unused export (#697)
labkey-klum e74559e
No need to exclude Jakarta Activation API everywhere (#699)
labkey-adam a29cfaf
24.3 fb notification revamps (#685)
aschmidt34 c5729d5
Don't encode parameters passed to WebTestHelper.buildUrl (#704)
labkey-tchad 8f9759d
Updates to current blood draw queries and adding visual plot (#703)
csebranek ec535e7
Changes to automated blood charges (#710)
csebranek d09d9cc
24.3 fb notification revamps (#712)
aschmidt34 43806bd
sort current blood by room and cage first (#723)
csebranek 56e04fa
24.3 fb notification revamps (#720)
aschmidt34 2ade761
Blood draw scheduling improvement (#724)
csebranek 49cf2cc
Migrate <permissionClasses> -> <requiresPermissions> (#730)
labkey-adam 0bafa8a
24.3 fb water system changes (#722)
dnicolalde edec526
revert package.json with new package-lock.json
csebranek 09d9673
redo package updates with new lock file version
csebranek 3746570
Merge other fbs in to this main one
csebranek 5d86b96
fix broken links
csebranek 0aeea63
Merge branch '24.11_fb_homepage_updates' into 24.11_fb_labkey_update
csebranek File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Not sure what are the ^M character at the en of every line |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -4,4 +4,3 @@ Name: DBUtils | |
| RequiredServerVersion: 16.3 | ||
| SchemaVersion: 17.2 | ||
| ManageVersion: false | ||
| SupportedDatabases: pgsql | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,5 +1,5 @@ | ||
| <view xmlns="http://labkey.org/data/xml/view" title="ChemistryAnalyzer ETL Module"> | ||
| <permissionClasses> | ||
| <requiresPermissions> | ||
| <permissionClass name="org.labkey.api.security.permissions.ReadPermission"/> | ||
| </permissionClasses> | ||
| </requiresPermissions> | ||
| </view> |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. how are we dealing with people having multiple cards. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
23 changes: 23 additions & 0 deletions
23
WNPRC_Compliance/resources/schemas/dbscripts/postgresql/wnprc_compliance-25.000-25.001.sql
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,23 @@ | ||
| ALTER TABLE wnprc_compliance.card_info ADD COLUMN IF NOT EXISTS card_type text; | ||
| ALTER TABLE wnprc_compliance.card_info ADD COLUMN IF NOT EXISTS date_issued timestamp; | ||
| ALTER TABLE wnprc_compliance.card_info ADD COLUMN IF NOT EXISTS date_expire timestamp; | ||
| ALTER TABLE wnprc_compliance.card_info ADD COLUMN IF NOT EXISTS issue_code int; | ||
|
|
||
| ALTER TABLE wnprc_compliance.access_report_data RENAME TO access_report_data_old; | ||
|
|
||
|
|
||
| DROP TABLE IF EXISTS wnprc_compliance.access_report_data; | ||
| CREATE TABLE wnprc_compliance.access_report_data ( | ||
| report_id TEXT, | ||
| access_level TEXT, | ||
| card_id TEXT, | ||
|
|
||
| container entityid NOT NULL, | ||
| createdby userid, | ||
| created TIMESTAMP, | ||
| modifiedby userid, | ||
| modified TIMESTAMP, | ||
|
|
||
| CONSTRAINT PK_access_report_data_new PRIMARY KEY (report_id, access_level, card_id), | ||
| CONSTRAINT FK_access_report_data_access_reports_new FOREIGN KEY (report_id) REFERENCES wnprc_compliance.access_reports (report_id) | ||
| ); |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Could this be the problem with Jakarta we are having
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.
Snapshot is passing the test for this module