Improve the legacy wild controller route in app templates #1727

Merged
merged 2 commits into from Jun 16, 2011

Conversation

Projects
None yet
3 participants
@ihower
Contributor

ihower commented Jun 16, 2011

The original version matchs as follows

  • /foo
  • /foo/bar
  • /foo/bar/1
  • /foo.json (Routing Error)
  • /foo/bar.json (Routing Error)
  • /foo/bar/1.json

This pull request corrects these two routing erros.

@dmathieu

This comment has been minimized.

Show comment Hide comment
@dmathieu

dmathieu Jun 16, 2011

Contributor

This is only documentation changes. You should commit it on docrails. https://github.com/lifo/docrails
It has public commit policy. You don't have to make a pull request there.

Contributor

dmathieu commented Jun 16, 2011

This is only documentation changes. You should commit it on docrails. https://github.com/lifo/docrails
It has public commit policy. You don't have to make a pull request there.

josevalim added a commit that referenced this pull request Jun 16, 2011

Merge pull request #1727 from ihower/improve_legacy_wild_controller_r…
…oute

Improve the legacy wild controller route in app templates

@josevalim josevalim merged commit 43ed24e into rails:master Jun 16, 2011

jake3030 pushed a commit to jake3030/rails that referenced this pull request Jun 28, 2011

Support OPTIONS verb in route conditions [#1727 state:resolved]
Signed-off-by: Pratik Naik <pratiknaik@gmail.com>

@ihower ihower deleted the ihower:improve_legacy_wild_controller_route branch Aug 16, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment