Skip to content
This repository was archived by the owner on Mar 2, 2020. It is now read-only.

allow access to the request method#73

Open
benoist wants to merge 3 commits into
basecamp:masterfrom
benoist:master
Open

allow access to the request method#73
benoist wants to merge 3 commits into
basecamp:masterfrom
benoist:master

Conversation

@benoist

@benoist benoist commented Apr 28, 2014

Copy link
Copy Markdown

I wanted to preview my templates with a different locale.
To make this process easier I wanted to change the locale based on the locale param.

@hilem

hilem commented May 5, 2014

Copy link
Copy Markdown

+1

@lukeasrodgers

Copy link
Copy Markdown

Something like this would be useful for me as well, but as-is, a bunch of specs fail because to_param is undefined.

@lukeasrodgers

Copy link
Copy Markdown

FWIW, here is an alternate implementation that accomplishes roughly the same, without using to_param. All specs pass, afaict.

CreatorCollabs@8f5669a

@dtaniwaki

Copy link
Copy Markdown

👍

@dramalho

dramalho commented May 9, 2016

Copy link
Copy Markdown

👍 it's been a couple of years, I guess the code has settled down nicely no? :)

Honest question, what's up with this repo?

@benoist

benoist commented May 9, 2016

Copy link
Copy Markdown
Author

I think the latest version included in rails is already sufficient, at least it served my purpose :)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants