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

Correct values of AllocTransType(71) and AllocType(626) #38

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

Ryno-Olivier
Copy link

Code set AllocTransTypeCodeSet(71): deprecated values “3”, “4”, “5”, "6" were missing.
Nr 6 was not mentioned in the GAP E&O document but is visible in the FIXLatest version. So it was also added.

…"6" were added

•	Code set AllocTransTypeCodeSet(71): deprecated values “3”, “4”, “5”, "6" were missing.
Nr 6 was not mentioned in the GAP E&O document but is visible in the FIXLatest version. So it was also added.
…issing

Code set AllocTypeCodeSet(626): deprecated values “3”, “4, “6” were added according to the FIXLatest version.
@kleihan kleihan self-requested a review November 21, 2024 06:44
Copy link
Member

@kleihan kleihan left a comment

Choose a reason for hiding this comment

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

AllocTransType(71)=6 was added with EP5 only after the release of FIX 4.4. It is not missing.

@kleihan kleihan changed the title Code set AllocTransTypeCodeSet(71): deprecated values “3”, “4”, “5”, and 6" were added. Correct values of AllocTransType(71) and AllocType(626) Nov 21, 2024
AllocTransType(71)= "Value 6" has been removed.
Copy link
Member

@kleihan kleihan left a comment

Choose a reason for hiding this comment

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

Wrong one removed, AllocType(626)=6 was already added with FIX 4.3.

…Set”

Updated Field RefMsgType(372) to define type=“MsgTypeCodeSet” instead of “String”
@Ryno-Olivier
Copy link
Author

Please ignore this Pull Request. I will re-create all these issues in seperate PR's

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