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

Add requirements to cover IsValidMsgId and GetPktType #587

Closed
skliper opened this issue Apr 7, 2020 · 0 comments · Fixed by #789 or #816
Closed

Add requirements to cover IsValidMsgId and GetPktType #587

skliper opened this issue Apr 7, 2020 · 0 comments · Fixed by #789 or #816
Labels
docs This change only affects documentation. enhancement
Milestone
Caelum

Comments

@skliper
Copy link
Contributor

skliper commented Apr 7, 2020

Is your feature request related to a problem? Please describe.
Exposing CFE_SB_IsValidMsgId and CFE_SB_GetPktType needs to flow from a requirement given the current cFE requirements pattern.

Describe the solution you'd like
Add the requirements

Describe alternatives you've considered
None

Additional context
#263 #543

Requester Info
Jacob Hageman - NASA/GSFC
@skliper skliper added this to the 6.8.0 milestone Apr 7, 2020
@skliper skliper mentioned this issue Apr 8, 2020
Merged
@skliper skliper added the docs This change only affects documentation. label Apr 21, 2020
@skliper skliper modified the milestones: 6.8.0, 7.0.0 Jun 5, 2020
skliper pushed a commit to skliper/cFE that referenced this issue Aug 3, 2020
Fix nasa#787, nasa#587, Update requirements
26f9c0b 
 nasa#787
 - cES1005.1: Remove application load address from start app command
 - cTBL6003.1: Clarify table validation pior to activate
 - cEVS3103.4, cEVS3103.5, cEVS3001: Remove SB qualifier
 nasa#587
 - cSB4344: Added get packet type API requirement
 - cSB4345: Added message ID validation API requirement
@skliper skliper mentioned this issue Aug 3, 2020
Merged
astrogeco added a commit that referenced this issue Aug 14, 2020
@astrogeco
Merge pull request #789 from skliper/fix787-update-requirements
848d434 
Fix #787, #587, Update requirements
@astrogeco astrogeco mentioned this issue Aug 18, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
docs This change only affects documentation. enhancement
Projects
None yet
Milestone
Caelum
Development

Successfully merging a pull request may close this issue.

Fix #787, #587, Update requirements skliper/cFE
Integration Candidate 2020-08-12 nasa/cFE
1 participant
@skliper