Skip to content

fix: support template factory#146

Merged
dfreeman merged 1 commit into
salsify:masterfrom
buschtoens:fix/template-factory
Jul 8, 2019
Merged

fix: support template factory#146
dfreeman merged 1 commit into
salsify:masterfrom
buschtoens:fix/template-factory

Conversation

@buschtoens
Copy link
Copy Markdown
Contributor

emberjs/ember.js#18096

This PR changed how templates are used under the hood. Template modules now export a factory function that expects to be called with the owner.

@dfreeman
Copy link
Copy Markdown
Member

dfreeman commented Jul 1, 2019

Thanks @buschtoens! If you rebase master this should go green https://travis-ci.org/salsify/ember-css-modules/builds/552810259

emberjs/ember.js#18096

This PR changed how templates are used under the hood. Template modules
now export a factory function that expects to be called with the
`owner`.
@buschtoens buschtoens force-pushed the fix/template-factory branch from 456bc34 to 42fc271 Compare July 1, 2019 14:51
@buschtoens
Copy link
Copy Markdown
Contributor Author

@dfreeman dfreeman merged commit 4012e68 into salsify:master Jul 8, 2019
@dfreeman
Copy link
Copy Markdown
Member

dfreeman commented Jul 8, 2019

Merged and released in v1.2.1! Thanks again @buschtoens 🙂

@buschtoens
Copy link
Copy Markdown
Contributor Author

The pleasure is mine 🎩

@buschtoens buschtoens deleted the fix/template-factory branch July 8, 2019 11:54
shahrozAala pushed a commit to shahrozAala/ember-switchback that referenced this pull request Oct 24, 2025
* Upgrade dependencies

* Upgrade addon

* Add glimmer

* Fix button submit

* Fix Frame

* Upgrade FormInput to glimmer

* Upgrade TextInput to glimmer

* Convert radioset

* Convert checkbox

* Update Range to glimmer

* Convert Rater to glimmer

* Remove sparkles

* Fixes for glimmer conversion

* Fix double header

* Fix form validation

* Update how events are bound

* Fix form tests

* Refactor bind-attribute-to-min-width

* Refactor bind-attribute into class-based modifier

* Fix bind-attribute-min-width size calculation

* Fix window being undefined when calling getComputedStyle

* Fix element not being defined

* Remove double nav

* Pin click-addon-docs to master branch

* Fix linting

* Fix linting

* Fix textarea events

* Fix fieldset error display

* Code review changes

* Bump version

* Make class-based-modifier a dependency

* Upgrade ember-css-modules to deal with template factory layout (see salsify/ember-css-modules#146)

* Final 7.0.0
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