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

Set speakingurl-rails to 8.0.0, switch to poltergeist gem. #3084

Merged
merged 1 commit into from
Jan 6, 2016

Conversation

parndt
Copy link
Member

@parndt parndt commented Dec 30, 2015

Replaces #3080

@parndt
Copy link
Member Author

parndt commented Dec 30, 2015

@bricesanchez the sprockets error has disappeared, but now getSlug is undefined in the javascript.

@parndt
Copy link
Member Author

parndt commented Dec 30, 2015

@bricesanchez this error is because speakingurl.js is not found

@parndt
Copy link
Member Author

parndt commented Dec 30, 2015

I have to say this is quite a large dependency for such a small feature (used in one place, optionally)

@parndt parndt mentioned this pull request Dec 30, 2015
@bricesanchez
Copy link
Member

Yeah you're right, that's not good for us

@bricesanchez
Copy link
Member

I think i've the fix https://travis-ci.org/refinery/refinerycms/builds/100263612
I've posted a new PR to speakingurl repo: pid/speakingurl#71
We should wait for his version bump

@parndt
Copy link
Member Author

parndt commented Jan 5, 2016 via email

@bricesanchez
Copy link
Member

I'm a little confused on what is the best way to fix this issue.
We could just add the js directly in our assets directory ?

bricesanchez added a commit that referenced this pull request Jan 6, 2016
Set speakingurl-rails to 8.0.0, switch to poltergeist gem.
@bricesanchez bricesanchez merged commit d91d3fe into master Jan 6, 2016
@bricesanchez
Copy link
Member

I've merged this PR because it's fixing the sprocket-rails 3.0 blocker issue.

@bricesanchez bricesanchez deleted the bugfix/speakingurl-rails branch January 27, 2016 14:37
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