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

change cacao.Command to CommandData and related #299

Open
lucamrgs opened this issue Dec 20, 2024 · 0 comments · May be fixed by #300
Open

change cacao.Command to CommandData and related #299

lucamrgs opened this issue Dec 20, 2024 · 0 comments · May be fixed by #300
Labels
cacao OASIS cacao model related issue

Comments

@lucamrgs
Copy link
Collaborator

Is your feature request related to a problem? Please describe.
In the CACAO spec, what we now call Command object, is actually the command-data type.

Also frankly I think that using Context.Command.Command, or any cacao.Command.Command, is quite counter-intuitive.

Describe the solution you'd like
context.CommandData.Command
cacao.CommandData.Command
@lucamrgs lucamrgs linked a pull request Dec 20, 2024 that will close this issue
Feature/299 change cacao command to commanddata and related #300
Open
@lucamrgs lucamrgs added the cacao OASIS cacao model related issue label Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
cacao OASIS cacao model related issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Feature/299 change cacao command to commanddata and related COSSAS/SOARCA
1 participant
@lucamrgs