Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move ember-cli-string-helpers to dependencies #386

Merged
merged 1 commit into from
Apr 20, 2017
Merged

Conversation

ynnoj
Copy link
Contributor

@ynnoj ynnoj commented Apr 19, 2017

The {{html-safe}} helper here is the cause for the PR. The string helpers in ember-composable-helpers were moved to ember-cli-string-helpers, which is why that add-on is a devDependency of ember-light-table.

I have an Ember add-on which houses a paginated table component built using ELT. ember-light-table is listed as a dependency of that add-on. Installing that add-on in my engines and consuming app results in the following error: Compile Error: html-safe is not a helper.

My fork has fixed this issue for me 🙃

@alexander-alvarez
Copy link
Collaborator

@ynnoj is there any background for this change?
It seems we may not even need this as we already have ember-composable-helpers as a dependency...

https://github.com/romulomachado/ember-cli-string-helpers

@ynnoj
Copy link
Contributor Author

ynnoj commented Apr 20, 2017

@alexander-alvarez I've updated the PR description ☝️

@alexander-alvarez
Copy link
Collaborator

yup. that makes sense.
Thanks for the headsup and clarification. I wasn't following the addon / reading closely enough...
also not a semver change 🤒. Thanks for this @ynnoj

@alexander-alvarez alexander-alvarez merged commit 8bbd8ff into adopted-ember-addons:master Apr 20, 2017
@ynnoj
Copy link
Contributor Author

ynnoj commented Apr 20, 2017

@alexander-alvarez No problem. I'll continue to use my fork until there's a bump. Cheers!

@alexander-alvarez
Copy link
Collaborator

we'll try to get that out for you quick. for posterity, this was introduced only in https://github.com/offirgolan/ember-light-table/releases/tag/v1.8.5
cc @offirgolan

@offirgolan
Copy link
Collaborator

Released with v1.8.6. Thanks for the fix @ynnoj 😄

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.

3 participants