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

Best changes from forked repos #12

Open
aaronchi opened this issue Jan 25, 2012 · 2 comments
Open

Best changes from forked repos #12

aaronchi opened this issue Jan 25, 2012 · 2 comments

Comments

@aaronchi
Copy link

Hey,

Since this plugin seems to be getting some attention again, I would suggest pulling in the changes from this repo:

https://github.com/insphire/userstamp

Specifically, the issues that are addressed that haven't been updated in this repo yet:

  • deleter option defined as a preference instead of looking for acts_as_paranoid
  • creator/updater ids set before_validation instead of on save (so that presence can be verified in validations)
  • a few issues related to class names

I can put in a pull request if it would help

@delynn
Copy link
Owner

delynn commented Jan 25, 2012

I'll be sure to look at this version before pushing an update. Thanks for the heads up!

@rmcsharry
Copy link

@delynn Is this gem dead?

richwallett pushed a commit to richwallett/userstamp that referenced this issue Jan 13, 2018
Delay the definition of the associations until the table schema is loaded
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

No branches or pull requests

3 participants