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

Angle brackets and contributors for Ember Times #3777

Merged
merged 4 commits into from
Jan 25, 2019

Conversation

amyrlam
Copy link
Member

@amyrlam amyrlam commented Jan 25, 2019

let me know if any feedback!

@amyrlam amyrlam requested a review from a team January 25, 2019 06:08
@locks locks temporarily deployed to ember-website-staging-pr-3777 January 25, 2019 06:08 Inactive
@amyrlam
Copy link
Member Author

amyrlam commented Jan 25, 2019

Note: it seems safe to say "is coming soon" because https://guides.emberjs.com/release/components/block-params/ still reflects the old although ember-learn/guides-source#357 is merged (guessing the deployments for this repo are not automatic intentionally?)

---

## [SECTION TITLE](#section-url)
## [Bringing Angle Brackets to the Guides 🥳](https://github.com/ember-learn/guides-source/issues/139)
Copy link
Member

Choose a reason for hiding this comment

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

Not sure if only on my laptop but the emoji is just a square for me?

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks - it's the party "face" but maybe it's too new of an emoji, changed to the older 🎉

@kennethlarsen
Copy link
Member

@amyrlam Yes it is intentional that it isn't deployed yet so I think saying "coming soon" is the right approach

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.

This reads great ✨

@amyrlam amyrlam requested a deployment to ember-website-staging-pr-3777 January 25, 2019 06:21 Abandoned
@amyrlam
Copy link
Member Author

amyrlam commented Jan 25, 2019

Thanks for confirming that @kennethlarsen!

@amyrlam amyrlam merged commit e0b2edc into blog/embertimes-82 Jan 25, 2019
@amyrlam amyrlam deleted the amy/embertimes-82 branch January 25, 2019 06:22
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