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

Ember Times No. 86 - February 22nd, 2019 #3821

Merged
merged 25 commits into from
Feb 22, 2019
Merged

Ember Times No. 86 - February 22nd, 2019 #3821

merged 25 commits into from
Feb 22, 2019

Conversation

jayjayjpg
Copy link
Member

@jayjayjpg jayjayjpg commented Feb 18, 2019

Ideas, feel free to add to list or claim!

Todos:

@locks locks temporarily deployed to ember-website-staging-pr-3821 February 18, 2019 16:24 Inactive
@locks locks temporarily deployed to ember-website-staging-pr-3821 February 19, 2019 18:09 Inactive
@locks locks temporarily deployed to ember-website-staging-pr-3821 February 21, 2019 10:09 Inactive
@locks locks temporarily deployed to ember-website-staging-pr-3821 February 21, 2019 12:54 Inactive

---

## RFCs for [Decorator Support](https://github.com/emberjs/rfcs/pull/440) and [Tracked Property Classic Class Descriptor](https://github.com/emberjs/rfcs/pull/442) 👣
Copy link
Member

Choose a reason for hiding this comment

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

How come this doesn't appear on the deployment rn? Because it's a WIP PR? Writing a comment to check this locally (header with 2 links) later, we can do one link /rfcs for Goodbits

Copy link
Member Author

Choose a reason for hiding this comment

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

I think the post doesn't show up in the app we currently use for the blog if the date is in the future. It should show up today though or you can check it out by running the app locally and setting your computer's time settings into the future or check it out today:
https://ember-website-staging-pr-3821.herokuapp.com/blog/2019/02/22/the-ember-times-issue-86.html

Feel free to let me know if I missed something about the question and yes I agree with what you suggested about the header title - I can update it in a copy-edit 👍

Copy link
Member

Choose a reason for hiding this comment

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

Oh I see, I thought it was something with WIP PR.

And yes I meant two links in one header wouldn't work for Goodbits.

Thank you!

@amyrlam amyrlam temporarily deployed to ember-website-staging-pr-3821 February 22, 2019 04:56 Inactive
@amyrlam amyrlam temporarily deployed to ember-website-staging-pr-3821 February 22, 2019 05:02 Inactive
@locks locks temporarily deployed to ember-website-staging-pr-3821 February 22, 2019 09:26 Inactive
@locks locks temporarily deployed to ember-website-staging-pr-3821 February 22, 2019 10:17 Inactive
@locks locks temporarily deployed to ember-website-staging-pr-3821 February 22, 2019 15:49 Inactive
@amyrlam amyrlam changed the title WIP: Ember Times No. 86 - February 22nd, 2019 Ember Times No. 86 - February 22nd, 2019 Feb 22, 2019
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.

another 💯 issue!

@amyrlam amyrlam merged commit 174a5d5 into master Feb 22, 2019
@amyrlam amyrlam deleted the blog/embertimes-86 branch February 22, 2019 17:59
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.

7 participants