Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ember Times #133: add community feedback from flashrecruit #511

Merged

Conversation

EndangeredMassa
Copy link
Contributor

What it does

This PR adds a section to the next edition of Ember Times that highlight's my company's use of Ember Octane. This is similar to a recent issue that highlighted Outdoorsy.

I followed the guidance as best as I could, but am happy to make any adjustments. Feel free to close this if it's not appropriate for publication.

@probot-autolabeler probot-autolabeler bot added the embertimes PR for an Ember Times issue label Jan 29, 2020
@amyrlam amyrlam requested a review from a team January 29, 2020 05:48
This change should make CI green
@amyrlam
Copy link
Member

amyrlam commented Jan 29, 2020

shared that the PR is up in #support-ember-times ... also wanted to mention, i would ignore the builds, we can address in the parent branch

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.

@EndangeredMassa i think it reads great, thank you, much appreciated!

i thought it might be easier to add a commit in this case, and because there was a bit of a delay on our end

i'm going to merge this in and we can make further edits off the main branch if that's ok!

@amyrlam amyrlam merged commit 5d252d5 into ember-learn:issue/embertimes-133 Jan 30, 2020
@EndangeredMassa
Copy link
Contributor Author

Sounds good, thanks!

@EndangeredMassa EndangeredMassa deleted the smassa/flashrecruit branch January 31, 2020 03:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
embertimes PR for an Ember Times issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants