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

Adding IOSpecification to called process (issue #808) #809

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

benmichel
Copy link
Member

Adding IOSpecification to the process "Check for connected clients" matching IOSpecification of CallActivity "Check for connected clients"

Adding IOSpecification to the process "Check for connected clients" matching IOSpecification of CallActivity "Check for connected clients"
@falko
Copy link
Member

falko commented Nov 20, 2019

@benmichel this seems to be a solution for issue #808 but has conflicts with the master branch. Is this still current? Could the conflict be resolved?

@falko
Copy link
Member

falko commented Jun 30, 2020

@benmichel any reply to my question above?

@falko
Copy link
Member

falko commented Aug 4, 2020

@dgagne could we please add this to the agenda of the next meeting?

@falko
Copy link
Member

falko commented Sep 2, 2020

@benmichel any update here?

@falko falko changed the title Adding IOSpecificationto called process (issue #808) Adding IOSpecification to called process (issue #808) Sep 2, 2020
@zmisiak
Copy link
Contributor

zmisiak commented Nov 4, 2020

Provided diagram imports for me with duplicate data states. Data state is available as an atribute (as expected) but also as a part of name.

@SimonRinguette
Copy link
Contributor

I believe this is the proper way of serializing the data state. This is certainly how we serialize it at Trisotech. If I remember correctly, we had a discussion on that a few (>2) years ago and we agreed to interpret the spec in that way even if it could be more black and white in the spec.

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.

4 participants