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.

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

Hotfix protocol handling #176

Merged
merged 2 commits into from
Aug 17, 2023
Merged

Hotfix protocol handling #176

merged 2 commits into from
Aug 17, 2023

Conversation

HLWeil
Copy link
Member

@HLWeil HLWeil commented Aug 15, 2023

Parsing between ArcTables and Protocols lead to breaking changes when writing and reading an ARC.

This is because the name of the ArcTables was not included into the protocol, leading to empty protocols in the study section of the investigation file when writing. These would then be read to nameless ArcTables, leasing to breaks.

  • Fixed this broken behaviour
  • added another safetynet, namely using Missing identifiers when reading nameless protocols
  • added tests

@HLWeil HLWeil merged commit 21f56b2 into developer Aug 17, 2023
2 checks passed
@HLWeil HLWeil deleted the hotfix_protocolHandling branch September 5, 2023 07:48
@HLWeil HLWeil mentioned this pull request Sep 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants