Skip to content

Conversation

@iteman
Copy link
Member

@iteman iteman commented Mar 30, 2016

Q A
Branch? master
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Tests pass? no
Fixed tickets -
License BSD-2-Clause

This PR introduces WorkflowContext instead of bare workflow ID. Process::getWorkflowContext() will return the WorkflowContext object for the process.

@iteman iteman self-assigned this Mar 30, 2016
@iteman iteman modified the milestone: 1.1.0 Mar 30, 2016
@iteman iteman merged commit 1f28445 into phpmentors-jp:master Mar 30, 2016
iteman added a commit that referenced this pull request Mar 30, 2016
This PR was merged into the 1.1-dev branch.

Discussion
----------

Using WorkflowContextInterface

| Q             | A
| ------------- | ---
| Branch?       | `master`
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | no
| Fixed tickets | -
| License       | BSD-2-Clause

This PR introduces `WorkflowContext` instead of bare workflow ID. `Process::getWorkflowContext()` will return the `WorkflowContext` object for the process.

Commits
-------

bfa9e7e change the version of `phpmentors/workflower` to `~1.1@dev`
55f08d2 add `WorkflowContext`
1f28445 change `PHPMentorsWorkflowerExtension` to use `WorkflowContext` instead of bare workflow ID
@iteman iteman deleted the using-workflow-context-interface branch March 30, 2016 03:57
iteman added a commit that referenced this pull request Apr 27, 2016
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.

1 participant