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

blog(embertimes85): core panel, Boston meetup #3813

Merged

Conversation

jaredgalanis
Copy link
Member

No description provided.

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.

Thank you so much for this, this reads great! ✨
It would be cool if you could add your name to the top of the file and then again in the bottom where the other names are listed.

@@ -27,8 +27,13 @@ Check out the [full documentary](https://www.youtube.com/watch?v=Cvz-9ccflKQ) fe

---

## [SECTION TITLE](#section-url)
## [Ember.js core panel, Boston meetup](https://www.youtube.com/watch?v=Cvz-9ccflKQ)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
## [Ember.js core panel, Boston meetup](https://www.youtube.com/watch?v=Cvz-9ccflKQ)
## [Ember.js core panel, Boston meetup](https://www.youtube.com/watch?v=Jp4jrwz4zV8&feature=youtu.be&t=105)

fixed the link

@amyrlam
Copy link
Member

amyrlam commented Feb 14, 2019

This is awesome, thank you @jaredgalanis!

You'll definitely need to add our name like Kenneth said, and add a blurb to the top part

Watch the Ember.js Documentary 🍿, <SOME-INTRO-HERE-TO-KEEP-THEM-SUBSCRIBERS-READING>

@jaredgalanis jaredgalanis force-pushed the emberjs-core-panel-boston branch from 6609aa5 to 92fdec2 Compare February 14, 2019 17:47
@amyrlam amyrlam merged commit 425ca20 into emberjs:blog/embertimes-85 Feb 14, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants