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.

Sign up for GitHub

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

Jump to bottom

Fix tests #230

Merged
merged 7 commits into from
Oct 31, 2017
Merged

Fix tests #230

merged 7 commits into from
Oct 31, 2017

Conversation

gjvoosten
Copy link
Collaborator

No description provided.

@gjvoosten
Delete unused import
67b22ff
@gjvoosten
Fix failing test
fab5c11 
Make sure the blur event is fired after typing some input.
Changed behaviour was introduced in commit 42a1a01.
@gjvoosten
Fix failing tests
8b5758e 
Use new-style name formatting introduced in commit 5ca365d.
@gjvoosten
Change people's names to canonical form
efebb86
@gjvoosten
Prompt user to input their name in canonical form
85a76fc
@gjvoosten
Fix failing tests
3fb02c6 
Use canonical names.
@gjvoosten
Delete unused test resource
0ebc0b6
@gjvoosten gjvoosten merged commit 1fb5db1 into candidate Oct 31, 2017
@gjvoosten gjvoosten deleted the test-fixes branch October 31, 2017 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maradragan maradragan maradragan approved these changes

@evdhoudt evdhoudt Awaiting requested review from evdhoudt

@VassilIordanov VassilIordanov Awaiting requested review from VassilIordanov

@timward77 timward77 Awaiting requested review from timward77

Assignees
No one assigned
Labels
debt
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@gjvoosten @maradragan @VassilIordanov