Skip to content

Worflow Component: Add events which are dispatched #7840

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

Closed
wants to merge 1 commit into from
Closed

Worflow Component: Add events which are dispatched #7840

wants to merge 1 commit into from

Conversation

tomaszhanc
Copy link

More evnts are going to be dispatched - not only 'guard' events.

@HeahDude
Copy link
Contributor

Thank you for this first contribution, but those event are now documented, and this place does not seem appropriate since this is a section about guard only.

So I'm 👎 on this, in the current state.

@xabbuh
Copy link
Member

xabbuh commented Apr 26, 2017

@HeahDude You are right about the example making use of the guard event. However, the section itself just is about events in general. So maybe we can keep the example, but remove the events reference part and link to the new section instead?

@HeahDude
Copy link
Contributor

👍 Agreed!

@tomaszhanc
Copy link
Author

@HeahDude Hi. The article is named 'How to Use the Workflow' so I thought is not only for how to use guards. I can change it but I can't find a place where all events are documented. Can you help me somehow with that?

@HeahDude
Copy link
Contributor

@tomaszhanc Sorry for the delay. I'm closing here as #7884 as been merged in the meantime.

@HeahDude HeahDude closed this Jul 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants