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 fixes #227

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

Rails 5 fixes #227

wants to merge 74 commits into from

Conversation

niciliketo
Copy link

We are in the process of migrating to Rails 5 and have done the work to ensure DA continues to work for us.

Would you consider merging this one?

annaswims and others added 30 commits May 25, 2016 10:57
use parentheses around parameters in method definitions
Also, make depentencies clear in .gemspec
Don't explicitly support rails < 4.0.  or ruby <2.0
…rity_rebased

Don't inspect user;  too much sensitve data in log
Ensure no read-only records are returned when involving DA scopes.
Corrected rdoc inline code formatting
Aaron Huntsman and others added 30 commits May 23, 2018 08:10
- Added fix for the ActiveRecord::Reflection assocation class issues
- Updated text render syntax according to rails 5
ActiveRecord::Reflection issue fix
Added fix for sexp integer type errors
Security issues mean Rails 5.2 point release is now 5.2.1.1
Change the dependencies to be more permissive about Rails versions
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.

7 participants