Skip to content

Conversation

claudep
Copy link
Collaborator

@claudep claudep commented Nov 11, 2017

Fixes #239.

@claudep
Copy link
Collaborator Author

claudep commented Nov 11, 2017

Still WIP work.

@coveralls
Copy link

coveralls commented Nov 11, 2017

Coverage Status

Coverage increased (+0.02%) to 89.553% when pulling e4c017e on claudep:issue239 into 82b7398 on dokterbob:master.

@coveralls
Copy link

coveralls commented Nov 11, 2017

Coverage Status

Coverage increased (+0.1%) to 89.667% when pulling 89e13a0 on claudep:issue239 into 82b7398 on dokterbob:master.

@claudep
Copy link
Collaborator Author

claudep commented Nov 11, 2017

Tests look good now.

@dokterbob
Copy link
Collaborator

Wonderful work @claudep! Thanks!

Will look at it again some time soon, but so far it looks really great. :)

@dokterbob
Copy link
Collaborator

@claudep Have you, by any chance, have been able to manually check/eye the admin?

@claudep
Copy link
Collaborator Author

claudep commented Nov 11, 2017

No, I haven't.

@dokterbob
Copy link
Collaborator

dokterbob commented Nov 11, 2017 via email

@dokterbob dokterbob merged commit 89e13a0 into jazzband:master Nov 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants