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

Pass parameters in forward #10632

Merged
merged 1 commit into from Jun 19, 2017
Merged

Pass parameters in forward #10632

merged 1 commit into from Jun 19, 2017

Conversation

sdelamo
Copy link
Contributor

@sdelamo sdelamo commented May 9, 2017

Fixes #10583

Pass params infered from custom URL Mappings e.g. /book/$id/show to the forwarded action

Fixes #10583

Pass params infered from custom URL Mappings e.g. /book/$id/show to the forwarded action
@sdelamo sdelamo requested a review from graemerocher May 9, 2017 14:45
@sdelamo sdelamo changed the base branch from master to 3.2.x May 9, 2017 14:49
@jameskleeh
Copy link
Contributor

Did you add functional tests for this? If so, can you link here to the commit(s)?

@sdelamo
Copy link
Contributor Author

sdelamo commented May 9, 2017

This is the functional tests: https://github.com/grails/grails3-functional-tests/pull/6

@sdelamo sdelamo assigned jameskleeh and unassigned graemerocher May 11, 2017
@sdelamo sdelamo requested review from jameskleeh and removed request for graemerocher May 11, 2017 04:46
@sdelamo
Copy link
Contributor Author

sdelamo commented May 23, 2017

Fixes #9750

@graemerocher graemerocher merged commit a0ed7ef into 3.2.x Jun 19, 2017
@jameskleeh jameskleeh deleted the 3.2.x-grails-issue-10583 branch December 20, 2018 16:23
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.

None yet

3 participants