Skip to content

Conversation

@manosim
Copy link
Member

@manosim manosim commented Jun 19, 2015

Closes #60.

screen shot 2015-06-20 at 20 24 56

screen shot 2015-06-20 at 20 44 09

screen shot 2015-06-20 at 20 40 00

@manosim
Copy link
Member Author

manosim commented Jun 19, 2015

@JakeSidSmith Wanna review this one? Works great & coverage as always - 100%!

Copy link
Contributor

Choose a reason for hiding this comment

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

We should probably add a nice way to display errors.

@JakeSidSmith
Copy link
Contributor

Could you add a couple of screenshots? :)

@manosim
Copy link
Member Author

manosim commented Jun 20, 2015

@JakeSidSmith Ready? 🚀

Copy link
Contributor

Choose a reason for hiding this comment

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

Typo. :)

@manosim manosim added release Release and removed review needed labels Jun 20, 2015
@JakeSidSmith
Copy link
Contributor

🚀

manosim pushed a commit that referenced this pull request Jun 20, 2015
@manosim manosim merged commit 87316c6 into master Jun 20, 2015
@manosim manosim deleted the mark-repo-as-read branch June 20, 2015 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release Release

Development

Successfully merging this pull request may close these issues.

3 participants