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

Don't merge - Checking difference between versions #1

Open
wants to merge 1,148 commits into
base: 2-2-stable
Choose a base branch
from

Conversation

kurtfunai
Copy link

No description provided.

Peter Longyear and others added 30 commits September 16, 2014 17:59
A payment state could be `failed`.
This state was missing from the admin variables.
The payment state had no color, it was just a blank circle. I added the red color.
I noticed this in spree 2-2-stable.

Fixes spree#5378

Conflicts:
	backend/app/assets/stylesheets/spree/backend/globals/_variables.scss
…rface, and use pluck rather than collect to load the promotion name and id

Fixes spree#5379
Conflicts:
	core/app/models/spree/payment.rb
	core/spec/spec_helper.rb
Fixes spree#5278
Closes spree#5279
Closes spree#5388

return the new payment state in the update_payment_state, and add some spacing in the spec.

remove the stub to make sure this is not always returning "pending" *doh*

Conflicts:
	core/app/models/spree/order.rb
There was no table closing tag in the variants autocomplete.
Fixes spree#5396
Fixes spree#5397

Conflicts:
	backend/app/views/spree/admin/general_settings/edit.html.erb
Deleted the spinner from the states index
Fixes spree#5402
Added configuration_menu to 'stock locations new'

Fixes spree#5404
Fixes spree#5406

Conflicts:
	backend/app/views/spree/admin/stock_locations/_form.html.erb
…action

Add some missing preventDefault() and 'return false'
Fixes spree#5412
… additional_tax_total & included_tax_total on items back to 0.

Fixes spree#5392
Jeff Dutil and others added 29 commits March 4, 2015 11:04
Forgot to squash into bfb5f90 but it's still
fixed by having removed cookie session usage that was required to exploit this.

Are you sure you JSONP properly?
http://homakov.blogspot.com/2013/02/are-you-sure-you-use-jsonp-properly.html

Conflicts:
	api/app/controllers/spree/api/base_controller.rb
Conflicts:
	backend/app/assets/javascripts/spree/backend/option_value_picker.js
Conflicts:
	core/spree_core.gemspec
This should allow to create a new project with the `spree_i18n` gem and `Store` translation.

During this migration we don't have the store translation table yet, so the `save!` is going to fail.
see spree-contrib/spree_i18n#533

Fixes spree#6259
Previously introduced in 681a966 and later removed probably by accident

Conflicts:
	frontend/spec/controllers/spree/checkout_controller_spec.rb
* This method returns true if the payment amount can be edited

Change Spree::Api::PaymentsController#update to update editable payments

Change payments to be editable when the state is checkout or pending

Fixes spree#6393
Fixes spree#6391

Conflicts:
	core/app/models/spree/order.rb
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.