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

0.14.2 labguru fork #1

Draft
wants to merge 20 commits into
base: master
Choose a base branch
from
Draft

0.14.2 labguru fork #1

wants to merge 20 commits into from

Conversation

2xG
Copy link

@2xG 2xG commented Sep 27, 2022

There was a small change in lib/devise-security/models/password_archivable.rb since upstream 0.13.0 version so this fork's changes were modified a bit. Branches for 0.14.0 and 0.14.1 were also created, but these versions has some bugs in them. See v0.14.1 release notes and v0.14.2 release notes
This PR shouldn't be merged at the moment, but could be approved as the sign of acceptance.

olbrich and others added 19 commits February 13, 2019 20:11
* setup builds to work properly with pinned sqlite3 gem
* fix deprecations
* Removing mass assignment
* changed > 0 to .positive?
Fixed grammar, spelling, removed shouting.
Bump version for release 0.14.0
* Unit tests & refactor of session limitable.  Note that the previous implementation would happily save the user record if it was 1) unpersisted, or 2) invalid.  Now we update ONLY the appropriate column when updating an invalid record (so we don't accidentally save invalid data) and throw an exception when the record is not already persisted
* remove limit from db column
* add columns to base migration that were missing (normally these are created by devise)
…ecurity#104)

Fixes devise-security#103

Co-authored-by: Kevin Olbrich <22176+olbrich@users.noreply.github.com>
@2xG 2xG requested review from jgross and annakuts1 September 27, 2022 05:05
Copy link

@toppa toppa left a comment

Choose a reason for hiding this comment

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

Looks good, but needs to be rebased due to conflicts

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.

9 participants