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

[WIP] Encourage decorator-style Ember.computed/Ember.observer #456

Closed
wants to merge 2 commits into from
Closed

[WIP] Encourage decorator-style Ember.computed/Ember.observer #456

wants to merge 2 commits into from

Conversation

HeroicEric
Copy link
Member

  • Use Ember.computed over property()
  • Use Ember.observer over .observes()

for #455

@HeroicEric HeroicEric changed the title [WIP] Use Ember.computed over .property() [WIP] Encourage decorator-style Ember.computed/Ember.observer Jul 22, 2015
@HeroicEric HeroicEric force-pushed the encourage-decorator-stye branch from 1c0371f to 2df384c Compare July 22, 2015 03:57
HeroicEric
HeroicEric reviewed Jul 22, 2015
View reviewed changes
source/object-model/computed-properties.md
```

This is possible because Ember extends the `function` prototype. This is no
longer the recommended syntax.
Copy link
Member Author

Choose a reason for hiding this comment

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

Wasn't sure whether or not to just completely remove this and the other one like it for .observes()

@HeroicEric
Copy link
Member Author

closing for #457

this branch is haunted

@HeroicEric HeroicEric closed this Jul 22, 2015
@HeroicEric HeroicEric deleted the encourage-decorator-stye branch July 22, 2015 04:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@HeroicEric @whastings