Skip to content
This repository has been archived by the owner on Jan 24, 2019. It is now read-only.

Errors #2215

Merged
merged 2 commits into from Jun 16, 2015
Merged

Errors #2215

merged 2 commits into from Jun 16, 2015

Conversation

Pomax
Copy link
Contributor

@Pomax Pomax commented Jun 13, 2015

addresses #1922

@@ -24,15 +26,14 @@ var Make = React.createClass({
this.load();
},
onError: function (err) {
console.error(err);
reportError("Error loading project", err);
Copy link
Contributor

Choose a reason for hiding this comment

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

This should be "Error loading projects" (plural)

@thisandagain
Copy link
Contributor

This is great. I love how clean the snackbar component turned out. Two really minor comments, otherwise R+ [unflagged]

@thisandagain
Copy link
Contributor

RRRRRRR+ [unflagged]

thisandagain added a commit that referenced this pull request Jun 16, 2015
@thisandagain thisandagain merged commit 4bbe9ce into mozilla:develop Jun 16, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants