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

Bugfix and Feature Addition #12

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

csquared
Copy link

Hi Brendan:

I added a check in the format overriding to only override :html formats so :js and :xml will get through.

Also, I created another top level function that enables the plugin without setting the before_filter so you can set the before_fitler by hand if you want more fine grained control.

If you want me to send a pull request for just one of these commits let me know.

Cheers,
Chris

@mlc
Copy link

mlc commented Feb 7, 2011

I've no strong opinion on the second two commits here, but fd6f9fa is an absolutely crucial bugfix for our application as well.

@benlangfeld
Copy link

This project is abandoned. Please see the active fork at http://github.com/benlangfeld/mobile-fu. Please test with the released gem and master branch of the new home for the project, and file an issue on the other repo if you still have problems.

See #40.

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.

3 participants