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

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

Merged
amyrlam merged 25 commits into
masterfrom
blog/embertimes-86
Feb 22, 2019
Merged

amyrlam merged 25 commits into
masterfrom
blog/embertimes-86

Conversation

@jayjayjpg

@jayjayjpg jayjayjpg commented Feb 18, 2019 •

Copy link
Copy Markdown
Member

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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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!

jaredgalanis and others added 6 commits February 21, 2019 14:48
Like DecEmber, it was easier to just share the blog post text somewhat in full (word for word) here... 😳
Co-Authored-By: jaredgalanis <jaredgalanis@users.noreply.github.com>
@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

@amyrlam amyrlam left a comment

Copy link
Copy Markdown
Member

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

This branch was successfully deployed

1 active deployment
ember-website-staging-pr-3821 — e7c6f46a Deployed Feb 22, 2019 by amyrlam
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