Skip to content
This repository has been archived by the owner on Apr 18, 2023. It is now read-only.

t macro properties observe locale #319

Merged
merged 3 commits into from
Oct 6, 2015
Merged

Conversation

jamesarosen
Copy link
Owner

  • Fix locales test intermittent failures by sorting the locales
  • Fix ember-i18n on Ember 2.2 beta by reworking the private Stream import
  • Add i18n.locale to the list of dependencies for properties defined via the t macro.

James A. Rosen added 3 commits October 6, 2015 13:27
Tests were failing because the set of locales was correct, but wrongly
ordered. Ordering the list of locales is a sensible thing to do and not
terribly expensive.
Before Ember 2.1, we got the Stream function as the default export of
`ember-metal/streams/stream`. After v2.1, the default export changed to a
`BasicStream` class and the module exports a named `Stream` subclass.
Add `i18n.locale` to the list of dependencies for properties defined via
the `t` macro.
@jamesarosen jamesarosen merged commit 6373c1a into master Oct 6, 2015
jamesarosen pushed a commit that referenced this pull request Oct 6, 2015
@jamesarosen jamesarosen deleted the james/macro-dependencies branch October 6, 2015 20: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.

1 participant