-
Notifications
You must be signed in to change notification settings - Fork 56
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
Compatibility with Mongoid 5 #27
Comments
I'm using this gem, if you can raise a PR for Mongoid 5 compatibility would be great. |
@johnnyshields I'm sorry but I have never raised a PR. If you could guide me, I'd be glad to do it. Thanks |
@Mohakjuneja there are many guides online for this. Please google. |
I've forked the repository but I do not know what changes in the code would solve the compatibility issue. Which file would you suggest me to start? Thank you for your patience. |
A start should be to extend https://github.com/tbpro/mongoid_userstamp/blob/master/Gemfile#L5-L12 and https://github.com/tbpro/mongoid_userstamp/blob/master/.travis.yml#L13-L14 to extend the tests to mongoid 5. I have not followed mongoid for a long time as I'm not using mongo in any project anymore, so I don't know what made it incompatible. |
|
So for some reason the created_by_id and updated_by_id attributes are still not being saved along the object after updating the travis.yml and Gemfile. |
@Mohakjuneja you're going to have to debug this yourself. Please write a test case that reproduces the issue. |
+1 |
@sahilchopra None at my end. I'll try and publish a test case this week |
Thank you! Right now this Gem is not working with mongoid 5. |
Hi there! I'm using mongoid 5. The gem is not compatible with it. Is anyone actively maintaining this gem or should we look for alternatives?
The text was updated successfully, but these errors were encountered: