-
Notifications
You must be signed in to change notification settings - Fork 16
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
Release/8.2.0 #1867
Merged
Merged
Release/8.2.0 #1867
Conversation
This file contains 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
…sk/RDMP-122-update-a-database-extraction
…sk/RDMP-122-update-a-database-extraction
…sk/RDMP-122-update-a-database-extraction
…sk/RDMP-122-update-a-database-extraction
…sk/RDMP-122-update-a-database-extraction
…sk/RDMP-122-update-a-database-extraction
…onToDatabaseMSSql.md
* Fix initial INSERTs of logging table * Only MSSQL needs integer-as-bool hack, sane DBs can use TRUE/FALSE keywords --------- Co-authored-by: James A Sutherland <>
…nto task/RDMP_190-offer-change-to-depricated-catalogue
…to-depricated-catalogue Task/rdmp 190 offer change to depricated catalogue
* start og groupby toggle working * ui toggles * add patch * fix versioning issue * add changelog * tidy up * attempt test fix * fix changelog * tidy up code * codeql * update patch number * ad dmissing file * update patch * updtae patch * add go * update patch
* working settings interface * add documentation * add missing files * update namespace * add icon * update from run * make key unique * add tests * update tests * update for tests * fix unique name issue * tidy up * add db change * init * fix merge * working build * interim * initial save working * add correct versioning * working ui * interim * better clone * tidy up code * working filtered list * tidy up ui * remove old code * add basic test * tidy up * add tests * interim ui tidyup * tidy up * tidy up * add skip * fix tests * add control * open new window * tidy up code * tidy up * tidy up * fix build * list cohort versions * add docs * updates from uat * tidy up * tidy up
* add keyboard shortcut * add .rdmp plugin handling * fix picker * working with RC * add todo note * tidy up * tidy up * add changelog * update build to pull .rdmp files * fix changelog * fix changelog
* WORKING TRIGGER * working update * fix build * better trigger * sort of working * fix clash * add null check * add todo note * add come notes * update changelog * attempt to fix tests * only add column once * tidy up code * move pk dropping * put pk in correct place * reorder * add check for trigger * better check for PK * start of index * working indexes * tidy up code * index tests * interim test * working triggers * allow nulls * fix patch * fix typo * tidy up * comment out test * re-add test * dispose of pipeline * tidy up * add more tests * update test * improved pk matching * fuix up tests * updates from review * tidy up tests
…nto RDMP-197-updates-from-8.2.0-release-testing
…-release-testing Rdmp 197 updates from 8.2.0 release testing
…nto release/8.2.0
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.
CodeQL found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.
bpeacock001
approved these changes
Jul 2, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Proposed Change
Release v8.2.0 feature branch into main RDMP for release
Type of change
What types of changes does your code introduce? Tick all that apply.
Checklist
By opening this PR, I confirm that I have: