generated from IHE/supplement-template
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Update R1.4 Branch to Latest master #314
Closed
Closed
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
β¦#300) * #152 Add discovery proxy transactions scaffold and actor description. * #152 Add more content; add ack message to transaction sequence. * #152 Tweak register message text * #152 Add DEV-47, apply tweaks von DEV-46 * #152 Add DEV-47 probe and resolve * #152 Various fixes * #152 Various fixes * #152 Add MDPWS binding contents for DEV-46 * #152 Add MDPWS binding contents for DEV-47; minor fixes to adhoc message excerpts * #152 Add MDPWS binding contents to outline. * #152 Fixes to DEV-47 * #152 Fixes to DEV-47 * Fix yml to use new versions of github action plugins * Fix yml to use new versions of github action plugins * #152 Update references to DEV-47 outlines * Replace gradle-build-action with newer setup-gradle * #152 Add discovery proxy sources and additional chapter * Change path of setup-gradle@v3 * Use setup-gradle@v4 * Use setup-gradle@v4 * Use setup-gradle@v4 * #152 Fix findings * DP Updates to TF-0 & TF-1 - Initial Initial updates to add a transaction name to the TF-0 Transactions listing + the SDPi-P Actor model update & transaction table update & placeholder for the DP option description section. * #152 Add discovery proxy basic sequence diagram * #152 Enrich OID table caption in the discovery proxy section * TF-1 Updates Updated DP TF-1 content, specifically to the SDPi-P transaction table, the Actors and Options section (with its own table) and the Discovery Proxy Option section. Content is rough but sufficient for discussion. * Updated Sequence Diagram & Actor Diagram Sequence Diagram - added secure groupings for all transactions Actor Diagram - simplified to "Discovery Proxy" Updated Change Log with #152 addition A fix to the TF-1 SDPiP Transactions table to re-enable rendering of the end notes ... hopefully! * Trying to Fix SDPi-P Transaction Table Notes Notes are not appearing on last row of the transaction table. Also changed the note numbering based on order from the top of the table. * Fixed SDPi-P Transactions Table Notes Found the extra vertical bar! * Updated DP Transaction Names & PUML Diagram Titles & SDPi Transaction Table Updates * Removed "SDPi" from many of the TF-2 transaction sequence diagram titles - transactions are not profile specific * Refactored DEV-46 and DEV-47 transaction names + reference label text * changed the DP transactions in the SDPi-P Actors & Transactions table to "O" vs. "C" * Integrated comments from David's review of the option bullets * Commented out the version text box from the options section but in the comments kept the potential future options,. * Updated TF-1 DP Option Text Converted the bulleted content into something readable and (generally) tagged. * #152 Fix formatting * #152 Fix formatting * #152 Try to fix wrong directory for PUML includes in CI * #152 Minor fixes * #152 Add more space to images, tables and listings. * Changed "SOMDS Discovery Proxy" to "Discovery Proxy" Left over changes * Editorial Updates based on review & DCC discussion Numerous updates including ... * removing "SOMDS" from "Discovery Proxy" actor name * Discovery Proxy actor brief description more generic * Use of "Transaction" for a specific transaction now uppercase "T" * Managed Discovery Option renamed (from Discovery Proxy Option) * Added "heartbeat" requirement for DP as part of managed discovery behaviour Others ... * Fixed PUML file name error Refactoring PUML file name inserted a space in the file name ... ! * Fixed another file name error Or didn't fix it previously? * Highlighted "shall" statements for review & requirements block addition #shall# added in option block to ensure review and either rewording, removal, or requirements block addition * Updates to Discovery Proxy Option & DEV-47 Title Changed include: * Reset "Transaction" to lowercase "transaction" per guidance from Mary J. * Simplified DEV-47 to "Retrieve Network Presence", leaving off "Metadata" * Added requirements blocks for the option "shall" statements * Transaction Description Updates + MD Option Updates * Updated DEV-46 & DEV-47 transaction summaries to match the updated transaction names * Added as a test, additional requirement metadata to the MD Option section * Changed the arrow direction of the Bye() message for DEV-47 * Test of Inter-Requirement Links Added forward / backward requirement anchor references to R1023. fingers crossed * Tweak update because workflow automation failed 2nd try at triggering build workflow * Updates from SDPi Friday Review Updates from review corrections. --------- Co-authored-by: Todd "AFC!" Cooper <[email protected]>
* Add release workflow * Update release workflow
* Add release workflow * Update release workflow * Update release workflow
* Add release workflow * Update release workflow * Bump version in CHANGELOG.md
* Initial resolution proposal for #294 * Updates to support 1.4 release notes including referenced standard maturity Updated intro text to include a NOTE that referenced standards may be in development or in revision and that when that is the case, a clear indication will be made in the references section. Added some labels to support linking from intro text. Updated release 1.3 note to reflect 1.4 content. * Fixed Bogus Reference Syntax {} instead of <<>> --------- Co-authored-by: Todd "AFC!" Cooper <[email protected]>
Updated locally instead. |
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.
π Description
Bringing RI for 1.4 branch up-to-date with the master branch.
β Mandatory Tasks
The following aspects have been respected by the pull request assignee and at least one reviewer: