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

Short writeup on EmberConf brainstorm #3644

Merged
merged 3 commits into from
Oct 19, 2018
Merged

Conversation

amyrlam
Copy link
Member

@amyrlam amyrlam commented Oct 19, 2018

Also, added authors and cleaned up the branch

Still need:

  • hello in language we haven't used...?
  • intro summarizing content

Also, added authors and cleaned up the branch
@amyrlam amyrlam requested a review from a team October 19, 2018 08:05
@kennethlarsen
Copy link
Member

Here's a hello I think we haven't used: "Goddaw"

Copy link
Member

@kennethlarsen kennethlarsen left a comment

Choose a reason for hiding this comment

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

Looks great ✨

Copy link
Member

@Alonski Alonski left a comment

Choose a reason for hiding this comment

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

Looks good 👍 Maybe add some bold around EmberConf and interactive video brainstorm for more pizzazz :)

@Alonski
Copy link
Member

Alonski commented Oct 19, 2018

Also, this will probably conflict with my PR. The one that gets merged second will need to have conflicts fixed.

@amyrlam
Copy link
Member Author

amyrlam commented Oct 19, 2018

Thanks for the hello suggestions! I'll let Alon's namaste get in 🙏

Good idea, I'll add bolds and fix up this one re: merge conflicts

Thanks for the reviews!

@amyrlam amyrlam merged commit 7d99ec9 into blog/embertimes-69 Oct 19, 2018
@amyrlam amyrlam deleted the amyrlam-patch-2 branch October 19, 2018 16:31
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.

3 participants