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

Do not show optional (.:format) block for wildcard route #243

Closed
wants to merge 1 commit into from

Conversation

sikachu
Copy link
Member

@sikachu sikachu commented Mar 28, 2011

Do not show optional (.:format) block for wildcard route [#6605 state:resolved]

This will make the output of rake routes to be correctly match to the behavior of the application, as the regular expression used to match the path is greedy and won't capture the format part by default

This commit is the second attempt on fixing the issue, as the regular expression on another commit on master was invalid.

…state:resolved]

This will make the output of `rake routes` to be correctly match to the behavior of the application, as the regular expression used to match the path is greedy and won't capture the format part by default

This commit is the second attempt on fixing the issue, as the regular expression on another commit on `master` was invalid.
@sikachu sikachu closed this Mar 28, 2011
matthewd pushed a commit that referenced this pull request Apr 24, 2018
PostgreSQL bugfix for invalid SQL in subqueries
seanpdoyle added a commit to seanpdoyle/rails that referenced this pull request Apr 13, 2021
Closes [@hotwired/turborails#243][].

Set the `useCapture` parameter of [addEventListener][] to `true` so that
cancelling the event occurs early enough in the process so that the
resulting upload is cancelled.

[@hotwired/turborails#243]: hotwired/turbo#243 (comment)
[addEventListener]: https://developer.mozilla.org/en-US/docs/Web/API/EventTarget/addEventListener#parameters

Co-authored-by: Javan Makhmali <javan@basecamp.com>
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

1 participant