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

For EmberTimes No. 73: EmberConf CfP + Ember Hooks#3687

Merged
amyrlam merged 3 commits intoemberjs:blog/embertimes-73from
jayjayjpg:embertimes-jj
Nov 14, 2018
Merged

For EmberTimes No. 73: EmberConf CfP + Ember Hooks#3687
amyrlam merged 3 commits intoemberjs:blog/embertimes-73from
jayjayjpg:embertimes-jj

Conversation

@jayjayjpg
Copy link
Member

What it does

Related Issue(s)

Sources

@jayjayjpg jayjayjpg requested a review from a team November 14, 2018 17:02

## [SECTION TITLE](#section-url)
<!--alex ignore hooks-->
## [Embereact Hooks ⚛️ ](https://github.com/lifeart/hooked-components)
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
## [Embereact Hooks ⚛️ ](https://github.com/lifeart/hooked-components)
## [Embereact Hooks ⚛️](https://github.com/lifeart/hooked-components)

blog(embertimes#73): add review suggestions

Co-Authored-By: jessica-jordan <jessica.jordan.fe.dev@gmail.com>
@jayjayjpg
Copy link
Member Author

@amyrlam Thank you for all the suggestions, updated ✨

Copy link
Member

@amyrlam amyrlam left a comment

Choose a reason for hiding this comment

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

Thanks for making the changes! 💛

There is an alex failure but it's strange since you've mentioned <!--alex ignore hooks--> on every new paragraph? If if fails on merge will look into it...

@amyrlam amyrlam merged commit 87fb4ee into emberjs:blog/embertimes-73 Nov 14, 2018
@amyrlam amyrlam deleted the embertimes-jj branch November 14, 2018 22:21
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.

2 participants