Skip to content
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.

Sign up for GitHub

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

Jump to bottom

always encode concept name code sequence in scheduled processing parameters sequence #83

Merged
merged 1 commit into from
Oct 18, 2024
Merged

always encode concept name code sequence in scheduled processing parameters sequence #83

merged 1 commit into from
Oct 18, 2024

Conversation

sjswerdloff
Copy link
Owner

if the code sequence has been defined
should fix #81

@sjswerdloff
always encode concept name code sequence in scheduled processing para…
3fbd41c 
…meters sequence if the code sequence has been defined
@sjswerdloff sjswerdloff self-assigned this Oct 18, 2024
@sjswerdloff sjswerdloff requested a review from dwikler October 18, 2024 03:35
dwikler
dwikler approved these changes Oct 18, 2024
View reviewed changes
Copy link
Collaborator

@dwikler dwikler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me.

@sjswerdloff sjswerdloff merged commit 12d576e into main Oct 18, 2024
3 checks passed
@sjswerdloff sjswerdloff deleted the always_add_concept_name_to_ups branch October 18, 2024 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dwikler dwikler dwikler approved these changes

Assignees

@sjswerdloff sjswerdloff

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

UPS Scheduled Processing Parameters Sequence items are missing Concept Name Code Sequence
2 participants
@sjswerdloff @dwikler