Skip to content

Add support to glimmer-wrapper for MU namespaces#218

Merged
mixonic merged 2 commits intoember-cli:masterfrom
201-created:namespaces
Jan 20, 2018
Merged

Add support to glimmer-wrapper for MU namespaces#218
mixonic merged 2 commits intoember-cli:masterfrom
201-created:namespaces

Conversation

@iezer
Copy link
Contributor

@iezer iezer commented Jan 19, 2018

resolver.resolve now takes a third argument rawString which is the string used at the invocation site of the lookup. For example for:

{{ember-power-select::option}}

The lookup should be:

resolver.resolve('template:component/', null, 'ember-power-select::option')

And for a service example:

Ember.service.inject('auth-addon::main-service')

The lookup would be:

resolver.resolve('service', null, 'auth-addon::main-service')

Refs: #214
Refs: emberjs/ember.js#15350 (comment)
Replaces: #215

resolver.resolve now takes a third argument `rawString` which is the
string used at the invocation site of the lookup. For example for:

```
{{ember-power-select::option}}
```

The lookup should be:

```
resolver.resolve('template:component/', null, 'ember-power-select::option')
``

And for a service example:

```
Ember.service.inject('auth-addon::main-service')
```

The lookup would be:

```
resolver.resolve('service', null, 'auth-addon::main-service')
```

Refs: ember-cli#214
Refs: emberjs/ember.js#15350 (comment)
@mixonic mixonic merged commit ec94b6c into ember-cli:master Jan 20, 2018
@mixonic mixonic deleted the namespaces branch January 20, 2018 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants