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

Explain Msg ID values #131

Closed
zanzaben opened this issue Mar 11, 2021 · 0 comments · Fixed by #132 or #137
Closed

Explain Msg ID values #131

zanzaben opened this issue Mar 11, 2021 · 0 comments · Fixed by #132 or #137
Assignees
@zanzaben

Comments

@zanzaben
Copy link
Contributor

Is your feature request related to a problem? Please describe.
Message Id's seem like they can be anything but in the sample app cmd and tlm need to be formatted to 0x18xx and 0x08xx.

Describe the solution you'd like
Add some info to better explain why the Msg IDs have those requirements.

Additional context
Part of nasa/cFE#590

Requester Info
Alex Campbell GSFC
@zanzaben zanzaben self-assigned this Mar 11, 2021
zanzaben added a commit to zanzaben/sample_app that referenced this issue Mar 11, 2021
@zanzaben
Fix nasa#131, add context to the values for MsgIDs
ca55d89
@zanzaben zanzaben mentioned this issue Mar 11, 2021
Merged
zanzaben added a commit to zanzaben/sample_app that referenced this issue Mar 16, 2021
@zanzaben
Fix nasa#131, add context to the values for MsgIDs
ba6148a
astrogeco added a commit that referenced this issue Mar 18, 2021
@astrogeco
Merge pull request #132 from zanzaben/fix131_SB_MsgID_doc
ffe4aae 
Fix #131, add context to the values for MsgIDs
@astrogeco astrogeco mentioned this issue Mar 18, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zanzaben zanzaben

Labels
None yet
Projects
None yet
Milestone
No milestone
1 participant
@zanzaben