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

Retrieve forward/display route from Process object. #1863

Merged
merged 1 commit into from
Sep 3, 2014

Conversation

kayue
Copy link
Contributor

@kayue kayue commented Sep 3, 2014

We should prevent hardcoding the route in template and purchase step, so the step / template can be reuse in different process.

@stloyd
Copy link
Contributor

stloyd commented Sep 3, 2014

👍

1 similar comment
@arnolanglade
Copy link
Contributor

👍

pjedrzejewski pushed a commit that referenced this pull request Sep 3, 2014
Retrieve forward/display route from Process object.
@pjedrzejewski pjedrzejewski merged commit 30cd5a3 into Sylius:master Sep 3, 2014
@pjedrzejewski
Copy link
Member

Thanks Ka Yue! 👍

@kayue kayue deleted the patch/forward-route branch September 3, 2014 13:35
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