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

[Ember Times] Amy RecordData writeup#3554

Merged
jayjayjpg merged 3 commits intoblog/embertimes-63from
amy/embertimes-63-writeup
Sep 7, 2018
Merged

[Ember Times] Amy RecordData writeup#3554
jayjayjpg merged 3 commits intoblog/embertimes-63from
amy/embertimes-63-writeup

Conversation

@amyrlam
Copy link
Member

@amyrlam amyrlam commented Sep 7, 2018

@amyrlam amyrlam requested a review from a team September 7, 2018 07:56
@amyrlam
Copy link
Member Author

amyrlam commented Sep 7, 2018

I think the Travis failure is from the alex warnings?

Feel free to edit at will, be back in the morning SF-time.

@jayjayjpg jayjayjpg force-pushed the amy/embertimes-63-writeup branch from b1fba33 to c141b4d Compare September 7, 2018 15:01
Copy link
Member

@jayjayjpg jayjayjpg 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, I just rebased onto the latest embertimes branch. Thank you for another great writeup this week, Amy! ✨

@jayjayjpg jayjayjpg merged commit c349aa4 into blog/embertimes-63 Sep 7, 2018
@amyrlam amyrlam deleted the amy/embertimes-63-writeup branch September 21, 2018 02:38
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