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

Release - 0.4.0 #53

Merged
merged 131 commits into from
Jul 2, 2021
Merged

Release - 0.4.0 #53

merged 131 commits into from
Jul 2, 2021

Conversation

malparty and others added 30 commits June 23, 2021 19:04
[#28] [Backend] As a User, I can reset my password via the Web UI
[#29] [Frontend] As a User, I can reset my password via the Web UI
@malparty malparty added this to the 0.4.0 milestone Jul 1, 2021
@malparty malparty requested review from junan and ankitkalia1195 July 1, 2021 02:21
@malparty malparty self-assigned this Jul 1, 2021
@github-actions
Copy link

github-actions bot commented Jul 1, 2021

7 Warnings
⚠️ app/models/user.rb#L16 - Remove unused methods (user#authenticate)
⚠️ app/models/user.rb#L18 - Is controlled by argument 'password'
⚠️ app/models/user.rb#L21 - Remove unused methods (user#create_access_token)
⚠️ spec/support/vcr.rb#L46 - Has approx 8 statements
⚠️ spec/support/vcr.rb#L47 - Refers to 'vcr_options' more than self (maybe move it to another class?)
⚠️ spec/support/vcr.rb#L48 - Refers to 'vcr_options' more than self (maybe move it to another class?)
⚠️ spec/support/vcr.rb#L49 - Refers to 'vcr_options' more than self (maybe move it to another class?)

Generated by 🚫 Danger

@malparty malparty merged commit 8c274b7 into main Jul 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants