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

Fix an issue in marking missing words - Closes #717 #741

Merged
merged 1 commit into from Apr 20, 2018

Conversation

reyraa
Copy link
Contributor

@reyraa reyraa commented Apr 20, 2018

What was the problem?

The logic to mark the word options validity was correct but CSS selector to depict the. status was wrong.

How did I fix it?

Fixed the CSS selector

How to test it?

Check if the passphrase confirmation form behaves as expected.

Review checklist

@reyraa reyraa self-assigned this Apr 20, 2018
@reyraa reyraa requested a review from slaweet April 20, 2018 09:04
@reyraa reyraa added this to Previous Issues in Sprint Board 16-04-18 via automation Apr 20, 2018
@reyraa reyraa added this to Issues in Version 0.6.0 via automation Apr 20, 2018
@reyraa reyraa added this to Issues in Version 0.5.0 via automation Apr 20, 2018
@reyraa reyraa removed this from Issues in Version 0.6.0 Apr 20, 2018
@reyraa reyraa changed the base branch from 0.6.0 to 0.5.0 April 20, 2018 09:05
@reyraa reyraa force-pushed the 717-issue-inmarking-missing-wrods branch from 480cc5c to ffd2629 Compare April 20, 2018 09:07
@reyraa reyraa moved this from Previous Issues to Pull Requests in Sprint Board 16-04-18 Apr 20, 2018
@slaweet slaweet changed the title Fixed an issue inmarking missing words - Closes #717 Fix an issue in marking missing words - Closes #717 Apr 20, 2018
Copy link
Contributor

@slaweet slaweet left a comment

Choose a reason for hiding this comment

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

Good job, Ali

@reyraa reyraa merged commit 1a6d9ac into 0.5.0 Apr 20, 2018
Version 0.5.0 automation moved this from Issues to Merged Pull Requests Apr 20, 2018
Sprint Board 16-04-18 automation moved this from Pull Requests to Merged Pull Requests Apr 20, 2018
@reyraa reyraa deleted the 717-issue-inmarking-missing-wrods branch April 20, 2018 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Sprint Board 16-04-18
  
Merged Pull Requests
Version 0.5.0
  
Merged Pull Requests
Development

Successfully merging this pull request may close these issues.

None yet

2 participants