Change devise unlock strategy from :both to :time

[jayjay-w]

Description

Currently, we are using an unlock strategy of :both, which allows both time based unlocking and email based unlocking. This leads to an email being sent to the user when their account is locked, which us currently not intended.

References: CV2-4164

How has this been tested?

Confirmed that all tests are passing

Things to pay attention to during code review

Ensure that locking and unlocking of user accounts works as expected.

Checklist

• I have performed a self-review of my own code
• I have added unit and feature tests, if the PR implements a new feature or otherwise would benefit from additional testing
• I have added regression tests, if the PR fixes a bug
• I have added logging, exception reporting, and custom tracing with any additional information required for debugging
• I considered secure coding practices when writing this code. Any security concerns are noted above.
• I have commented my code in hard-to-understand areas, if any
• I have made needed changes to the README
• My changes generate no new warnings
• If I added a third party module, I included a rationale for doing so and followed our current guidelines

[codeclimate]

Code Climate has analyzed commit db7f042 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (100% is the threshold).

This pull request will bring the total coverage in the repository to 99.9% (0.0% change).

View more on Code Climate.