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

Reroute to develop [#16] [Frontend] As a User, I can sign up and sign in/out with a username and password #43

Merged
merged 31 commits into from Jun 23, 2021

Conversation

malparty
Copy link
Owner

This PR contains only changes from PR #37

I forgot to update the target of PR #37 to develop. So I just need to bring these changes into develop now.

Feature:

@malparty malparty added $user-login Authentication: Sign up, sign in and sign out @0.3.0 v0.3.0 third week release Feature Frontend labels Jun 22, 2021
@malparty malparty added this to the 0.3.0 milestone Jun 22, 2021
@malparty malparty self-assigned this Jun 22, 2021
@malparty malparty added this to In development in Product backlog via automation Jun 22, 2021
@malparty malparty changed the title Feature/user login backend Reroute to develop [#16] [Frontend] As a User, I can sign up and sign in/out with a username and password Jun 22, 2021
</a>
</div>
<div class="collapse" id="newPassword">
<% if @minimum_password_length %>
Copy link

@github-actions github-actions bot Jun 22, 2021

Choose a reason for hiding this comment

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

⚠️ Replace instance variable with local variable

</div>
<div class="collapse" id="newPassword">
<% if @minimum_password_length %>
<em>(<%= "#{@minimum_password_length} #{t('min_char')}" %>)</em>
Copy link

@github-actions github-actions bot Jun 22, 2021

Choose a reason for hiding this comment

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

⚠️ Replace instance variable with local variable

</div>
<div class="form-group">
<%= f.label :password %><br/>
<% if @minimum_password_length %>
Copy link

@github-actions github-actions bot Jun 22, 2021

Choose a reason for hiding this comment

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

⚠️ Replace instance variable with local variable

<div class="form-group">
<%= f.label :password %><br/>
<% if @minimum_password_length %>
<em>(<%= "#{@minimum_password_length} #{t('min_char')}" %>)</em>
Copy link

@github-actions github-actions bot Jun 22, 2021

Choose a reason for hiding this comment

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

⚠️ Replace instance variable with local variable

@github-actions
Copy link

github-actions bot commented Jun 22, 2021

2 Warnings
⚠️ spec/support/authentication_helper.rb#L26 - Has approx 8 statements
⚠️ spec/support/authentication_helper.rb#L35 - Is controlled by argument 'password_confirm'

Generated by 🚫 Danger

@malparty malparty moved this from In development to In code review in Product backlog Jun 23, 2021
Product backlog automation moved this from In code review to Ready for QA Jun 23, 2021
Copy link
Contributor

@junan junan left a comment

Choose a reason for hiding this comment

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

The branch should have been feature/user-login-frontend 😄

@malparty malparty merged commit afd4b1e into develop Jun 23, 2021
@malparty malparty deleted the feature/user-login-backend branch June 23, 2021 10:57
@malparty malparty mentioned this pull request Jun 25, 2021
@malparty malparty moved this from Ready for QA to Completed in Product backlog Jun 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Frontend $user-login Authentication: Sign up, sign in and sign out @0.3.0 v0.3.0 third week release
Projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants