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

Rails 5.1 support #1310

Merged
merged 5 commits into from
Oct 1, 2017
Merged

Rails 5.1 support #1310

merged 5 commits into from
Oct 1, 2017

Conversation

tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented Sep 7, 2017

Not much needed actually to make Alchemy Rails 5.1 compatible

Verified

This commit was signed with the committer’s verified signature. The key has expired.
tvdeyen Thomas von Deyen
Rails 5.1 removed (without deprecation notice!) the `tag_options` helper and move it to the `tag_builder` object.
The dummy apps `Event` model has a `belongs_to` `Location` relation that is mandatory since Rails 5.1. This changes the specs so we always have a location for an event.
@tvdeyen tvdeyen requested a review from mamhoff September 7, 2017 10:16
@tvdeyen tvdeyen added this to the 4.0 milestone Sep 7, 2017
@tvdeyen tvdeyen mentioned this pull request Sep 7, 2017
Copy link
Contributor

@mamhoff mamhoff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hooray!

@tvdeyen tvdeyen merged commit ddde1e7 into AlchemyCMS:master Oct 1, 2017
@tvdeyen tvdeyen deleted the rails-5.1-support branch October 1, 2017 22:55
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

2 participants