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

Remove invalid call to $.css. Fixes #6319. #6651

Closed
wants to merge 1 commit into from

Conversation

polarblau
Copy link
Contributor

94127f5 introduces a bug by calling $.css without arguments. For details see https://api.jquery.com/css/ .

Fixed at eurucamp 2015 😍 in collaboration with @robinboening for https://rubyissues.ongoodbits.

spree@94127f5 introduces a bug by calling $.css without arguments.
For details see https://api.jquery.com/css/ .
@TeatroIO
Copy link

TeatroIO commented Aug 1, 2015

I've prepared a stage to preview changes. Open stage or view logs.

@tomash
Copy link
Contributor

tomash commented Aug 2, 2015

You should probably try a re-build at circleci, the error seems to stem from db deadlock.

@robinboening
Copy link
Contributor

Yeah, I also think you should try a re-build. (I could not do that, but probably @polarblau can)

JDutil pushed a commit to JDutil/spree that referenced this pull request Aug 5, 2015
spree@94127f5 introduces a bug by calling $.css without arguments.
For details see https://api.jquery.com/css/ .

Fixes spree#6651
@JDutil JDutil closed this in bf152e8 Aug 5, 2015
@JDutil
Copy link
Member

JDutil commented Aug 5, 2015

Thanks

@robinboening
Copy link
Contributor

👍

yushine pushed a commit to yushine/spree that referenced this pull request Dec 8, 2015
spree@94127f5 introduces a bug by calling $.css without arguments.
For details see https://api.jquery.com/css/ .

Fixes spree#6651
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.

5 participants