Skip to content

Conversation

@JungMinu
Copy link
Member

@JungMinu JungMinu commented Nov 6, 2015

Please review this post. ❤️,@nodejs/evangelism

Copy link
Member

Choose a reason for hiding this comment

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

Maybe instead of Please click here, we can go with something more like:

* [Submit your images and ideas on GitHub!](https://github.com/nodejs/evangelism/issues/179)

@Trott
Copy link
Member

Trott commented Nov 6, 2015

LGTM with one comment

@JungMinu
Copy link
Member Author

JungMinu commented Nov 7, 2015

@Trott Thanks, I've done :)

Copy link
Contributor

Choose a reason for hiding this comment

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

Could you give a more solid description here? The way it's worded here is very uncertain, which is uncomfortable given the topic.

Copy link
Member

Choose a reason for hiding this comment

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

You could even just remove the bullet point with Marketing Committee in it. That isn't critical info at this point and it's there if you click the link.

@bnb
Copy link
Contributor

bnb commented Nov 7, 2015

Other than my comments, LGTM.

@JungMinu
Copy link
Member Author

JungMinu commented Nov 8, 2015

@Trott @bnb Thanks, I updated 😄

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.

4 participants