Luis Ramos
3d94ce39dd
Merge pull request #5793 from luisramos0/payment_fees
...
Make charges update method update the first pending payment
2020-07-20 21:36:19 +01:00
Luis Ramos
21227d7482
Make charges update method update the first pending payment
...
Updating the first overall payment could select a failed payment and ignore the pending payment that is about to be processed
2020-07-20 18:23:06 +01:00
Luis Ramos
62a3b6b720
Merge pull request #5406 from kristinalim/fix/5300-optimistic_locking_in_stock_items
...
5300 Avoid race conditions in Spree::StockItem
2020-07-17 22:24:59 +01:00
Luis Ramos
152e432f78
Merge pull request #5749 from mbudm/issue/1253
...
Ensure the hero image doesn't pixelate on hamburger menu open
2020-07-17 22:15:46 +01:00
Pau Pérez Fabregat
89906f581d
Merge pull request #5778 from openfoodfoundation/transifex
...
Transifex
2020-07-17 18:37:22 +02:00
Maikel
ea1e23abab
Merge pull request #5756 from mkllnk/broken-spec
...
Make broken spec fail reliably and set it pending
2020-07-17 09:56:56 +10:00
Maikel Linke
b0ac188430
Make broken spec fail reliably and set it pending
...
This spec has been broken for a long time, at least eight months. But it
regularly passed because the search filter is applied with a delay and
in that time the content matches. And once the filter is applied, no
products are shown and the negative matchers pass.
2020-07-17 09:28:17 +10:00
Transifex-Openfoodnetwork
f31a1ff59c
Updating translations for config/locales/en_GB.yml
2020-07-17 04:10:57 +10:00
Luis Ramos
2fe37b5237
Update all locales with the latest Transifex translations
2020-07-16 18:07:40 +01:00
Luis Ramos
981f1c0932
Merge pull request #5754 from openfoodfoundation/transifex
...
Transifex
2020-07-16 18:06:17 +01:00
Maikel
4e3502cce7
Merge pull request #5766 from mkllnk/5764-fix-pin-payments
...
5764 Provide credit card brand to Pin Payments
2020-07-16 10:28:18 +10:00
Matt-Yorkley
19b3041e1d
Merge pull request #5768 from Matt-Yorkley/pin-payments-params
...
Update StrongParams for Pin Payments
2020-07-16 00:01:34 +02:00
Pau Perez
4e00c45782
Doc defensive coding needed by pin payments
...
[skip ci]
2020-07-15 16:46:09 +02:00
Luis Ramos
31e072179b
Make method a little simple by extracting method
2020-07-15 15:23:16 +01:00
Luis Ramos
57fd1bdee2
Merge pull request #5744 from openfoodfoundation/dependabot/bundler/i18n-js-3.7.1
...
Bump i18n-js from 3.7.0 to 3.7.1
2020-07-15 15:04:55 +01:00
Matt-Yorkley
c009fed6be
Update StrongParams for Pin Payments
2020-07-15 14:00:19 +02:00
dependabot-preview[bot]
5fae2c08c8
Bump i18n-js from 3.7.0 to 3.7.1
...
Bumps [i18n-js](https://github.com/fnando/i18n-js ) from 3.7.0 to 3.7.1.
- [Release notes](https://github.com/fnando/i18n-js/releases )
- [Changelog](https://github.com/fnando/i18n-js/blob/master/CHANGELOG.md )
- [Commits](https://github.com/fnando/i18n-js/compare/v3.7.0...v3.7.1 )
Signed-off-by: dependabot-preview[bot] <support@dependabot.com >
2020-07-15 08:13:42 +00:00
Pau Pérez Fabregat
d4fea5223d
Merge pull request #5762 from openfoodfoundation/dependabot/bundler/oj-3.10.7
...
Bump oj from 3.10.6 to 3.10.7
2020-07-15 10:13:19 +02:00
Maikel Linke
25155cab18
Provide credit card brand to Pin Payments
...
Spree replaced the Ruby code providing the credit card brand with a
jquery plugin providing it on the checkout screen.
I re-added Ruby code because it's easier and more robust than updating
the user interface with new Javascript.
2020-07-15 17:13:46 +10:00
Steve Roberts
dbc7632c4e
Add inline comment to explain two height properties
2020-07-15 09:52:04 +10:00
Steve Roberts
c4d7899a99
Use vh units for new browsers and align tagline bg to top.
2020-07-14 19:26:12 +10:00
Transifex-Openfoodnetwork
f79269e96b
Updating translations for config/locales/ca.yml
2020-07-14 17:46:12 +10:00
Steve Roberts
60870a1215
Fix linting errors
2020-07-14 12:58:48 +10:00
Steve Roberts
63a080266e
Merge branch 'master' of https://github.com/openfoodfoundation/openfoodnetwork into issue/1253
2020-07-14 12:45:14 +10:00
dependabot-preview[bot]
f4cb14ab1a
Bump oj from 3.10.6 to 3.10.7
...
Bumps [oj](https://github.com/ohler55/oj ) from 3.10.6 to 3.10.7.
- [Release notes](https://github.com/ohler55/oj/releases )
- [Changelog](https://github.com/ohler55/oj/blob/develop/CHANGELOG.md )
- [Commits](https://github.com/ohler55/oj/compare/v3.10.6...v3.10.7 )
Signed-off-by: dependabot-preview[bot] <support@dependabot.com >
2020-07-14 00:10:40 +00:00
Luis Ramos
9c8318d828
Merge pull request #5613 from luisramos0/calculators
...
Move all Calculators from spree to OFN and out of the Spree namespace
2020-07-13 15:16:15 +01:00
Luis Ramos
d536cc8108
Merge pull request #5755 from Matt-Yorkley/groups-images
...
Fix StrongParameters for groups images
2020-07-13 14:50:24 +01:00
Matt-Yorkley
f450c21308
Merge pull request #5759 from luisramos0/vo_spec
...
Comment very flaky spec
2020-07-10 21:44:03 +02:00
Luis Ramos
eb4d648a23
Comment this very flaky spec for now
2020-07-10 19:27:31 +01:00
Transifex-Openfoodnetwork
50427472c4
Updating translations for config/locales/en_NZ.yml
2020-07-10 20:25:20 +10:00
Maikel
c1d700196b
Merge pull request #5739 from Matt-Yorkley/flaky-spec-complex-oc
...
Flaky spec order_cycles/complex_updating_specific_time
2020-07-10 14:59:59 +10:00
Luis Ramos
cc3361aaac
Merge pull request #5659 from dmarcoux/issue-5555
...
Filter orders on inclusive dates in admin/orders
2020-07-09 20:40:45 +01:00
Luis Ramos
ed915006fa
Merge pull request #5738 from openfoodfoundation/dependabot/bundler/wicked_pdf-2.1.0
...
Bump wicked_pdf from 1.4.0 to 2.1.0
2020-07-09 20:40:11 +01:00
Matt-Yorkley
1a132924d2
Fix StrongParameters for groups images
2020-07-09 10:00:02 +02:00
Transifex-Openfoodnetwork
e3980521ea
Updating translations for config/locales/fr_CA.yml
2020-07-09 07:58:41 +10:00
Transifex-Openfoodnetwork
e6a2eb8af9
Updating translations for config/locales/en_CA.yml
2020-07-09 07:53:34 +10:00
Luis Ramos
fabddbd1c0
Merge pull request #5746 from coopdevs/remove-specs-dead-code
...
Remove dead specs helper method
2020-07-08 12:38:24 +01:00
Steve Roberts
2f562809c0
Ensure the hero image doesn't attempt to use the full height of all page content
...
Not sure exactly why this happens, but when the mobile nav is opened the hero image at #tagline:before uses the height of the full window - often around 4000px. Adding max-height of 100% to the nearest safe parent prevents this behaviour.
2020-07-08 21:31:52 +10:00
Matt-Yorkley
a763a5ee0b
Update all locales with the latest Transifex translations
2020-07-08 12:49:20 +02:00
Matt-Yorkley
d718eb9a23
Merge pull request #5726 from openfoodfoundation/transifex
...
Transifex
2020-07-08 12:33:09 +02:00
Matt-Yorkley
d78db16abe
Merge pull request #5718 from Matt-Yorkley/available-locales-bug
...
Available locales bug
2020-07-08 12:30:18 +02:00
Matt-Yorkley
e742069959
Refactor UI interactions for opening exchange product tabs in OC edit
2020-07-08 11:11:39 +02:00
Matt-Yorkley
c8254b8338
Adjust exchange_row assertion to use slightly more specific criteria
2020-07-08 11:11:39 +02:00
Matt-Yorkley
be1e39f0cb
Ensure next page has loaded before interacting with form elements
2020-07-08 11:11:39 +02:00
Pau Perez
a1e8c8ad03
Remove dead specs helper method
...
I found it while reviewing
https://github.com/openfoodfoundation/openfoodnetwork/pull/5718#discussion_r451390809 .
2020-07-08 10:59:13 +02:00
Luis Ramos
312e78ba61
Merge pull request #5742 from Matt-Yorkley/flaky-spec-shopping
...
Fix flaky shopping_spec
2020-07-07 18:11:00 +01:00
Luis Ramos
207c6bc205
Merge pull request #5741 from Matt-Yorkley/flaky-spec-exchange-renderer
...
Re-enable and fix flaky exchange products renderer spec
2020-07-07 18:09:02 +01:00
Luis Ramos
f8f7b30095
Merge pull request #5716 from openfoodfoundation/dependabot/bundler/database_cleaner-1.8.5
...
Bump database_cleaner from 1.7.0 to 1.8.5
2020-07-07 17:29:10 +01:00
Luis Ramos
ebc1ab4469
Merge pull request #5710 from openfoodfoundation/dependabot/bundler/mini_racer-0.2.15
...
Bump mini_racer from 0.2.14 to 0.2.15
2020-07-07 17:28:33 +01:00
Luis Ramos
5e6739c9f7
Fix long lines in new spec
2020-07-07 17:26:24 +01:00
Luis Ramos
99e12b6f07
Convert calculators in new spec
2020-07-07 17:26:24 +01:00
Luis Ramos
0b79e7c48c
Fix rubucop issue in enterprise_fee model
2020-07-07 17:26:24 +01:00
Luis Ramos
d92f97716a
Extract compute_for and thus resolve the rubocop complexity issue
2020-07-07 17:26:24 +01:00
Luis Ramos
68359d4d1a
Remove unnecessary variable (this sum and += come from the original
...
spree code and is not necessary here now)
2020-07-07 17:26:24 +01:00
Luis Ramos
70432e301d
Early exit if max is zero to make method shorted
2020-07-07 17:26:24 +01:00
Luis Ramos
fbe0a3246d
Extract applicable_rate? and thus resolve rubocop complexity issue
2020-07-07 17:26:24 +01:00
Luis Ramos
b096717172
Extract per_order_fees_total out of compute_order
2020-07-07 17:26:24 +01:00
Luis Ramos
21120dd6ab
Extract per_item_fees_total out of compute_order
2020-07-07 17:26:24 +01:00
Luis Ramos
4b12a5f592
Extract line_items_total from compute_order
2020-07-07 17:26:24 +01:00
Luis Ramos
eedf31e449
Make migration a bit easier to read
2020-07-07 17:26:24 +01:00
Luis Ramos
e225c5ce3a
Migrate calculators to outside spree namespace
2020-07-07 17:26:24 +01:00
Luis Ramos
f62546254f
Define DefaultTax calculator outside of spree namespace the tax_rate to be used by the app and make spree specs use calculators outside the spree namespace
2020-07-07 17:26:24 +01:00
Luis Ramos
6a94168ee5
Remove Spree namespace from DefaultTax
2020-07-07 17:26:24 +01:00
Luis Ramos
9c7cb23262
Move calculator specs out of the spree namespace
2020-07-07 17:26:24 +01:00
Luis Ramos
5e8438c446
Move all calculators outside the spree namespace
2020-07-07 17:26:23 +01:00
Luis Ramos
9c3eb3725e
Remove dead code related to promotions, we dont have promotions in OFN
2020-07-07 17:20:22 +01:00
Luis Ramos
a544102b1b
Fix some rubocop issues
2020-07-07 17:20:22 +01:00
Luis Ramos
f2a46d2cef
Merge decorators into original classes brought from spree_core
2020-07-07 17:20:22 +01:00
Luis Ramos
6b04df0dea
Fix indentation and extract Calculator as a separate module declaration
2020-07-07 17:20:22 +01:00
Luis Ramos
c2898ba389
Bring calculators from spree_core
2020-07-07 17:20:22 +01:00
Transifex-Openfoodnetwork
6be0ae543a
Updating translations for config/locales/pt_BR.yml
2020-07-07 21:46:44 +10:00
Matt-Yorkley
f2e0b0f35a
Ensure products list has loaded before interacting with product list UI elements
2020-07-07 13:14:11 +02:00
Matt-Yorkley
0f659d0f5e
Merge pull request #5728 from luisramos0/ams_comment
...
Update ams dependency comment
2020-07-07 12:32:03 +02:00
Matt-Yorkley
30492157f9
Re-enable and fix flaky exchange products renderer spec
...
For some inexplicable reason `order_cycle.exchanges.incoming.first` and `order_cycle.exchanges.incoming.second` were returning the same exchange. Subsequently the inventory items being created were for the same variant, which throws an error due to a uniqueness validation in InventoryItem on variant_id. Changing from #first and #second to #first and #last results in the correct exchanges being assigned.
2020-07-07 11:21:18 +02:00
Luis Ramos
c1fa4fcc85
Merge pull request #5736 from Matt-Yorkley/flaky-spec-embedded
...
Fix flaky embedded shopping spec
2020-07-07 09:36:27 +01:00
dependabot-preview[bot]
4bee038109
Bump wicked_pdf from 1.4.0 to 2.1.0
...
Bumps [wicked_pdf](https://github.com/mileszs/wicked_pdf ) from 1.4.0 to 2.1.0.
- [Release notes](https://github.com/mileszs/wicked_pdf/releases )
- [Changelog](https://github.com/mileszs/wicked_pdf/blob/master/CHANGELOG.md )
- [Commits](https://github.com/mileszs/wicked_pdf/compare/1.4.0...2.1.0 )
Signed-off-by: dependabot-preview[bot] <support@dependabot.com >
2020-07-06 23:50:22 +00:00
Matt-Yorkley
447a873e5c
Fix flaky embedded spec
...
The new cart sidebar takes 300ms to animate in. If we try to click the button on the left during this time, we may click the button on the right as it slides in.
2020-07-07 00:23:43 +02:00
Matt-Yorkley
3d955a952c
Merge pull request #5674 from mkllnk/caching_spec
...
Fix flaky caching spec
2020-07-06 21:56:02 +02:00
Luis Ramos
ce1114556d
Merge pull request #5578 from mbudm/issue/3770-manual-update
...
Update orders swagger spec and fix api_key access to orders endpoint
2020-07-06 20:32:25 +01:00
Luis Ramos
761871c90e
Merge pull request #5715 from luisramos0/ship_method
...
Fix issue in Spree 2.1 and bring Spree::Stock classes to OFN 🎉
2020-07-06 19:15:50 +01:00
Transifex-Openfoodnetwork
2a68e0fbba
Updating translations for config/locales/tr.yml
2020-07-07 03:55:56 +10:00
Luis Ramos
ceac8a5d40
Merge pull request #5725 from Matt-Yorkley/admin-order-images
...
Fix default image display in admin order edit
2020-07-06 14:03:56 +01:00
Luis Ramos
b0dd83b773
Update ams dependency comment
2020-07-05 19:55:53 +01:00
Dany Marcoux
3dfabdc117
Filter orders on inclusive dates in admin/orders
...
Closes #5555
Co-authored-by: Luis Ramos <luisramos0@gmail.com >
2020-07-05 20:12:16 +02:00
Transifex-Openfoodnetwork
b951bcc379
Updating translations for config/locales/en_IN.yml
2020-07-05 04:34:29 +10:00
Transifex-Openfoodnetwork
6f6e8c0c92
Updating translations for config/locales/en_IN.yml
2020-07-05 04:29:50 +10:00
Transifex-Openfoodnetwork
787648eff2
Updating translations for config/locales/en_FR.yml
2020-07-04 23:52:01 +10:00
Transifex-Openfoodnetwork
7a4a673578
Updating translations for config/locales/fr.yml
2020-07-04 23:51:45 +10:00
Matt-Yorkley
96138e9129
Refactor UserLocaleSetter public methods
2020-07-04 10:15:35 +02:00
Matt-Yorkley
f08a530877
Make #valid_locale_for_user an instance method
2020-07-04 09:51:00 +02:00
Luis Ramos
d2adcd20e5
Merge pull request #5719 from coopdevs/index-state-changes
...
Index spree_state_changes.stateful_id column
2020-07-03 23:02:01 +01:00
Luis Ramos
ff9133b7aa
Merge pull request #5699 from coopdevs/fix-order-button-radius
...
Make button border radius consistent
2020-07-03 23:01:14 +01:00
Luis Ramos
b917238d1e
Merge pull request #5446 from luisramos0/sca_refunds
...
Make StripeSCA void action work
2020-07-03 22:45:35 +01:00
Transifex-Openfoodnetwork
45b960a7ee
Updating translations for config/locales/pt_BR.yml
2020-07-04 07:39:50 +10:00
Matt-Yorkley
e2626a0c3b
Add unit tests for UserLocaleSetter service
2020-07-03 23:14:38 +02:00
Matt-Yorkley
46c40bdf33
Fix default image display in admin order edit
2020-07-03 22:58:29 +02:00
Luis Ramos
a472b6525f
Merge pull request #5707 from luisramos0/missing_spaces
...
Add space after translations so that translators dont have to guess a space is needed
2020-07-03 16:10:55 +01:00
Matt-Yorkley
05c1f093b2
Delete dead code
...
There's nothing done in this bit of code that isn't already done somewhere else, and more effectively.
2020-07-03 16:44:37 +02:00
Matt-Yorkley
0c2fd4bfd2
Fix ensure_valid_locale_persisted and change public interface of service
2020-07-03 16:39:24 +02:00
Matt-Yorkley
cc7b5e2df3
Add pending test for setting locale from cookies during login
...
This test currently fails
2020-07-03 16:39:23 +02:00
Luis Ramos
804450bcc5
Fix buggy spec
...
The different shipping method was in the page but only as an option in the dropdown, not as the final selected shipping method! That was the cause of bug #5694 . We now check for the label Shipping which preceeds the final shipping method selection in the order page
2020-07-03 15:35:13 +01:00
Luis Ramos
b883a0eb75
Fix easy rubocop issues in shipment_spec
2020-07-03 13:48:12 +01:00
Luis Ramos
7b89b52ab8
Transpec shipment_spec brough from spree_core
2020-07-03 13:48:12 +01:00
Luis Ramos
07a44cfaf3
Update selected shipping rate if there is an original shipping method to keep and it is different from the one selected through the Estimator process
...
Make sure the shipment is saved (callbacks!) whenever the ship method has changed in the refresh_rates process
2020-07-03 13:48:12 +01:00
Luis Ramos
cd60ee2116
Use flat_map to make ship methods selection faster
2020-07-03 13:48:08 +01:00
Matt-Yorkley
ab63d2234c
Guard against nils in conditions
2020-07-03 14:22:32 +02:00
Matt-Yorkley
078726dcca
Add explanatory comment on saving selected locale in cookies
2020-07-03 14:21:45 +02:00
Matt-Yorkley
dab0add492
Make conditional more concise
2020-07-03 14:20:51 +02:00
Matt-Yorkley
ebffa381c6
Update cookies[:locale] logic
2020-07-03 14:06:05 +02:00
Matt-Yorkley
faa7c0a7c5
Extract save_cookies_from_locale comment-method
2020-07-03 14:03:52 +02:00
Matt-Yorkley
02549d1b0f
Extract all locale-setting logic to a service
2020-07-03 13:44:13 +02:00
Matt-Yorkley
7a00a3ba1e
Rename method and add explanatory comment
2020-07-03 11:52:21 +02:00
Matt-Yorkley
c726340ba3
Extract #save_locale_from_params comment-method
2020-07-03 11:48:39 +02:00
Luis Ramos
fcaa9d1b37
Remove extra space from base translations in closing and welcome to
2020-07-03 10:32:09 +01:00
Matt-Yorkley
d2db9b3a66
Merge pull request #5599 from luisramos0/submit_on_enteer
...
Make Enter submit the form on both products and orders admin
2020-07-03 11:14:57 +02:00
Luis Ramos
91026e6606
Merge pull request #5601 from jeduardo824/enhancement/5454-links-to-shops-in-new-tabs
...
Enhancement/5454 make links to shops under Groups > Producers open in new tab
2020-07-02 23:51:59 +01:00
Luis Ramos
d9ab7a8d49
Merge pull request #5667 from mkllnk/shopping-spec
...
Stabilise shopping specs and open them for change
2020-07-02 23:51:25 +01:00
Luis Ramos
758bb17142
Fix some easy rubocop issues and add some exceptions to to manual todo list
2020-07-02 21:24:15 +01:00
Luis Ramos
659de3d24d
Replay spree commit a4622ee13a723f0dba2943967b445b9989f67fb2 to fix issue introduced in spree 2.1
2020-07-02 21:24:15 +01:00
Luis Ramos
21ec6ccf0d
Remove unused sort packages from prioritizer
2020-07-02 21:24:15 +01:00
Luis Ramos
d323c5bdcb
Simplify packer and coordinator baed on the fact that there's only one stock_location so there will only be one package per order
2020-07-02 21:24:15 +01:00
Luis Ramos
ff046f7a6c
Remove conditionals related to Config.track_inventory_levels, this config is always true in OFN
2020-07-02 21:24:14 +01:00
Luis Ramos
ee937988e8
Fix easy rubocop issues
2020-07-02 21:24:14 +01:00
Luis Ramos
ee66e37521
Move adjuster, estimator, packer and prioritizer to order management engine
2020-07-02 21:24:14 +01:00
Luis Ramos
83974a832c
Move Coordinator from Spree::Stock to OrderManagement::Stock
2020-07-02 21:24:14 +01:00
Luis Ramos
01b1abbd52
Bring method from Spree::Order so that we can move Coordiantor to the order management engine
2020-07-02 21:24:14 +01:00
Luis Ramos
b487185a65
Remove package factory, it is no longer needed, we can just call the Package class in the two places where it is used
2020-07-02 21:24:14 +01:00
Luis Ramos
f0b3ed0d79
Merge Spree::Stock::Package into OrderManagement::Stock::Package
2020-07-02 21:24:14 +01:00
Luis Ramos
943cb7bf05
Move Stock::Package to OrderManagement::Stock::Package
2020-07-02 21:24:14 +01:00
Luis Ramos
1b28592f58
Now that the stock code is on out side we can clean up!
...
Remove everything related to splitters (including bringing environment.rb so we remove the splitters variable from it
2020-07-02 21:24:14 +01:00
Luis Ramos
8e116dd58a
Make ensure_correct_adjustment a public method because we call it in OFN
2020-07-02 21:24:14 +01:00
Luis Ramos
46cf106047
Fix shipment spec brought from spree
2020-07-02 21:24:14 +01:00
Luis Ramos
2e33e02d7f
Remove dead code, this method was removed in spree 2.0.4
2020-07-02 21:24:14 +01:00
Luis Ramos
7a03f57da0
Merge shipment decorator with class brought from spree_core
2020-07-02 21:24:14 +01:00
Luis Ramos
494251b7cf
Fix simple rubocop issues
2020-07-02 21:24:14 +01:00
Luis Ramos
4e5259f491
Bring shipment from spree_core
2020-07-02 21:24:14 +01:00
Luis Ramos
55a4021157
Convert to modern rspec syntax
2020-07-02 21:24:14 +01:00
Luis Ramos
bdb40d68e9
Fix easy rubocop issues
2020-07-02 21:24:14 +01:00
Luis Ramos
b7255130b6
Bring Package from spree_core
2020-07-02 21:24:14 +01:00
Luis Ramos
0ca8b6aab6
Fix easy rubocop issue
2020-07-02 21:24:14 +01:00
Luis Ramos
3e14c9777e
Merge decorator with class brought from spree_core
2020-07-02 21:24:14 +01:00
Luis Ramos
d505fc2131
Bring availability validator from spree_core
2020-07-02 21:24:14 +01:00
Luis Ramos
720ad9de0e
Convert specs to modern rsspec syntax
2020-07-02 21:24:14 +01:00
Luis Ramos
feadbb086f
Adapt spec to OFN context
2020-07-02 21:24:14 +01:00
Luis Ramos
b16db2f40e
Fix easy rubocop issues
2020-07-02 21:24:14 +01:00
Luis Ramos
8c3b8c4db5
Bring estimator from spree_core
2020-07-02 21:24:14 +01:00
Luis Ramos
4711a7469a
Adapt coordinator spec to ofn and remove spec that is not applicable (multi stock locations)
2020-07-02 21:24:14 +01:00
Luis Ramos
69b9cfbad2
Make packer use BasicSplitter by default
2020-07-02 21:24:14 +01:00
Luis Ramos
fdc085f701
Convert to modern rspec and remove specs not applicable to ofn
2020-07-02 21:24:14 +01:00
Luis Ramos
ccf928df12
Fix simple rubocop issues
2020-07-02 21:24:14 +01:00
Luis Ramos
e0f9894b7a
Bring packer from spree_core
2020-07-02 21:24:14 +01:00
Luis Ramos
ec50a788a6
Fix easy rubocop issues
2020-07-02 21:24:14 +01:00
Luis Ramos
c2ec34ab9f
Bring coordinator from spree_core
2020-07-02 21:24:14 +01:00
Luis Ramos
a6d7acb6f1
Convert spec to modern rspec syntax
2020-07-02 21:24:14 +01:00
Luis Ramos
eb13595fd3
Fix simple rubocop issues
2020-07-02 21:24:14 +01:00
Luis Ramos
3ae2877d4e
Bring adjuster and prioritizer from spree_core
2020-07-02 21:24:14 +01:00
Luis Ramos
d18fec7125
Move Base splitter from main app models to order management engine services
2020-07-02 21:24:14 +01:00
Luis Ramos
735ee1e7ed
Fix simple rubocop issues
2020-07-02 21:24:14 +01:00
Luis Ramos
4fce506201
Bring splitter/base from spree
2020-07-02 21:24:14 +01:00
Luis Ramos
658c77f3f5
Merge pull request #5701 from Matt-Yorkley/noimage
...
Noimage
2020-07-02 20:17:04 +01:00
Luis Ramos
510449547f
Merge pull request #5644 from coopdevs/manage-font-with-asset-pipeline
...
Manage OFN's font through the asset pipeline
2020-07-02 20:09:08 +01:00
Matt-Yorkley
8fc407a9d5
Extract repeated variant thumbnail code to a shared partial
2020-07-02 18:36:43 +02:00
Matt-Yorkley
eeb6c57f99
Update missing image paths in other order views
2020-07-02 18:36:27 +02:00
Matt-Yorkley
b10fd99a53
Update all locales with the latest Transifex translations
2020-07-02 16:46:53 +02:00
Matt-Yorkley
e328dc7657
Merge pull request #5717 from openfoodfoundation/transifex
...
Transifex
2020-07-02 16:43:58 +02:00
Pau Perez
dee61e8b82
Index spree_state_changes.stateful_id column
...
This speeds up a great deal one of the most awful queries our DB servers
execute. It's not rare to see traces above 20s in Datadog 😱 .
In staging, with no traffic, we go from
```
EXPLAIN ANALYZE SELECT COUNT ( * )
FROM spree_state_changes
WHERE spree_state_changes . stateful_id = 2024
AND spree_state_changes . stateful_type = 'Spree::Order';
Planning time: 0.142 ms
Execution time: 9.073 ms
```
to
```
EXPLAIN ANALYZE SELECT COUNT ( * )
FROM spree_state_changes
WHERE spree_state_changes . stateful_id = 2024
AND spree_state_changes . stateful_type = 'Spree::Order';
Planning time: 0.284 ms
Execution time: 0.202 ms
```
2020-07-02 15:39:08 +02:00
Matt-Yorkley
4e08d2049f
Ensure a valid locale is persisted during login
...
This action has to be performed here and not in I18nHelper, as spree_current_user is not initialized yet during the other checks / setting the selected locale value in the app
2020-07-02 15:33:19 +02:00
Matt-Yorkley
91880cdbec
Make I18nHelper #available_locale? method public
2020-07-02 14:35:59 +02:00
Matt-Yorkley
d70d61439a
Always return an available locale
2020-07-02 14:35:59 +02:00
Matt-Yorkley
aa6f4d4fb9
Don't set unavailable locales on the current user
2020-07-02 14:35:59 +02:00
Matt-Yorkley
8dfaea629b
Refactor current_user_locale to a new method
2020-07-02 14:35:59 +02:00
Matt-Yorkley
3326366c6e
Add specs for applying stored locales during login
2020-07-02 14:35:58 +02:00
dependabot-preview[bot]
cc34f126ec
Bump mini_racer from 0.2.14 to 0.2.15
...
Bumps [mini_racer](https://github.com/discourse/mini_racer ) from 0.2.14 to 0.2.15.
- [Release notes](https://github.com/discourse/mini_racer/releases )
- [Changelog](https://github.com/rubyjs/mini_racer/blob/master/CHANGELOG )
- [Commits](https://github.com/discourse/mini_racer/compare/v0.2.14...v0.2.15 )
Signed-off-by: dependabot-preview[bot] <support@dependabot.com >
2020-07-02 11:22:57 +00:00
Pau Pérez Fabregat
4ced16c206
Merge pull request #5709 from luisramos0/empty
...
Replace empty? with blank? and replace unless empty? with if present?
2020-07-02 11:36:37 +02:00
Transifex-Openfoodnetwork
1b8669bb0b
Updating translations for config/locales/en_NZ.yml
2020-07-02 18:28:33 +10:00
Pau Pérez Fabregat
c1b635903c
Merge pull request #5658 from openfoodfoundation/transifex
...
Transifex
2020-07-02 10:05:16 +02:00
dependabot-preview[bot]
eff9d6edd0
Bump database_cleaner from 1.7.0 to 1.8.5
...
Bumps [database_cleaner](https://github.com/DatabaseCleaner/database_cleaner ) from 1.7.0 to 1.8.5.
- [Release notes](https://github.com/DatabaseCleaner/database_cleaner/releases )
- [Changelog](https://github.com/DatabaseCleaner/database_cleaner/blob/master/History.rdoc )
- [Commits](https://github.com/DatabaseCleaner/database_cleaner/compare/v1.7.0...v1.8.5 )
Signed-off-by: dependabot-preview[bot] <support@dependabot.com >
2020-07-02 00:06:17 +00:00
Luis Ramos
7dcc2bb27c
Merge pull request #5703 from Matt-Yorkley/checkout-addresses
...
Checkout addresses
2020-07-01 19:06:29 +01:00
Luis Ramos
e8417b8be6
Remove specs testing filtering of master variants
...
Master variants are not used in the report
2020-07-01 18:13:30 +01:00
Luis Ramos
6555f8bfba
Add specs to cover stripeSCA refunds
...
Duplication between stripe connect and stripeSCA is done on purpose so we can easily delete stripeConnect code when the migration is done
2020-07-01 18:13:27 +01:00
Pau Pérez Fabregat
66ddde1808
Merge pull request #5682 from Matt-Yorkley/in-digest-ion
...
Move default versions of Paperclip-managed images to /public
2020-07-01 18:02:15 +02:00
Pau Pérez Fabregat
28376b1056
Merge pull request #5561 from coopdevs/defend-from-invoice-without-bill-address
...
Do not print the bill addr. name when there's none
2020-07-01 17:50:09 +02:00
Luis Ramos
2d4326ded3
Replace empty? with blank? which is equivalent but a bit more resilient, returns false for nil
...
Present is not blank, so unless blank? becomes if present?
2020-07-01 14:08:07 +01:00
Luis Ramos
ce493866f9
Extract refunds specs from payments controller spec
2020-07-01 13:19:38 +01:00
Luis Ramos
ecb1920fa9
Move payment_controller_spec to specific folder so we can break it in more specific parts
2020-07-01 13:14:39 +01:00
Luis Ramos
9e4a793b24
Fix rubocop issues
2020-07-01 13:14:39 +01:00
Luis Ramos
2179cc7faf
Make StripeSCA void action make a refund instead
...
StripeSCA does not support voiding confirmed payment intents so we need to make a refund instead
2020-07-01 13:10:56 +01:00
Luis Ramos
63934b7863
Add space after translation so that translators dont have to guess a space is needed after the translation
2020-07-01 11:56:37 +01:00
Pau Pérez Fabregat
819af03239
Merge pull request #5706 from luisramos0/ent
...
Fix admin enterprises controller spec
2020-07-01 12:28:19 +02:00
Pau Pérez Fabregat
5a72526107
Merge pull request #5692 from luisramos0/taxonomies
...
Fix taxonomies flaky spec
2020-07-01 12:26:56 +02:00
Pau Perez
8c371fd0d1
Do not print bill addr. in invoice2 either
2020-07-01 11:27:57 +02:00
Matt-Yorkley
49fb01a3bb
Add assign default value of current time whilst altering NOT NULL constraint
...
This argument in #change_column_null assigns this value where any NULL values are found, but it doesn't alter the table's `:default` value
2020-07-01 10:00:21 +02:00
Luis Ramos
2dce10bec2
Fix admin enterprises controller spec, for some reason the correct controller is not being picked up in the test
2020-06-30 10:49:45 +01:00
Matt-Yorkley
11da385624
Enable pending checkout spec
2020-06-30 11:30:16 +02:00
Matt-Yorkley
3e7f5a4ea9
Update user address tests
...
I considered deleting these tests, as they're not very good and are testing Rails functionality. I decided to leave them in case something explodes in a future upgrade. For reference: there are issues in Rails 4 when using `object.clone.attributes`, and with assigning a hash containing `created_at` and `updated_at` values with `object.update(attributes_hash)`.
2020-06-30 11:30:16 +02:00
Matt-Yorkley
87cd936c94
Blacklist timestamp attributes when assigning for update
...
Doing #clone or #dup of an object's attributes then passing it to an #update/#update_attributes call means we are manually passing values for created_at and updated_at, which can cause problems, especially if the object being duped hasn't been persisted yet: in this case we would be manually attempting to save timestamps with nil values, which is not a good idea. Here they are blacklisted from the attributes hash.
2020-06-30 11:30:16 +02:00
Matt-Yorkley
ce8908f53b
Use #dup when copying attributes hash
...
In Rails 4, #clone behaves differently. The attributes hash of the cloned object is shared with the original, it's not a separate object! https://github.com/rails/rails/blob/4-0-stable/activerecord/lib/active_record/core.rb#L217-L220
2020-06-30 11:30:16 +02:00
Matt-Yorkley
abe96c6f32
Add specs for updating existing default addresses during checkout
...
These specs fail with the "NULL values for created_at / updated_at" errors we've been seeing.
2020-06-30 11:30:14 +02:00
Pau Pérez Fabregat
e69d3e5e0c
Merge pull request #5693 from luisramos0/knapsack
...
Reload knapsack report
2020-06-30 09:43:36 +02:00
Pau Pérez Fabregat
2a1fe3f05f
Merge pull request #5684 from openfoodfoundation/dependabot/bundler/ddtrace-0.37.0
...
Bump ddtrace from 0.36.0 to 0.37.0
2020-06-30 09:41:48 +02:00
Matt-Yorkley
57ba2481c4
Require timestamps
2020-06-30 09:09:35 +02:00
Pau Perez
0c93de8298
Manage OFN's font through the asset pipeline
2020-06-30 09:02:36 +02:00
Pau Perez
23178763b0
Do not print the bill addr. name when there's none
...
The error
```
ActionView::Template::Error: undefined method `full_name' for nil:NilClass
```
happens a few times a day and raises exceptions we don't pay attention to.
They add unnecessary noise that hides other more relevant issues.
This, however, is a symptom of a deeper data integrity problem that needs
solving at some point. This is just a countermeasure.
2020-06-30 08:46:01 +02:00
Luis Ramos
de8b97b2e1
Merge pull request #5526 from cillian/drop-blockenspiel
...
Drop blockenspiel
2020-06-29 20:09:26 +01:00
Matt-Yorkley
be0131b94c
Merge pull request #5645 from coopdevs/fix-scss-deprecation-warning
...
Rename *.css.scss to *.scss
2020-06-29 20:36:42 +02:00
Matt-Yorkley
bd0e4c75c4
Merge pull request #5636 from fatihorhan/make-user-guide-links-consistent
...
Update Sell page to use User Guide links from configuration
2020-06-29 20:33:00 +02:00
Matt-Yorkley
b441ac2644
Update paths to "noimage" images
2020-06-29 20:07:08 +02:00
Matt-Yorkley
5bec887f93
Move /assets/images/noimage/* to /public/noimage/*
2020-06-29 19:58:38 +02:00
Cillian O'Ruanaidh
a9664e103d
Merge latest :master into :drop-blockenspiel branch and resolve conflicts related to Rubocop fixes.
2020-06-29 18:11:08 +01:00
Pau Perez
c892f9c69b
Make button border radius consistent
2020-06-29 12:01:59 +02:00
Transifex-Openfoodnetwork
bb80c81097
Updating translations for config/locales/en_US.yml
2020-06-29 05:22:13 +10:00
Luis Ramos
6516037420
Reload knapsack report
2020-06-27 20:36:33 +01:00
Luis Ramos
10b8c2b789
Relax the assertion and dont check row number so that it passes even if order of elements is not the same
2020-06-27 15:54:58 +01:00
Matt-Yorkley
b325e48957
Update content config spec to use new default image paths
2020-06-27 10:32:38 +02:00
Matt-Yorkley
a5f541b59d
Update admin logo path
2020-06-27 08:53:05 +02:00
Matt-Yorkley
ff8cb43717
Move default versions of Paperclip-managed images to /public
...
Image paths for these files are handled by Paperclip, which (with our current version) does not handle fingerprints on assets. Therefore we need to make these images available under /public
2020-06-27 08:45:59 +02:00
dependabot-preview[bot]
032a70fa7e
Bump ddtrace from 0.36.0 to 0.37.0
...
Bumps [ddtrace](https://github.com/DataDog/dd-trace-rb ) from 0.36.0 to 0.37.0.
- [Release notes](https://github.com/DataDog/dd-trace-rb/releases )
- [Changelog](https://github.com/DataDog/dd-trace-rb/blob/master/CHANGELOG.md )
- [Commits](https://github.com/DataDog/dd-trace-rb/compare/v0.36.0...v0.37.0 )
Signed-off-by: dependabot-preview[bot] <support@dependabot.com >
2020-06-26 19:16:32 +00:00
Transifex-Openfoodnetwork
4f2d32b262
Updating translations for config/locales/tr.yml
2020-06-27 04:22:14 +10:00
Luis Ramos
5e3d7be545
Merge pull request #5630 from openfoodfoundation/dependabot/bundler/ransack-1.8.10
...
Bump ransack from 1.2.3 to 1.8.10
2020-06-26 13:37:03 +01:00
Transifex-Openfoodnetwork
0d5ad6fac3
Updating translations for config/locales/en_IE.yml
2020-06-26 21:04:54 +10:00
Transifex-Openfoodnetwork
a1227d93ae
Updating translations for config/locales/en_GB.yml
2020-06-26 20:27:34 +10:00
Transifex-Openfoodnetwork
f47271ba98
Updating translations for config/locales/fr.yml
2020-06-26 16:59:37 +10:00
Transifex-Openfoodnetwork
1266bcde1b
Updating translations for config/locales/en_FR.yml
2020-06-26 16:56:19 +10:00
Maikel Linke
e567348a69
Ensure that enough time passes for caching spec
2020-06-26 14:24:01 +10:00
Maikel Linke
4b25c2d49a
Correct typo in file naming
2020-06-26 14:21:21 +10:00
Maikel Linke
d7c9bb2e5a
Navigate more efficiently in spec
...
The spec is not supposed to test the navigation to the shop. Going
directly to the shop reduces the test execution time by 7%.
2020-06-26 09:55:57 +10:00
Maikel Linke
70347148a1
Stabilise shopping specs and open them for change
...
The way we add items to the cart will change. Encapsulating that code in
a common place will make the mobile ux work clearer and avoid merge
conflicts.
The waiting for background requests has also been improved and made more
consistent which should make these specs more reliable.
2020-06-26 09:55:57 +10:00
Maikel Linke
a0c7dc2ccb
Remove leftover empty spec context
2020-06-26 09:55:57 +10:00
Luis Ramos
5929b9c1e5
Merge pull request #5627 from openfoodfoundation/dependabot/bundler/geocoder-1.6.3
...
[Security] Bump geocoder from 1.5.2 to 1.6.3
2020-06-25 20:34:19 +01:00
Luis Ramos
2ad2836ac6
Merge pull request #5563 from luchiago/fix-missing-closed-oc-info-i18n
...
add missing i18n key for closed OC
2020-06-25 20:33:55 +01:00
Luis Ramos
93a2e57bc6
Merge pull request #5538 from jeduardo824/bug/4592-payment-methods-table-in-dashboard
...
Fix Payment Methods table to show proper provider name
2020-06-25 20:32:04 +01:00
Luis Ramos
cb2ccf76d2
Merge pull request #5590 from rmklaus12/5558-shippingcategory-translation
...
Added translation key to config/locales/en.yml file for shipping_cat…
2020-06-25 20:31:48 +01:00
Luis Ramos
f44533bbd5
Merge pull request #5589 from rmklaus12/2617-mainmenu-translation
...
Added more key to en.yml to translate More dropdown text
2020-06-25 20:27:46 +01:00
Luis Ramos
bfc134c9e6
Merge pull request #5615 from openfoodfoundation/dependabot/bundler/acts-as-taggable-on-4.0.0
...
Bump acts-as-taggable-on from 3.5.0 to 4.0.0
2020-06-25 20:26:59 +01:00
Pau Pérez Fabregat
7338f196e9
Merge pull request #5663 from luisramos0/constraints
...
Make db timestamps nullable
2020-06-25 13:14:41 +02:00
Luis Ramos
68946a765e
Make timestamps nullable
2020-06-25 08:47:18 +01:00
Transifex-Openfoodnetwork
0b48a53c2f
Updating translations for config/locales/tr.yml
2020-06-25 16:02:01 +10:00
Transifex-Openfoodnetwork
54ce5d8c0f
Updating translations for config/locales/en_CA.yml
2020-06-25 09:15:11 +10:00
Luis Ramos
ba50491c6d
Restructure the spec a little
2020-06-24 16:16:58 +01:00
Luis Ramos
34207fc20f
Bring changes to stock_item from spree 2.1, the previous version was from spree 2.0.4
2020-06-24 16:16:58 +01:00
Luis Ramos
e12e50aa84
Move rubocop exception to rubocop todo
2020-06-24 16:16:58 +01:00
Kristina Lim
20fd3c2642
Reset negative count on hand in existing non backorderable stock items
2020-06-24 16:16:58 +01:00
Kristina Lim
4694f1b21a
Require count on hand in non backorderable StockItem to be positive or zero
...
Fix setting of count on hand in line item specs
2020-06-24 16:16:45 +01:00
Kristina Lim
e53913756c
Add lock_version to Spree::StockItem
2020-06-24 16:15:37 +01:00
Kristina Lim
774b3720d5
Update stock item count on hand in Spree core specs
2020-06-24 16:15:09 +01:00
Kristina Lim
13ecf0ec73
Update specs for StockItem with transpec
2020-06-24 16:15:09 +01:00
Kristina Lim
fb20f220c0
Use break instead of return in StockItem#process_backorders
...
We are not using the return value of this method anywhere.
2020-06-24 16:15:09 +01:00
Kristina Lim
0a1cb71ee4
Ignore Rails/UniqueValidationWithoutIndex for unique index of StockItem#stock_location
2020-06-24 16:15:09 +01:00
Kristina Lim
bc530b92b5
Address violation of Rubocop Rails/Validation:
2020-06-24 16:15:09 +01:00
Kristina Lim
2acf61fd0f
Address violation of Rubocop Rails/Delegate
2020-06-24 16:15:09 +01:00
Kristina Lim
1e8543dfe7
Address violation of Rubocop Rails/ReadWriteAttribute
2020-06-24 16:15:09 +01:00
Kristina Lim
22c0693beb
Address violation of Rubocop Style/NumericPredicate
2020-06-24 16:15:09 +01:00
Kristina Lim
d1725014c4
Auto-correct violationso of Rubocop Layout/*
2020-06-24 16:15:09 +01:00
Kristina Lim
0fd66f9a55
Auto-correct violationso of Rubocop Style/*
2020-06-24 16:15:09 +01:00
Kristina Lim
b783118700
Auto-correct violationso of Rubocop Style/RedundantSelf
2020-06-24 16:15:09 +01:00
Kristina Lim
84d973d383
Specify RSpec.describe in StockItem spec file
2020-06-24 16:15:09 +01:00
Kristina Lim
0e711832fd
Bring Spree::StockItem code from spree_core into the app
2020-06-24 16:15:09 +01:00
Transifex-Openfoodnetwork
8d4a9cba21
Updating translations for config/locales/nb.yml
2020-06-24 23:33:12 +10:00
Steve Roberts
3d2a0d4d67
Remove UI and API engine as we will not host the swagger spec right now
2020-06-24 15:51:08 +10:00
Steve Roberts
9541dbf495
Fix order api spec and add incomplete generated swagger.yml
2020-06-24 15:15:18 +10:00
Steve Roberts
b59dbebba6
Revert Gemfile change to specific version of rubocop
...
Turns out it wasn't necessary. I thought it was needed to install a compatible version of rswag
2020-06-24 13:21:49 +10:00
Steve Roberts
e619ae621c
Add rswag and coverage for orders endpoint
...
Add coverage for core orders endpoint queries (WIP)
2020-06-24 13:21:49 +10:00
Steve Roberts
9e19d79337
Add more detail on the ransack based queries in the description
2020-06-24 13:21:49 +10:00
Steve Roberts
8b79d41b81
Update orders swagger spec and fix api_key access to orders endpoint
2020-06-24 13:21:49 +10:00
Luis Ramos
9e39d970ca
Result of running db:migrate
2020-06-23 19:19:18 +01:00
Pau Pérez Fabregat
9e730e6445
Merge pull request #5660 from coopdevs/fix-spree-preferences
...
Migrate all spree pref. keys to the new format
2020-06-23 18:00:01 +02:00
Transifex-Openfoodnetwork
99096eee2d
Updating translations for config/locales/ca.yml
2020-06-24 01:41:21 +10:00
Pau Perez
a81e17f23c
Flush the cache
...
So all preferences are fetch from DB now that they have changed.
2020-06-23 17:23:22 +02:00
Pau Perez
190797717a
Defend from the running the migration twice
...
We only migrate preferences if there are any to migrate.
2020-06-23 17:22:11 +02:00
Pau Perez
0ffa7b3734
Migrate all spree pref. keys to the new format
...
We realized in Spree v2.1 they follow this format instead and this is
what's causing issues to Katuma production.
This will remove the duplicate ones and convert the current preferences
to the new thus, keeping the values.
2020-06-23 16:21:20 +02:00
Transifex-Openfoodnetwork
923b905a9c
Updating translations for config/locales/en_GB.yml
2020-06-23 22:30:53 +10:00
Pau Pérez Fabregat
973ea2ea2d
Merge pull request #5641 from openfoodfoundation/transifex
...
Transifex
2020-06-23 10:43:26 +02:00
Pau Pérez Fabregat
3939febc2f
Merge pull request #5653 from luisramos0/rubocop_refresher
...
Auto correct rubocop, refresh todo lists and fix some more rubocop issues
2020-06-23 10:25:59 +02:00
Luis Ramos
91fc0cd26f
Adapt specs to new Ar method update that replaces update_attributes
2020-06-22 17:39:20 +01:00
Luis Ramos
f004b3d573
Fix rubocop issue
2020-06-22 17:39:20 +01:00
Luis Ramos
b49eef5fa9
Fix rubocop issue
2020-06-22 17:39:20 +01:00
Luis Ramos
1c550a594f
Fix rubocop issue
2020-06-22 17:39:20 +01:00
Luis Ramos
0ba1642bd2
Fix rubocop issues
2020-06-22 17:39:20 +01:00
Luis Ramos
1e5975c6e2
Fix rubocop issues
2020-06-22 17:39:20 +01:00
Luis Ramos
aa14fd236c
Fix rubocop issue
2020-06-22 17:39:20 +01:00
Luis Ramos
ab59ab45f5
Use auto correct to fix rubocop issues
2020-06-22 17:39:20 +01:00
Luis Ramos
f54abbd376
Fix rubocop issues
2020-06-22 17:39:20 +01:00
Luis Ramos
730959f85f
Fix rubocop issue
2020-06-22 17:39:20 +01:00
Luis Ramos
8870dc4b06
Move rubocop exception to code
2020-06-22 17:39:20 +01:00
Luis Ramos
1b5f9f105a
Fix block alignment
2020-06-22 17:39:20 +01:00
Luis Ramos
ed5fc29c44
Fix hash alignment issue
2020-06-22 17:39:20 +01:00
Luis Ramos
ff68303ef6
Fix long lines
2020-06-22 17:39:20 +01:00
Luis Ramos
0f2e07cc2d
Fix long lines
2020-06-22 17:39:20 +01:00
Luis Ramos
0e3b5ad4c3
Revert rubocop auto correct because update! is a method in the adjustment model
2020-06-22 17:39:17 +01:00
Luis Ramos
41ff406e86
Fix rubocop issue by avoiding rails method name update_attributes
2020-06-22 16:51:46 +01:00
Luis Ramos
6a7ed6e0c9
Rebuild .rubocop_manual_todo.yml
2020-06-22 16:51:46 +01:00
Luis Ramos
4e9c21d525
Add new regenerated rubocop_todo
2020-06-22 16:51:46 +01:00
Luis Ramos
b879439d58
Use rubocop auto correct to fix Style/SafeNavigation issue
2020-06-22 16:51:46 +01:00
Luis Ramos
1b6e51c79e
Fix rubocop issue
2020-06-22 16:51:46 +01:00
Luis Ramos
f848a89a00
Rename update_attributes to update #rails4
2020-06-22 16:51:43 +01:00
Luis Ramos
8a61257547
Rename all before_filter to before_action, it's the same function with a new name
2020-06-22 13:11:57 +01:00
Luis Ramos
46a1c1cf26
Fix rubocop issue
2020-06-22 13:08:54 +01:00
Luis Ramos
adfe1f89b6
Revert previous rubocop auto correct fix that breaks the cart spec
2020-06-22 13:08:38 +01:00
Luis Ramos
473fee19eb
Fix rubocop issues
2020-06-22 13:03:18 +01:00
Luis Ramos
6eebb0e399
Use rubocop autocorrect to fix rubocop issue
2020-06-22 12:58:33 +01:00
Luis Ramos
e574e254db
Use rubocop autocorrect to fix rubocop issue
2020-06-22 12:57:12 +01:00
Luis Ramos
3b91dd9dea
Use rubocop auto-correct to fix more issues
2020-06-22 12:51:47 +01:00
Luis Ramos
5d710f1e24
Use ruboccop autocorrect to fix issues
2020-06-22 12:46:07 +01:00
Luis Ramos
fb92745db4
Fix rubocop issues
2020-06-22 12:41:31 +01:00
Luis Ramos
8a9dae0ee2
Run rubocop autocorrect
...
This is the result of bundle exec rubocop --auto-correct
2020-06-22 12:23:10 +01:00
Luis Ramos
15babd8bc4
Merge pull request #5651 from luisramos0/xhr
...
[Rails 4.1] Change for xhr rquest to avoid csrf during testing
2020-06-22 12:20:41 +01:00
Luis Ramos
57d925a7de
Add new rubocop rules and fix the single offence
2020-06-22 12:19:19 +01:00
Lucas Hiago
357c2c7b25
change for xhr to avoid csrf during testing
2020-06-21 20:22:33 +01:00
Transifex-Openfoodnetwork
eb55d00051
Updating translations for config/locales/en_US.yml
2020-06-20 03:34:37 +10:00
Transifex-Openfoodnetwork
c5d3156351
Updating translations for config/locales/fr.yml
2020-06-20 02:37:11 +10:00
Transifex-Openfoodnetwork
28a6f694de
Updating translations for config/locales/en_FR.yml
2020-06-20 02:34:30 +10:00
Pau Pérez Fabregat
dd9d9e3316
Merge pull request #5639 from luchiago/fix-seeds-on-setup
...
FIx setup seed
2020-06-19 16:49:16 +02:00
Pau Perez
f36e08a163
Rename *.css.scss to *.scss
...
This fixes the following deprecation warning
```
DEPRECATION WARNING: Extra .css in SCSS file is unnecessary. Rename /usr/src/app/app/assets/stylesheets/darkswarm/variables.css.scss to /usr/src/app/app/assets/stylesheets/darkswar
m/variables.scss. (called from _app_views_layouts_darkswarm_html_haml___3990351141569700945_46957256630860 at /usr/src/app/app/views/layouts/darkswarm.html.haml:18)
(...)
```
2020-06-19 16:29:08 +02:00
Matt-Yorkley
66de0a4a71
Merge pull request #5642 from Matt-Yorkley/fix-table-width
...
Ensure cart page table is full-width
2020-06-19 14:27:00 +02:00
Matt-Yorkley
cda5a042b3
Ensure cart page table is full-width
2020-06-19 12:56:47 +02:00
Transifex-Openfoodnetwork
fd6cbe064d
Updating translations for config/locales/tr.yml
2020-06-19 20:27:09 +10:00
Pau Pérez Fabregat
db80992326
Merge branch 'master' into drop-blockenspiel
2020-06-19 12:12:10 +02:00
Matt-Yorkley
444097c75f
Merge pull request #5559 from luisramos0/display_only_shop
...
Make display only shops work again
2020-06-19 11:12:55 +02:00
Matt-Yorkley
be4a4c48d9
Merge pull request #5480 from Matt-Yorkley/mobile-cart-restyle
...
[Mobile] Cart Dropdown
2020-06-19 09:25:04 +02:00
Matt-Yorkley
b189d06eb7
Update shopping CTA logic
...
Shows "Take me shopping!" button on /shops page if a shop is selected
2020-06-19 08:59:43 +02:00
Lucas Hiago
52da896b03
Fix name when try to search on seed
2020-06-18 21:06:04 -03:00
Eduardo
573f7b1087
make shop links on the producers page open in new tab
2020-06-18 19:55:06 -03:00
Matt-Yorkley
ff6ba8819f
Use image_tag helper with relative paths when loading static images in /app/assets/images folder
...
This seems to be needed for Rails 4
2020-06-18 22:22:31 +02:00
Matt-Yorkley
7c7ab322bb
Ensure cart-header has the same height when the cart is empty
...
Fixes a minor visual bug
2020-06-18 22:21:55 +02:00
Matt-Yorkley
5a480439ca
Remove test for items from previous orders looking in cart dropdown
...
This feature has been removed
2020-06-18 22:21:55 +02:00
Matt-Yorkley
a6edc1c973
Enable control over body scroll with BodyScroll service
2020-06-18 22:21:55 +02:00
Matt-Yorkley
ed086c2db5
Add character limit to product and variant names in cart sidebar
2020-06-18 22:21:55 +02:00
Matt-Yorkley
2cc02d9711
Make cart header stick to top of cart
2020-06-18 22:21:55 +02:00
Matt-Yorkley
886df0e87d
Fix cart page button style regression
2020-06-18 22:21:55 +02:00
Matt-Yorkley
076ecfdb43
Adjust scrollbar visibility in sidebar
2020-06-18 22:21:55 +02:00
Matt-Yorkley
7093e0f7a3
Use plural or singular description in cart, depending on number of items
2020-06-18 22:21:55 +02:00
Matt-Yorkley
850e1b0364
Reduce specificity in cart css rules
2020-06-18 22:21:55 +02:00
Matt-Yorkley
b58ccc02ee
Adjust shop.css.scss and remove indentation
2020-06-18 22:21:55 +02:00
Matt-Yorkley
234f0f94cf
Change images in serializer to use product.images over variant.images
2020-06-18 22:21:55 +02:00
Matt-Yorkley
9c6dd1b324
Add comment on z-index
2020-06-18 22:21:55 +02:00
Matt-Yorkley
1eb9d11e27
Update colour variables
2020-06-18 22:21:55 +02:00
Matt-Yorkley
17c2721ab6
Remove unused translation keys
2020-06-18 22:21:55 +02:00
Matt-Yorkley
656c23f407
Replace previous min-width breakpoint.
...
This was mistakenly changed to use a max-width breakpoint in a previous commit.
2020-06-18 22:21:55 +02:00
Matt-Yorkley
1d8386bee8
Disable two SCSS-lint rules
2020-06-18 22:21:55 +02:00
Matt-Yorkley
c0282b57b6
Fix some linting warnings
2020-06-18 22:21:55 +02:00
Matt-Yorkley
042b4fc395
Remove tests for viewing previous order items in cart dropdown
...
This behavior has been removed
2020-06-18 22:21:55 +02:00
Matt-Yorkley
bd97939062
Move homepage "register here" CTA to bottom of screen.
...
The CTA breaks the layout when the new cart sidebar is open. I can't see a nice way to keep it at the top without making a mess.
2020-06-18 22:21:55 +02:00
Matt-Yorkley
2bcadd52e8
Hide "Take me shopping" button on /shop and /shops pages
2020-06-18 22:21:55 +02:00
Matt-Yorkley
387a5ec950
Implement "Take me shopping" button
2020-06-18 22:21:55 +02:00
Matt-Yorkley
10b8470e29
Set a fixed width for cart sidebar on tablet up
2020-06-18 22:21:55 +02:00
Matt-Yorkley
4acb08e52b
Update hamburger menu icon
2020-06-18 22:21:55 +02:00
Matt-Yorkley
786348bc71
Remove colon from cart total text
2020-06-18 22:21:55 +02:00
Matt-Yorkley
8ac971651c
Fix flaky spec
2020-06-18 22:21:55 +02:00
Matt-Yorkley
459e53f43d
Adjust specs for new cart display
2020-06-18 22:21:55 +02:00
Matt-Yorkley
2f2ef28351
Use links instead of buttons and fix up styles
2020-06-18 22:21:55 +02:00
Matt-Yorkley
b74b9fbd1d
Shift sidebar down vertically (below navigation) and bring navigation up (z-index)
2020-06-18 22:21:55 +02:00
Matt-Yorkley
4a0f43a831
Ensure cart item count is visible in header cart button (for screens smaller than desktop)
2020-06-18 22:21:55 +02:00
Matt-Yorkley
439674ad9c
Make cart view fullwidth on mobile
2020-06-18 22:21:55 +02:00
Matt-Yorkley
8ca0119f5d
Add empty cart feedback
2020-06-18 22:21:55 +02:00
Matt-Yorkley
95686eda78
Update translations
2020-06-18 22:21:54 +02:00
Matt-Yorkley
7efccac7bb
Add close functionality to close button
2020-06-18 22:21:54 +02:00
Matt-Yorkley
4362b8b25b
Improve line spacing on cart product list
2020-06-18 22:21:54 +02:00
Matt-Yorkley
be8c0f3600
Remove old cart dropdown view and directive
2020-06-18 22:21:54 +02:00
Matt-Yorkley
33a2dd100e
Add new layout for cart sidebar
2020-06-18 22:21:54 +02:00
Matt-Yorkley
400fce8ef9
Separate cart page and cart dropdown styles
2020-06-18 22:21:54 +02:00
Matt-Yorkley
b4e5542e5f
Extract re-usable sidebar styes
2020-06-18 22:21:54 +02:00
Matt-Yorkley
a8b981e9cd
Add product thumbnais to serializer for cart dropdown images
2020-06-18 22:21:54 +02:00
Matt-Yorkley
b1ce7f2c15
Simplify product naming in cart
2020-06-18 22:21:54 +02:00
Matt-Yorkley
85abfd2056
Create Cart sidebar and controller
2020-06-18 22:21:54 +02:00
Matt-Yorkley
7290d68687
Update all locales with the latest Transifex translations
2020-06-18 20:46:41 +02:00
Fatih Orhan
27fb3f156c
Update sell.html.haml
2020-06-18 20:36:50 +03:00
Luis Ramos
208623bb0a
Merge pull request #5635 from Matt-Yorkley/map-icons
...
Ensure map icon URLs are correctly generated with fingerprints
2020-06-18 18:02:47 +01:00
Matt-Yorkley
130f639b61
Rename service and use #call
2020-06-18 17:14:50 +02:00
Pau Pérez Fabregat
63f8830eba
Merge pull request #5530 from cillian/remove-spinjs
...
Remove spinjs-rails dependency and use spinning-circles.svg instead.
2020-06-18 16:47:09 +02:00
Matt-Yorkley
5338e782f8
Extract serializer helper method to service so the method isn't globally available
2020-06-18 16:44:15 +02:00
Luis Ramos
66f955f00d
Merge pull request #5597 from luisramos0/bo_stripe_payments
...
[Spree 2.1] Make backoffice stripe payments work in rails 4
2020-06-18 14:50:27 +01:00
Matt-Yorkley
75e57e5c2c
Call #image_path on map cluster icon
2020-06-18 14:14:41 +02:00
Matt-Yorkley
9a39cbbbc5
Convert MapConfiguration service to .js.erb.coffee
2020-06-18 14:14:41 +02:00
Matt-Yorkley
551daaadea
Use #image_path correctly on map marker icons
2020-06-18 14:14:41 +02:00
Matt-Yorkley
2143122a30
Add helper method for correctly generating with fingerprints in Serializers
2020-06-18 14:14:39 +02:00
Pau Pérez Fabregat
e02191078d
Merge pull request #5595 from coopdevs/improve-payments-report-performance
...
Remove N+1 fetching payments in report
2020-06-18 12:32:40 +02:00
Pau Pérez Fabregat
762b3a4a59
Merge pull request #5618 from openfoodfoundation/dependabot/bundler/httparty-0.18.1
...
Bump httparty from 0.17.3 to 0.18.1
2020-06-18 12:30:18 +02:00
Pau Pérez Fabregat
98490efbb3
Merge pull request #5600 from openfoodfoundation/transifex
...
Transifex
2020-06-18 12:29:10 +02:00
Matt-Yorkley
7cc47d652c
Merge pull request #5624 from Matt-Yorkley/rails4-image-precompile
...
v3 image precompile
2020-06-18 12:01:23 +02:00
Luis Ramos
52dd7ce916
Improve comment about httparty
2020-06-18 10:12:37 +01:00
dependabot-preview[bot]
669330ee90
Bump httparty from 0.17.3 to 0.18.1
...
Bumps [httparty](https://github.com/jnunemaker/httparty ) from 0.17.3 to 0.18.1.
- [Release notes](https://github.com/jnunemaker/httparty/releases )
- [Changelog](https://github.com/jnunemaker/httparty/blob/master/Changelog.md )
- [Commits](https://github.com/jnunemaker/httparty/compare/v0.17.3...v0.18.1 )
Signed-off-by: dependabot-preview[bot] <support@dependabot.com >
2020-06-18 10:12:37 +01:00
Transifex-Openfoodnetwork
20c41ce69e
Updating translations for config/locales/it.yml
2020-06-18 19:11:23 +10:00
Luis Ramos
f6e3e01a10
Extract permitted payment_source attributes from checkout to use them in the backoffice payments controller
...
Add spec to verify payment source attributes are passed
2020-06-18 09:54:34 +01:00
Transifex-Openfoodnetwork
b5a67e65c5
Updating translations for config/locales/es.yml
2020-06-18 17:42:06 +10:00
Transifex-Openfoodnetwork
da7d36cd0e
Updating translations for config/locales/ca.yml
2020-06-18 17:41:48 +10:00
Pau Pérez Fabregat
8f885dd1c5
Merge pull request #5628 from Matt-Yorkley/flaky-caching-spec
...
Flaky caching spec
2020-06-18 09:37:45 +02:00
Transifex-Openfoodnetwork
fba5f96968
Updating translations for config/locales/nb.yml
2020-06-18 17:33:09 +10:00
Transifex-Openfoodnetwork
0c634c566c
Updating translations for config/locales/fr.yml
2020-06-18 17:08:17 +10:00
Transifex-Openfoodnetwork
a9ed081488
Updating translations for config/locales/en_FR.yml
2020-06-18 17:01:02 +10:00
dependabot-preview[bot]
9c643447c2
Bump ransack from 1.2.3 to 1.8.10
...
Bumps [ransack](https://github.com/activerecord-hackery/ransack ) from 1.2.3 to 1.8.10.
- [Release notes](https://github.com/activerecord-hackery/ransack/releases )
- [Changelog](https://github.com/activerecord-hackery/ransack/blob/master/CHANGELOG.md )
- [Commits](https://github.com/activerecord-hackery/ransack/compare/v1.2.3...v1.8.10 )
Signed-off-by: dependabot-preview[bot] <support@dependabot.com >
2020-06-17 19:15:45 +00:00
Luis Ramos
37cbcf4172
Merge pull request #5605 from luisramos0/spree_rev
...
Update spree revision and upgrade ransack to 1.2.3
2020-06-17 20:00:25 +01:00
Luis Ramos
06a37d8ff0
Merge pull request #5566 from rioug/5132-no-error-when-product-csv-malformed
...
Add handling of CSV::MalformedCSVError for product_importer
2020-06-17 19:58:35 +01:00
Luis Ramos
d0bb625c19
Merge pull request #5593 from coopdevs/fix-report-order-link
...
Fix link from report to order
2020-06-17 19:55:40 +01:00
Luis Ramos
5163317b10
Merge pull request #5623 from luisramos0/zones
...
Make Zones editable again
2020-06-17 19:41:20 +01:00
Luis Ramos
91489827a1
Merge pull request #5614 from openfoodfoundation/dependabot/bundler/coffee-rails-4.2.2
...
Bump coffee-rails from 4.0.1 to 4.2.2
2020-06-17 19:40:59 +01:00
Matt-Yorkley
64b5bde952
Wait for page to actually load after visit shops_path
2020-06-17 12:47:08 +02:00
Matt-Yorkley
86fe032cdb
Update calls to #update_attributes! (Rails 4 Rubocop violation)
2020-06-17 12:37:20 +02:00
Matt-Yorkley
640c9e6d2b
Re-enable flaky spec
2020-06-17 12:36:42 +02:00
dependabot-preview[bot]
d67fa6fa5a
[Security] Bump geocoder from 1.5.2 to 1.6.3
...
Bumps [geocoder](https://github.com/alexreisner/geocoder ) from 1.5.2 to 1.6.3. **This update includes a security fix.**
- [Release notes](https://github.com/alexreisner/geocoder/releases )
- [Changelog](https://github.com/alexreisner/geocoder/blob/master/CHANGELOG.md )
- [Commits](https://github.com/alexreisner/geocoder/compare/v.1.5.2...v1.6.3 )
Signed-off-by: dependabot-preview[bot] <support@dependabot.com >
2020-06-16 23:49:32 +00:00
Lucas Hiago
419b3ef26e
add missing i18n key for check hub ready for checkout
2020-06-16 19:49:43 -03:00
Lucas Hiago
e9e43c985f
add missing i18n key for closed OC
2020-06-16 19:48:24 -03:00
Luis Ramos
01e27dd68c
Merge pull request #5492 from pacodelaluna/include-negative-tax-adjustments-in-sales-report
...
Include adjustments with negative taxes on Sales Tax reports
2020-06-16 20:23:04 +01:00
Matt-Yorkley
add38bf550
Use #image_path helper correctly
...
Paths should be relative. For example an image under `/app/assets/images/home/banner.jpg` should be: `image_path("home/banner.jpg")`
2020-06-16 18:25:13 +02:00
Matt-Yorkley
c07476e22c
Use new Rails 4 #image-url SCSS helper
...
Paths should be relative. For example an image under `/app/assets/images/home/banner.jpg` should be: `image-url("home/banner.jpg")`
2020-06-16 18:25:12 +02:00
Matt-Yorkley
c370ab04d7
Ensure images in /assets/images** are precompiled and moved correctly to /public/assets folder
2020-06-16 17:01:58 +02:00
Pau Pérez Fabregat
6a31c56601
Merge pull request #5598 from luisramos0/checkout_error
...
Make checkout js error handling a bit more defensive
2020-06-16 16:18:49 +02:00
François Turbelin
6672cfdaf5
Readd price_adustments after bad rebase
2020-06-16 16:13:56 +02:00
Luis Ramos
2fcafdcf45
Add spec to cover editing zones
2020-06-16 15:13:54 +01:00
Pau Pérez Fabregat
3cfec157ed
Merge pull request #5621 from Matt-Yorkley/xit-caching-spec
...
Set flaky caching spec to pending
2020-06-16 16:11:34 +02:00
Luis Ramos
774b75b649
Make zones spec faster by going directly to zone list page withouth repeating the navigation through Configuration menu
2020-06-16 14:06:01 +01:00
Luis Ramos
3f7fb0ecc2
Add missing permitted attributes to zones
2020-06-16 13:56:39 +01:00
Pau Perez
ff4d7fbc45
Refactor with Arel to perform a single query
2020-06-16 14:01:27 +02:00
Pau Perez
00c9dd7ece
Remove N+1 fetching order and li adjustments
2020-06-16 14:01:27 +02:00
François Turbelin
b4952dcbfb
Cosmetics
2020-06-16 14:01:27 +02:00
François Turbelin
9abe41f6cb
Use OrderTaxAdjustmentsFetcher service
2020-06-16 14:01:27 +02:00
François Turbelin
c729f64fcf
Include adjustments without positive taxes
2020-06-16 14:01:27 +02:00
Matt-Yorkley
cf214251fa
Set flaky caching spec to pending
2020-06-16 12:59:58 +02:00
dependabot-preview[bot]
f8c66cae81
Bump acts-as-taggable-on from 3.5.0 to 4.0.0
...
Bumps [acts-as-taggable-on](https://github.com/mbleigh/acts-as-taggable-on ) from 3.5.0 to 4.0.0.
- [Release notes](https://github.com/mbleigh/acts-as-taggable-on/releases )
- [Changelog](https://github.com/mbleigh/acts-as-taggable-on/blob/master/CHANGELOG.md )
- [Commits](https://github.com/mbleigh/acts-as-taggable-on/compare/v3.5.0...v4.0.0 )
Signed-off-by: dependabot-preview[bot] <support@dependabot.com >
2020-06-16 02:30:03 +00:00
dependabot-preview[bot]
dfdf57746a
Bump coffee-rails from 4.0.1 to 4.2.2
...
Bumps [coffee-rails](https://github.com/rails/coffee-rails ) from 4.0.1 to 4.2.2.
- [Release notes](https://github.com/rails/coffee-rails/releases )
- [Changelog](https://github.com/rails/coffee-rails/blob/master/CHANGELOG.md )
- [Commits](https://github.com/rails/coffee-rails/compare/v4.0.1...v4.2.2 )
Signed-off-by: dependabot-preview[bot] <support@dependabot.com >
2020-06-16 02:29:08 +00:00
Luis Ramos
7db519ab62
Merge pull request #5570 from luisramos0/ship_methods_form
...
[Spree 2.1] Make Shipping methods form save calculator details
2020-06-15 15:05:50 +01:00
Luis Ramos
0c97278d69
Upgrade ransack to 1.2.3
2020-06-15 14:20:34 +01:00
Luis Ramos
2c794c148b
Update spree rev
2020-06-15 14:14:53 +01:00
Pau Pérez Fabregat
257b311e04
Merge pull request #5158 from openfoodfoundation/3-0-stable
...
Merge 3-0-stable to master
2020-06-15 13:35:58 +02:00
Pau Pérez Fabregat
62fe54d90d
Merge pull request #5604 from luisramos0/xit_vo_flaky_2
...
Comment out flaky variant overrides spec
2020-06-15 13:11:24 +02:00
Luis Ramos
d5bdf306a9
Comment out flaky variant overrides spec
2020-06-15 11:43:30 +01:00
Luis Ramos
538d886c83
Merge pull request #5573 from Matt-Yorkley/payment-report-500
...
Fix fatal error in reports helper for orders without payments
2020-06-15 11:27:17 +01:00
Luis Ramos
c113ee92e6
Merge pull request #5603 from luisramos0/xit_exchange_prods
...
Comment out Exchange Products Renderer spec
2020-06-15 11:13:50 +01:00
Luis Ramos
d82a1c3bf1
Merge pull request #5602 from coopdevs/fix-style-issues
...
Fix outstanding Rubocop violations
2020-06-15 11:13:21 +01:00
Luis Ramos
ea8c5d2711
Comment out flaky spec
2020-06-15 11:01:35 +01:00
Pau Pérez Fabregat
e2e3b16250
Merge pull request #5549 from luisramos0/vo_flaky
...
[Spree 2.1] Make Variant Overrides spec less flaky and... comment it anyway
2020-06-15 12:00:16 +02:00
Pau Perez
456f369b76
Fix outstanding Rubocop violations
2020-06-15 10:17:34 +02:00
Pau Perez
df887b5576
Wrap long line
...
pick e84e0aebe Fix fatal error in reports helper for orders without payments
pick e725f3331 Defend from an order w/o payments when building table
2020-06-15 09:25:30 +02:00
Transifex-Openfoodnetwork
40dd9a8fda
Updating translations for config/locales/en_NZ.yml
2020-06-14 08:11:27 +10:00
Eduardo
27c76cfae2
add view spec to replace feature spec
2020-06-13 14:24:17 -03:00
Cillian O'Ruanaidh
430e5ea9cc
Permission to view :bulk_coop report is now added in the #add_bulk_coop_abilities method.
2020-06-13 16:57:43 +01:00
Cillian O'Ruanaidh
0ef3a7e24f
Fix some more code climate violations for bulk coop specs which have been moved to the order management engine.
2020-06-13 16:23:11 +01:00
Cillian O'Ruanaidh
fdf0d92366
Move feature spec for bulk coop report to the order management engine.
2020-06-13 15:52:42 +01:00
Cillian O'Ruanaidh
ba126d5afc
Move the bulk coop controller specs to the order management engine.
...
Also deleting old :bulk_coop_report action from the Spree::Admin::ReportsController.
2020-06-13 15:42:26 +01:00
Cillian O'Ruanaidh
c1e57fcc12
Adjust CanCan ability to reflect that the bulk coop report now runs from the order management engine.
2020-06-13 15:41:09 +01:00
Cillian O'Ruanaidh
3abcb5b23c
Don't move the :parameter_not_allowed_error translation to a different scope yet.
...
It will make it harder to merge and deploy this PR if every translation locale file has to be updated too.
2020-06-13 15:37:44 +01:00
Cillian O'Ruanaidh
bcf79ef803
Fix code climate violations related to removing blockenspiel.
2020-06-13 13:27:44 +01:00
Cillian O'Ruanaidh
6dbb6f3f09
Add an underscore before unused method arguments rather than an empty splat operator so it's easier to understand what is being passed in.
2020-06-13 13:09:09 +01:00
Cillian O'Ruanaidh
293e8949a9
Remove unused AuthenticationWorkflow module from the BulkCoopReport specs.
2020-06-13 12:59:23 +01:00
Cillian O'Ruanaidh
bfc6daf076
Remove duplicate PackingReport tests from the BulkCoopReport spec.
...
These tests already exist in spec/lib/open_food_network/packing_report_spec.rb:72
2020-06-13 12:57:14 +01:00
Cillian O'Ruanaidh
ed82c7e3db
BulkCoopReport has been moved to the engines/order_management so move it's spec there too.
2020-06-13 12:51:29 +01:00
Cillian O'Ruanaidh
e9ff84a746
Add a partial for the AJAX progress spinner so it can be reused in the spree admin and bare admin layouts.
2020-06-13 12:26:17 +01:00
Luis Ramos
e6e4b6f95c
Add keypress events to text input fields on orders search page
2020-06-13 12:15:49 +01:00
Luis Ramos
1740879261
Revert directive and just use ng-kkeypress
2020-06-13 12:07:45 +01:00
Luis Ramos
f08b837115
Make enter submit the search form in products list
2020-06-13 11:51:00 +01:00
Eduardo
5ed5ed2873
fix payment methods table to show proper provider name
2020-06-12 18:26:54 -03:00
Luis Ramos
ed262a19a9
Make checkout js error handling a bit more defensive
2020-06-12 20:33:22 +01:00
Luis Ramos
02b351b9d5
Make display only shops work again, this will show the product list when the OC is open but ship and pay methods are not available
...
Add spec to cover display only shops, it's oficially a feature now :-)
2020-06-12 20:15:08 +01:00
Pau Perez
018772bbba
Remove N+1 fetching payments in report
2020-06-12 16:42:40 +02:00
Pau Perez
29246c15fe
Defend from order without billing address
...
This a data integrity issue that needs deeper investigation but while
this happens, our users can still render their reports.
2020-06-12 16:34:23 +02:00
Luis Ramos
6dcb2d4a3b
Add xit, spec is still too flaky on semaphore
...
It's not the most important spec, this scenario should never happen as
the user will not be able to access the page.
2020-06-12 15:33:34 +01:00
Luis Ramos
f56e0ba0c0
Mock current_spree_user to return a user without permissions so that the controller does not use an already loaded version of current_spree_user that still has enough permissions
2020-06-12 14:23:45 +01:00
Luis Ramos
cf5af68438
Make spec less flaky by making it wait for the Saving text to go away
2020-06-12 13:04:33 +01:00
Luis Ramos
72870530ce
Merge pull request #5594 from Matt-Yorkley/3-0-reports-subquery
...
[Spree 2.1] Fix "subquery has too many columns" in reports query
2020-06-12 12:42:04 +01:00
Matt-Yorkley
86db1b6d68
Fix "subquery has too many columns" in reports query
2020-06-12 12:55:50 +02:00
Pau Perez
6a7df0b165
Fix link from report to order
2020-06-12 12:26:56 +02:00
Pau Pérez Fabregat
034c3d9568
Merge pull request #5548 from luisramos0/3-0-stable-jun-2
...
[Spree 2.1] Merge master into 3-0-stable
2020-06-12 11:27:29 +02:00
Pau Pérez Fabregat
a91c81059f
Merge pull request #5588 from openfoodfoundation/transifex
...
Transifex
2020-06-12 11:00:10 +02:00
Luis Ramos
7245a80f94
Merge branch 'master' into 3-0-stable-jun-2
2020-06-12 09:22:47 +01:00
Robin Klaus
f8ebe11728
Added translation keyt to config/locales/en.yml file for shipping_category
2020-06-12 13:19:15 +10:00
Transifex-Openfoodnetwork
10f0958819
Updating translations for config/locales/fr_CA.yml
2020-06-12 12:19:08 +10:00
Transifex-Openfoodnetwork
fce74bd008
Updating translations for config/locales/es.yml
2020-06-12 12:19:00 +10:00
Transifex-Openfoodnetwork
9908692da9
Updating translations for config/locales/en_FR.yml
2020-06-12 12:16:07 +10:00
Transifex-Openfoodnetwork
ff0e3ed379
Updating translations for config/locales/fr.yml
2020-06-12 12:16:00 +10:00
Transifex-Openfoodnetwork
2914500d5b
Updating translations for config/locales/ca.yml
2020-06-12 12:15:53 +10:00
Robin Klaus
b8f228cc3d
Added more key to en.yml to translate More dropdown text
2020-06-12 11:34:22 +10:00
Luis Ramos
898d525201
Merge pull request #5552 from coopdevs/remove-forgotten-photos
...
Remove unused stock photo
2020-06-11 20:41:01 +01:00
Luis Ramos
e2265dd165
Fix issue in product_spec
2020-06-11 20:26:57 +01:00
Luis Ramos
983bed3c4e
Merge branch 'master' into 3-0-stable-jun-2
2020-06-11 19:50:56 +01:00
Transifex-Openfoodnetwork
17bf163378
Updating translations for config/locales/fr_CA.yml
2020-06-12 03:53:33 +10:00
Transifex-Openfoodnetwork
e371c5f788
Updating translations for config/locales/en_CA.yml
2020-06-12 03:48:39 +10:00
Pau Pérez Fabregat
5cca8693b3
Merge pull request #5540 from romale/patch-1
...
Missing translation key "return_authorizations"
2020-06-11 16:39:15 +02:00
Transifex-Openfoodnetwork
bfcf5f8ca0
Updating translations for config/locales/ca.yml
2020-06-12 00:38:46 +10:00
Transifex-Openfoodnetwork
ab76319861
Updating translations for config/locales/en_FR.yml
2020-06-12 00:35:51 +10:00
Transifex-Openfoodnetwork
30917c64f5
Updating translations for config/locales/fr.yml
2020-06-12 00:35:44 +10:00
Transifex-Openfoodnetwork
6d7c1b7933
Updating translations for config/locales/es.yml
2020-06-12 00:35:36 +10:00
Pau Perez
9d5c657ff9
Update all locales with the latest Transifex translations
2020-06-11 15:49:05 +02:00
Pau Pérez Fabregat
e0bcc71882
Merge pull request #5569 from rmklaus12/5204-inventory-translation
...
Add translation keys to en.yml file for variant_override error messag…
2020-06-11 15:35:30 +02:00
Pau Pérez Fabregat
06feae994d
Merge pull request #5353 from pacodelaluna/sort-invoices-in-bulk-invoice-with-forced-order
...
Sort invoices in bulk invoice with completed_at desc order
2020-06-11 15:34:39 +02:00
Pau Pérez Fabregat
2052756169
Merge pull request #5556 from Matt-Yorkley/geocoding-service
...
Rename Angular Geo service
2020-06-11 15:19:08 +02:00
Pau Pérez Fabregat
bc8a10c6a0
Merge pull request #5541 from luisramos0/has_variants
...
Delete dead code related to products without variants
2020-06-11 15:18:13 +02:00
Pau Pérez Fabregat
01a9b33fc9
Merge pull request #5482 from rmklaus12/5094-new-order-translation
...
created partial for order page title to resolve French translation of…
2020-06-11 13:23:28 +02:00
Pau Pérez Fabregat
ab743c3af9
Merge pull request #5407 from luisramos0/oc_sweep
...
Order Cycle design sweep
2020-06-11 12:49:04 +02:00
Pau Pérez Fabregat
246a4995cb
Merge pull request #5490 from luchiago/change-price-sack-amount-int-to-float
...
Change minimal amount in price sack calculator to float
2020-06-11 12:42:11 +02:00
Matt-Yorkley
e725f33318
Defend from an order w/o payments when building table
2020-06-11 12:37:51 +02:00
Matt-Yorkley
e84e0aebe6
Fix fatal error in reports helper for orders without payments
2020-06-11 11:22:07 +02:00
Pau Pérez Fabregat
75d24c19fc
Merge pull request #5206 from chrishil1/clear_filters
...
Clear filters on admin/products page
2020-06-11 10:37:17 +02:00
Pau Pérez Fabregat
bdc4340034
Merge pull request #5582 from openfoodfoundation/transifex
...
Transifex
2020-06-11 10:30:13 +02:00
Pau Pérez Fabregat
961be38734
Merge pull request #5581 from openfoodfoundation/dependabot/bundler/test-unit-3.3.6
...
Bump test-unit from 3.3.5 to 3.3.6
2020-06-11 09:16:57 +02:00
Transifex-Openfoodnetwork
e76bc07d72
Updating translations for config/locales/es.yml
2020-06-11 17:11:55 +10:00
Transifex-Openfoodnetwork
6ef94febe5
Updating translations for config/locales/ca.yml
2020-06-11 17:10:49 +10:00
Gaetan Riou
359c0b9d85
fix typo
2020-06-11 16:54:00 +10:00
Gaetan Riou
d3943bc92a
refactor error handling of CSV::MalformedCSVError and fix some typos
2020-06-11 16:47:58 +10:00
Transifex-Openfoodnetwork
472338b522
Updating translations for config/locales/fr.yml
2020-06-11 16:32:02 +10:00
Transifex-Openfoodnetwork
220c84075f
Updating translations for config/locales/en_FR.yml
2020-06-11 16:25:36 +10:00
Luis Ramos
ccf6cbb003
Merge pull request #5483 from rmklaus12/5281-missing-translation-ordercreation
...
Changed translation syntax to normal t() call and lazy lookup.
2020-06-10 23:00:44 +01:00
Luis Ramos
13a4a3edbe
Merge pull request #5532 from luchiago/fix-admin-config-separator-withou-highlight
...
Fix admin config separator without highlight
2020-06-10 23:00:30 +01:00
Luis Ramos
21746e72ce
Merge pull request #5484 from rmklaus12/5336-errormsg-translation
...
Added translation keys to error messages partial and en.yml file to a…
2020-06-10 23:00:07 +01:00
Luis Ramos
1e248c1848
Merge pull request #5354 from jeduardo824/enhancement/5277-be-able-to-filter-orders-according-to-shipping-method
...
Enhancement/5277 be able to filter orders according to shipping method
2020-06-10 20:50:06 +01:00
Luis Ramos
28f126eed1
Merge pull request #5545 from mbudm/task/matomo-tag-manager
...
Add configuration to enable Matomo Tag Manager
2020-06-10 20:49:34 +01:00
dependabot-preview[bot]
1a700f870c
Bump test-unit from 3.3.5 to 3.3.6
...
Bumps [test-unit](https://github.com/test-unit/test-unit ) from 3.3.5 to 3.3.6.
- [Release notes](https://github.com/test-unit/test-unit/releases )
- [Commits](https://github.com/test-unit/test-unit/compare/3.3.5...3.3.6 )
Signed-off-by: dependabot-preview[bot] <support@dependabot.com >
2020-06-10 19:16:40 +00:00
Luis Ramos
113533e3b1
Merge pull request #5557 from Matt-Yorkley/reports-improvement
...
Improve performance in various reports
2020-06-10 18:54:51 +01:00
Pau Pérez Fabregat
a547aae770
Merge pull request #5562 from openfoodfoundation/transifex
...
Transifex
2020-06-10 17:57:07 +02:00
Transifex-Openfoodnetwork
1abbd7261e
Updating translations for config/locales/es.yml
2020-06-11 01:13:33 +10:00
Transifex-Openfoodnetwork
9a2618cd39
Updating translations for config/locales/ca.yml
2020-06-11 01:09:41 +10:00
Steve Roberts
f8673fb575
Moved tag manager instructions to be directly below the field
2020-06-10 15:06:53 +10:00
Maikel Linke
7d53b12baf
Preserve sorting by completion date
2020-06-10 10:19:15 +10:00
Maikel Linke
680ee3dc86
Check the bulk invoice rendering order
...
As it turns out, our performance optimisation with `find_each` overrides
or custom sorting.
2020-06-10 10:05:47 +10:00
Luis Ramos
89003aece7
Merge pull request #5414 from openfoodfoundation/dependabot/bundler/stripe-5.22.0
...
Bump stripe from 5.15.0 to 5.22.0
2020-06-09 17:41:20 +01:00
Pau Pérez Fabregat
86d59c4e86
Merge pull request #5564 from dmarcoux/improve-dockerfile
...
Improve Dockerfile
2020-06-09 16:17:37 +02:00
Luis Ramos
0c1ed1731d
Add more specs to ship method controller
2020-06-09 12:33:55 +01:00
Luis Ramos
87292a8ce5
Refactor spec to add more similar specs
2020-06-09 12:15:52 +01:00
Luis Ramos
f05b0d1a57
Add missing fields to ship method controller permitted params list
2020-06-09 12:02:59 +01:00
Luis Ramos
fd5062429a
Add context to delete specs so we can add other specs
2020-06-09 12:02:59 +01:00
Luis Ramos
7924330321
Fix problem with stub_const, we need auto load the controller before the const is stubbed otherwise the controller will be broken
...
See details here: https://github.com/rspec/rspec-mocks/issues/1079#issuecomment-215620243
2020-06-09 10:52:53 +01:00
Robin Klaus
c4fbce30c8
Add translation keys to en.yml file for variant_override error message translation
2020-06-09 19:20:29 +10:00
François Turbelin
795106aaa2
Use real orders inside spec to extend coverage
2020-06-09 10:11:11 +02:00
François Turbelin
6056c1699f
Use batches in order to fetch orders
2020-06-09 10:10:34 +02:00
Steve Roberts
5fcc0dcd14
Add configuration to enable matomo tag manager
...
- Update embed codes and allow both scripts concurrently
- Add lines around blocks
- Update MTM copy based on PR feedback
- checked cookies unchanged
2020-06-09 10:21:11 +10:00
Luis Ramos
88896b7730
Merge branch 'master' into 3-0-stable-jun-2
2020-06-08 19:17:11 +01:00
jeffrey s hill md
c4e910a7c8
Make clear filters fetch products automatically without filters
...
The spec change is needed: fetchProducts() makes cleared filters values undefined in removeClearedValues.
2020-06-08 18:27:10 +01:00
Transifex-Openfoodnetwork
dceae2a98b
Updating translations for config/locales/it.yml
2020-06-08 23:23:41 +10:00
Transifex-Openfoodnetwork
7515b9c86c
Updating translations for config/locales/it.yml
2020-06-08 23:20:30 +10:00
Gaetan Riou
9b8a97aadd
Add handling of CSV::MalformedCSVError for product_importer
2020-06-08 16:21:02 +10:00
Dany Marcoux
a751d24562
Run bundler install in parallel with the amount of available CPUs
2020-06-07 23:32:50 +02:00
Dany Marcoux
6c564dc7b2
Shallow clone specific release for rbenv and ruby-build
...
We don't need the whole git history, so doing a shallow clone is
favourable. Cloning a specific release (the latest at the time of
writing this) allows us to have reproducible results since we're not
cloning whatever is on master at the time of building the Docker image.
2020-06-07 23:29:17 +02:00
Transifex-Openfoodnetwork
e8f59ca80f
Updating translations for config/locales/tr.yml
2020-06-06 20:02:55 +10:00
Cillian O'Ruanaidh
f90969134d
Fix a couple of Rubocop violations.
2020-06-05 20:03:17 +01:00
Pau Perez
a287f370b5
Remove unused home photos
2020-06-05 18:05:58 +02:00
Cillian O'Ruanaidh
bafa8beab3
Use :parameter_not_allowed_error_message to from base Reports::Authorizer.
2020-06-05 17:05:26 +01:00
Cillian O'Ruanaidh
e58852289f
Move the :require_valid_datetime_range method to OrderManagement::Reports::Parameters::Base so it can be reused on multiple reports.
2020-06-05 16:58:56 +01:00
Cillian O'Ruanaidh
9172606780
Allow the order management report partial to be reused on different reports.
2020-06-05 16:51:43 +01:00
Cillian O'Ruanaidh
406309c577
Move BulkCoop reports out of deprecated lib/ directory into OrderManagement engine.
...
The BulkCoop reports are not generated the same way as the EnterpriseFeeSummary report is generated yet so that may need to be updated.
2020-06-05 16:43:38 +01:00
Pau Pérez Fabregat
ca458018ca
Merge pull request #5547 from luisramos0/ctrl_hacks
...
Bring the spree_get spec helpers from spree and move them together with the api_get spec helpers
2020-06-05 17:34:16 +02:00
Matt-Yorkley
9e278d5b2f
Improve performance in various reports
2020-06-05 16:39:01 +02:00
Luis Ramos
e2e3ae68b7
Update all locales with the latest Transifex translations
2020-06-05 11:36:27 +01:00
Luis Ramos
8ba15931e3
Merge pull request #5521 from openfoodfoundation/transifex
...
Transifex
2020-06-05 11:34:13 +01:00
Cillian O'Ruanaidh
d436d18d19
Fix code climate violations in removal of Blockenspiel dependency.
2020-06-05 10:13:06 +01:00
Transifex-Openfoodnetwork
c317ae7bf3
Updating translations for config/locales/en_FR.yml
2020-06-05 17:51:47 +10:00
Transifex-Openfoodnetwork
76aded5fb5
Updating translations for config/locales/fr.yml
2020-06-05 17:51:06 +10:00
Matt-Yorkley
78fc6b7138
Rename "Geo" service to "GmapsGeo" for clarity.
...
This service directly calls methods in the google maps javascript included in the darkswarm layout. If we're moving away from it, we should clarify where it's used.
2020-06-05 08:35:15 +02:00
Robin Klaus
d5d6fbc01d
created partial for order page title to resolve French translation of new order
2020-06-05 11:27:51 +10:00
Luis Ramos
82290bb5e1
Merge pull request #5448 from rmklaus12/5302-paypal-missingkey
...
Added translation keys for PayPalExpress payment method
2020-06-04 21:32:27 +01:00
Luis Ramos
3a20de1465
Merge pull request #5398 from cillian/open-street-map
...
Add support for using Open Street Map on the Map and Group pages.
2020-06-04 17:01:25 +01:00
Transifex-Openfoodnetwork
52323887b6
Updating translations for config/locales/en_FR.yml
2020-06-04 15:58:45 +10:00
Transifex-Openfoodnetwork
095d0f4ca8
Updating translations for config/locales/fr.yml
2020-06-04 15:58:22 +10:00
Luis Ramos
12000d2d44
Merge pull request #5425 from luisramos0/pay_method_warning
...
Add warning to payment method edit page
2020-06-03 21:03:56 +01:00
Luis Ramos
cce26cb453
Merge branch '3-0-stable' into 3-0-stable-jun-2
2020-06-03 19:51:43 +01:00
Pau Pérez Fabregat
93dd61c6fc
Merge pull request #5456 from luisramos0/3-0-stable-with-spree-deps
...
[Spree 2.1] Move all spree_core dependencies to OFN so we can upgrade them later
2020-06-03 18:57:24 +02:00
Luis Ramos
0b76a25941
Add spree_core dependencies to OFN so we can upgrade them
...
This will enable us to upgrade these dependencies independently
2020-06-03 14:25:27 +01:00
Lucas Hiago
bfe17f54d7
Change minimal amount in price sack calculator to float
2020-06-03 08:51:41 -03:00
Pau Perez
7900cf9438
Style spinner SVG by inlining it
...
This way we can tweak the `fill` property of the SVG and style it with
CSS as the documented in the author's repo
(https://github.com/SamHerbert/SVG-Loaders )
> An icon's color can be manipulated by changing the fill attribute in
the SVG file.
2020-06-03 11:13:03 +02:00
Transifex-Openfoodnetwork
bbf381d130
Updating translations for config/locales/es.yml
2020-06-03 18:32:57 +10:00
Pau Pérez Fabregat
da3d51b500
Merge pull request #5539 from luisramos0/dev_docs
...
Improve Dev onboarding docs
2020-06-03 09:50:03 +02:00
Transifex-Openfoodnetwork
61d85eda61
Updating translations for config/locales/ca.yml
2020-06-03 17:40:19 +10:00
Pau Pérez Fabregat
8a62c8aeb5
Merge pull request #5544 from luisramos0/api_ctrl
...
[Spree 2.1] Fix specs in Api::EnterprisesController and Api::ShopsController
2020-06-03 09:34:43 +02:00
Pau Pérez Fabregat
833e2ff9c4
Merge pull request #5543 from luisramos0/product_stock
...
[Spree 2.1] Fix product_stock spec
2020-06-03 09:18:52 +02:00
Pau Pérez Fabregat
4935932e45
Merge pull request #5531 from openfoodfoundation/dependabot/bundler/i18n-js-3.7.0
...
Bump i18n-js from 3.6.0 to 3.7.0
2020-06-03 09:17:58 +02:00
Pau Pérez Fabregat
d9cf22fed5
Merge pull request #5528 from luisramos0/checkout_js
...
Make checkout JS code send bugsnag alert even if there's an error processing an error
2020-06-03 09:13:40 +02:00
Pau Pérez Fabregat
fb339d6a99
Merge pull request #5263 from luisramos0/pdp
...
[Mobile] Restyle product modal
2020-06-03 08:55:20 +02:00
Transifex-Openfoodnetwork
4e497d93b7
Updating translations for config/locales/en_CA.yml
2020-06-03 10:31:14 +10:00
romale
325cbbf3c7
formatting
2020-06-02 23:40:57 +03:00
Luis Ramos
f9d86eb7ed
Join module and describe declaration so that rspec picks up correct controller under Api namespace and not the controller with the same name in the base namespace
2020-06-02 20:17:46 +01:00
Luis Ramos
251c04f2d9
Make all tests in enterprises_controller_spec use api_post instead of spree_post
2020-06-02 20:17:07 +01:00
Luis Ramos
ff0c93a76b
Join module and describe declaration so that rspec picks up correct
...
controller under Api namespace and not the controller with the same name
in the base namespace
2020-06-02 20:15:25 +01:00
Luis Ramos
c68459f212
Make enterprises_controller_spec use api_post instead of spree_post
2020-06-02 20:06:52 +01:00
Luis Ramos
5fd0a6028f
Merge pull request #5420 from luisramos0/pi_fix
...
[Spree 2.1] Product Import - Add variant_unit_name to the list of fields to be ignored when creating a variant
2020-06-02 18:13:48 +01:00
Luis Ramos
53a80de503
Improve method names and set use_route with reverse_merge so it can be overridden
2020-06-02 17:32:46 +01:00
Luis Ramos
4438f620e9
Merge pull request #5421 from luisramos0/more_strong_params
...
[Spree 2.1] Add missing strong params to less important controllers missed by specs
2020-06-02 16:59:27 +01:00
Luis Ramos
23b677f547
Merge pull request #5388 from luisramos0/reg_proc
...
[Spree 2.1] Fix enterprise package selection submit button
2020-06-02 16:23:52 +01:00
Luis Ramos
beb1b3f3bc
Replay 02d46b7c51 lost in the last merge from master
2020-06-02 16:11:03 +01:00
Luis Ramos
c9a7a6e897
Merge branch 'master' into 3-0-stable-jun-2
2020-06-02 16:00:45 +01:00
Luis Ramos
8bb4310cba
Merge pull request #5383 from luisramos0/admin_orders_bug
...
[Spree 2.1] Fix Update and Calculate Fees button
2020-06-02 15:30:16 +01:00
Luis Ramos
7b89f4d6c5
Switch controller helper calls routes from spree to main_app, this will include all routes all the time
2020-06-02 15:00:57 +01:00
Luis Ramos
13836a62bb
Rename parameters to params and change default from nil to {}
2020-06-02 14:36:06 +01:00
Luis Ramos
3136aa5a8b
Make api_process use process_spree_action
2020-06-02 14:00:24 +01:00
romale
9d56c180b2
formatting
2020-06-02 15:56:59 +03:00
Luis Ramos
2f76e0b15b
Bring methods from Spree::TestingSupport::ControllerRequests to our ControllerRequestsHelper so we can merge them later
2020-06-02 13:54:33 +01:00
Luis Ramos
5d0856e5a3
Rename ControllerHacks to ControllerRequestsHelper and move it's configuration to spec_helper
2020-06-02 13:47:59 +01:00
Luis Ramos
cd3bc54c37
Remove resource_scoping to make things more simple
2020-06-02 13:43:33 +01:00
Luis Ramos
2c2263ab78
Fix rubocop issues in shipments_controller_spec
2020-06-02 13:41:50 +01:00
Luis Ramos
092e047b44
Remove resource_scoping from api/shipments_controller spec
2020-06-02 13:35:48 +01:00
Luis Ramos
a0aa42cd58
Remove resource_scoping from api/variants spec
2020-06-02 13:26:12 +01:00
romale
d16bfe3779
formatting
2020-06-02 14:44:16 +03:00
romale
9d3f010173
formating
2020-06-02 14:42:50 +03:00
Luis Ramos
89c461662f
Merge pull request #5502 from Matt-Yorkley/dropshadow-regression
...
[mobile ux] Fix missing drop-shadow
2020-06-02 12:07:37 +01:00
romale
35648025df
formating
2020-06-02 12:09:38 +03:00
romale
b21904ada1
moved to 2882
2020-06-02 10:38:36 +03:00
Robin Klaus
f4a64e89c2
Added translation keys to error messages partial and en.yml file to add translation to error messages
2020-06-02 15:41:01 +10:00
Luis Ramos
2d7f1ce283
Delete all variants from test product not just the first one
2020-06-01 21:21:26 +01:00
Luis Ramos
072da7316e
Remove views code that tests for the existence of product variants
...
A product always has the standard variant, the only case where a product will not have variants is immediatly after creation and before being saved
2020-06-01 14:09:14 +01:00
Luis Ramos
e7828e107b
Delete dead methods from product_decorator
2020-06-01 14:07:34 +01:00
Luis Ramos
a4a6431faf
Fix strong params problem where order object is not sent to controller
2020-06-01 13:42:22 +01:00
Luis Ramos
48985bbcd1
Use patch instead of put, because it's rails 4
...
Extend the registration process spec to cover package selection
2020-06-01 13:41:37 +01:00
Luis Ramos
c6842ada7f
Add variant_unit_name to the list of fields to be ignored when creating a variant, it's a product field
2020-06-01 13:41:15 +01:00
Luis Ramos
754b657f81
Move controller methods to protected to keeep consitency and isolation
2020-06-01 13:40:59 +01:00
Luis Ramos
bae128738b
Add spec for return authorizations controller
2020-06-01 13:40:59 +01:00
Luis Ramos
583b4a1df7
Add spec to countries_controller
2020-06-01 13:40:59 +01:00
Luis Ramos
73f2844fe3
Add spec for shipping categories
2020-06-01 13:40:59 +01:00
Luis Ramos
e77d9aba7b
Add missing strong params to less important controllers not tested in specs
2020-06-01 13:40:59 +01:00
romale
069241f307
Missing translation key "return_authorizations"
...
return_authorizations: Return Authorizations
2020-06-01 15:25:46 +03:00
Transifex-Openfoodnetwork
e5cc425662
Updating translations for config/locales/tr.yml
2020-06-01 16:39:17 +10:00
Luis Ramos
5d9e88a3bc
Restructure GETTING STARTED to make it clear what the wiki pages are for
2020-05-31 23:00:32 +01:00
Luis Ramos
fb45cb602e
Remove duplication from CONTRIBUTING
2020-05-31 21:07:52 +01:00
Matt-Yorkley
8e57f9d929
Adjust shared map partial
2020-05-31 19:21:20 +02:00
Cillian O'Ruanaidh
94b85fb9e3
Fix up Code Climate issues with Open Street Map code.
2020-05-31 18:42:55 +02:00
Cillian O'Ruanaidh
deafe32354
Add support for using Open Street Map on the Map and Group pages.
...
The map is displayed using https://leafletjs.com/
To enable Open Street Map go to the Admin -> Configuration -> Content section and click 'Open Street Map Enabled'.
The 'Open Street Map Provider Name' setting can be used to configure different tile providers thanks to the Leaflet-providers extension (https://github.com/leaflet-extras/leaflet-providers )
Some tile providers require an API key, this can provided in JSON format e.g. '{ apiKey: 123 }' in the 'Open Street Map Provider Options' setting.
Each tile provider has their own usage policy so this should be checked before enabling Open Street Map.
The search field for the Open Street Map works differently than searching on Google Maps. It matches producers by their name or address because it was easier to implement instead of matching place names all over the world.
2020-05-31 18:42:55 +02:00
Matt-Yorkley
1e3bf7fec4
Merge pull request #5487 from coopdevs/fix-schedule-factory
...
[Spree 2.1] Allow to create order_cycle_schedules
2020-05-30 13:49:47 +02:00
Luis Ramos
c01da01c70
Merge pull request #5518 from openfoodfoundation/dependabot/bundler/ddtrace-0.36.0
...
Bump ddtrace from 0.35.2 to 0.36.0
2020-05-30 11:05:40 +01:00
Lucas Hiago
528db54f36
Correct controller name for matching
2020-05-29 22:24:41 -03:00
Lucas Hiago
19ef66f3ad
Change mail method config route
2020-05-29 21:51:57 -03:00
dependabot-preview[bot]
f773e57dee
Bump i18n-js from 3.6.0 to 3.7.0
...
Bumps [i18n-js](https://github.com/fnando/i18n-js ) from 3.6.0 to 3.7.0.
- [Release notes](https://github.com/fnando/i18n-js/releases )
- [Changelog](https://github.com/fnando/i18n-js/blob/master/CHANGELOG.md )
- [Commits](https://github.com/fnando/i18n-js/compare/v3.6.0...v3.7.0 )
Signed-off-by: dependabot-preview[bot] <support@dependabot.com >
2020-05-29 19:20:40 +00:00
Cillian O'Ruanaidh
e6fa296d03
Remove spinjs-rails dependency and use spinning-circles.svg instead.
...
For https://github.com/openfoodfoundation/openfoodnetwork/issues/3491
The spinjs-rails gem provides a CSS3 a spinning activity indicator. This commit removes the gem so we don't have to keep the gem up-to-date and uses the spinning-circles.svg which is used already in other places.
The spinjs spinner can be seen when viewing or editing taxonomies in /admin/taxonomies. It is loaded from the .ajaxStart call in app/assets/javascripts/admin/spree/progress.coffee
To test or restyle this spinner you can change the display property of the #progress element in app/assets/stylesheets/admin/components/progress.scss to 'block'.
2020-05-29 18:04:49 +01:00
Pau Perez
24e897d145
Do not build OC with schedules at the same time
...
It's not possible when neither the OC nor the schedule has an id due to
the new join table order_cycle_schedules.
2020-05-29 17:00:52 +02:00
Pau Perez
538c53fe9e
Improve code readability
...
Also removing an useless vars
2020-05-29 15:56:05 +02:00
Luis Ramos
6e842212ea
Always throw back error even if there is an error loading the flash for example translating the error message
2020-05-29 14:45:22 +01:00
Luis Ramos
3e0839de73
Increase warning sign line height so that it renders correctly on smaller screens
2020-05-29 12:56:22 +01:00
Luis Ramos
2c55bef544
Remove unnecessary padding
...
This is looking broken in ipad mini 3
2020-05-29 12:56:22 +01:00
Luis Ramos
a376a17f64
Group related properties together
2020-05-29 12:56:20 +01:00
Cillian O'Ruanaidh
d02f64da06
Define bulk coop report summary row and columns with lists of method names instead of Procs.
2020-05-29 11:34:12 +01:00
Matt-Yorkley
c73defa8a1
Merge pull request #5494 from Matt-Yorkley/group-maps-fix
...
Fix group maps
2020-05-29 12:05:55 +02:00
Pau Pérez Fabregat
c0ad992352
Merge pull request #5465 from luisramos0/rename_accessible_by
...
[Rails 4.1] Rename order_cycle.accessible_by to remove name clash with active record
2020-05-29 12:04:19 +02:00
Pau Pérez Fabregat
8d1944d714
Merge pull request #5506 from Matt-Yorkley/3-0-dfc-specs
...
[Spree 2.1] DFC specs updates
2020-05-29 11:43:19 +02:00
Pau Perez
815cd73ff3
DRY specs
2020-05-29 11:15:47 +02:00
Pau Perez
032dce1a88
Move OC save within success block
...
This makes more evident there are two scenarios: schedule save success
or failure. We deal with OCs failures raising an exception.
2020-05-29 10:46:29 +02:00
Pau Perez
768568c75c
Rename @object to @schedule
...
This level of abstraction makes sense in the framework-like code in
ResourceController but here it just makes things more difficult.
2020-05-29 10:44:43 +02:00
Pau Perez
4f3d01bd93
Remove unused HTML response format
2020-05-29 10:43:16 +02:00
Pau Perez
56b590a6f8
Extract #sync_subscription
...
This makes it far easier to spot what's the difference between create
and update regarding subs syncing without having to mess with `#update`
and its callbacks (for now).
2020-05-29 10:41:42 +02:00
Pau Perez
65c53df2ef
Remove unnecessary callback invocations
...
There are no registered callback methods to execute.
2020-05-29 10:25:38 +02:00
Pau Perez
f25ee8c998
Extract #editable_order_cycles
...
This makes it far easier to spot what's the difference between create
and update regarding the editable order cycles.
I stop here before a slightly deeper refactor which would make the PR
quite hard to review. This will come an upcoming one.
2020-05-29 10:25:37 +02:00
Pau Perez
862364ebbb
Fix schedule destroy spec
2020-05-29 10:24:50 +02:00
Pau Perez
4f635339e3
Fix schedule creation and subs syncing
2020-05-29 10:24:42 +02:00
Transifex-Openfoodnetwork
5e0e88bf2c
Updating translations for config/locales/fr.yml
2020-05-29 17:24:01 +10:00
Transifex-Openfoodnetwork
9fbd8b4fe1
Updating translations for config/locales/en_FR.yml
2020-05-29 16:46:33 +10:00
Maikel
2f82a58055
Merge pull request #5365 from mbudm/task/orderCycleSpecSplit
...
Split order_cycle_spec into smaller specs grouped by theme
2020-05-29 13:46:42 +10:00
Transifex-Openfoodnetwork
11f33bd911
Updating translations for config/locales/pt_BR.yml
2020-05-29 08:12:24 +10:00
Luis Ramos
2203bca1ce
Merge pull request #5489 from luisramos0/3-0-stable-may-22
...
[Spree 2.1] Merge master into 3-0-stable
2020-05-28 19:55:14 +01:00
Luis Ramos
bb8401a0fc
Merge pull request #5444 from luisramos0/dead_js_code
...
Remove some dead JS code related to admin sub menus
2020-05-28 19:54:42 +01:00
Luis Ramos
bd5de9d82f
Delete unused translations
2020-05-28 19:51:23 +01:00
Luis Ramos
bec798c6c9
Bring some spree translations to ofn in payment methods pages
2020-05-28 19:51:23 +01:00
Luis Ramos
fe6bcb5093
Add de-activation warning to payment methods edit page
2020-05-28 19:50:58 +01:00
Luis Ramos
26f1dfb080
Move payment_method display field translations to OFN and adapt them to changes on same field in shipping method
2020-05-28 19:50:58 +01:00
Luis Ramos
83edfbd1dd
Merge pull request #5266 from luchiago/improve-order-cancellation-email
...
Improve order cancellation email
2020-05-28 19:48:26 +01:00
Cillian O'Ruanaidh
1fba998b9e
Drop Blockenspiel and use :rules/:columns methods instead of :organise/:summary_columns DSL.
...
For https://github.com/openfoodfoundation/openfoodnetwork/issues/3231
2020-05-28 19:04:18 +01:00
Pau Perez
9a29e634fc
Deal with Schedule creation in its own controller
...
This way it can assign the order cycles to the schedule when this is
persisted. An OrderCycleSchedule (the join table) can't be created until
both schedule and order_cycle got an id.
Also, we do not call `#adapt_params` when creating the Schedule as that
assigns `order_cycle_ids` to `@object.attributes` thus, attempting to
create the OrderCycleSchedule without a schedule_id.
2020-05-28 17:49:36 +02:00
Transifex-Openfoodnetwork
29f266d3a5
Updating translations for config/locales/tr.yml
2020-05-28 23:17:00 +10:00
Transifex-Openfoodnetwork
c428b2f133
Updating translations for config/locales/tr.yml
2020-05-28 23:07:06 +10:00
Pau Perez
06b7a95fb1
DRY duplicate subscription declaration
2020-05-28 14:00:53 +02:00
Matt-Yorkley
ea7a2c91ee
Update all locales with the latest Transifex translations
2020-05-28 13:12:12 +02:00
Matt-Yorkley
2ae84ad710
Merge pull request #5491 from openfoodfoundation/transifex
...
Transifex
2020-05-28 13:08:36 +02:00
dependabot-preview[bot]
8632383e60
Bump ddtrace from 0.35.2 to 0.36.0
...
Bumps [ddtrace](https://github.com/DataDog/dd-trace-rb ) from 0.35.2 to 0.36.0.
- [Release notes](https://github.com/DataDog/dd-trace-rb/releases )
- [Changelog](https://github.com/DataDog/dd-trace-rb/blob/master/CHANGELOG.md )
- [Commits](https://github.com/DataDog/dd-trace-rb/compare/v0.35.2...v0.36.0 )
Signed-off-by: dependabot-preview[bot] <support@dependabot.com >
2020-05-27 20:23:11 +00:00
Luis Ramos
6f0106251e
Move product-img css class to parent div so we can center align it and keep existing img styling
2020-05-27 19:16:31 +01:00
Pau Pérez Fabregat
3b4266cbc8
Merge pull request #5495 from luisramos0/byebug
...
Fix error in dependency requirement specification for byebug and pry-byebug
2020-05-27 08:54:32 +02:00
Pau Pérez Fabregat
22be41ebed
Merge pull request #5498 from mbudm/task/local_db_port_config
...
Add support for an env var to override DB port
2020-05-27 08:53:29 +02:00
Pau Pérez Fabregat
127e48e579
Merge pull request #5413 from luisramos0/edit_orders_mobile
...
Show order edit link on mobile instead of the cancel link
2020-05-27 08:51:41 +02:00
Pau Pérez Fabregat
4d3ae8fa53
Merge pull request #5439 from coopdevs/fix-misplaced-paragraph
...
[mobile ux] Fix misplaced paragraph in shop tab
2020-05-27 08:51:13 +02:00
Luis Ramos
7313e3c096
Merge pull request #5501 from luisramos0/grumpy_cat_take_2
...
Enterprises Controller: reset_distributor must be called before any call to memoized current_distributor
2020-05-26 23:06:59 +01:00
Transifex-Openfoodnetwork
d2e46b8561
Updating translations for config/locales/ca.yml
2020-05-27 00:32:12 +10:00
Transifex-Openfoodnetwork
8d3c8c62fb
Updating translations for config/locales/en_CA.yml
2020-05-27 00:02:52 +10:00
Transifex-Openfoodnetwork
daf9fab16d
Updating translations for config/locales/en_CA.yml
2020-05-26 23:59:48 +10:00
Matt-Yorkley
89dd7ddb86
Update setting of request headers in DFC controller spec
2020-05-26 15:32:07 +02:00
Matt-Yorkley
bde4acd22f
Update get calls in DFC controller spec
2020-05-26 15:21:02 +02:00
Matt-Yorkley
953e1f8ff4
Adjust z-index on overlapping elements to fix missing drop-shadow regression
2020-05-26 12:59:51 +02:00
Luis Ramos
52810b0f4a
Make enterprises controller spec assert the corrrect state of current_distributor
...
This validates bug #5496 is fixed
2020-05-26 11:26:20 +01:00
Luis Ramos
4026e072d5
Break OrderCartReset in two steps so that memoized method current_customer (that uses memoized current_distributor) is called after reset_distributor
2020-05-26 11:13:16 +01:00
Luis Ramos
a582261d7a
Merge pull request #5493 from luisramos0/display_on_4_managers
...
Make display_on attribute available for managers to change in shipping and payment methods
2020-05-26 09:57:56 +01:00
Pau Pérez Fabregat
c3f66203bb
Merge pull request #5497 from openfoodfoundation/dependabot/bundler/rack-mini-profiler-2.0.2
...
Bump rack-mini-profiler from 2.0.1 to 2.0.2
2020-05-26 09:55:26 +02:00
Robin Klaus
50a63139e3
Changed translation syntax to normal t() call and lazy lookup.
2020-05-26 15:14:26 +10:00
Steve Roberts
d4a256fcca
Add support for an env var to override DB port
2020-05-26 11:04:42 +10:00
Steve Roberts
2bb3cf1c54
Name checkbox element id var more informatively
2020-05-26 09:53:51 +10:00
dependabot-preview[bot]
fc186f5342
Bump rack-mini-profiler from 2.0.1 to 2.0.2
...
Bumps [rack-mini-profiler](https://github.com/MiniProfiler/rack-mini-profiler ) from 2.0.1 to 2.0.2.
- [Release notes](https://github.com/MiniProfiler/rack-mini-profiler/releases )
- [Changelog](https://github.com/MiniProfiler/rack-mini-profiler/blob/master/CHANGELOG.md )
- [Commits](https://github.com/MiniProfiler/rack-mini-profiler/compare/v2.0.1...v2.0.2 )
Signed-off-by: dependabot-preview[bot] <support@dependabot.com >
2020-05-25 21:24:42 +00:00
Luis Ramos
513ee6393a
Make product modal display image on the right for medium screens like it was doing for large screens
2020-05-25 19:31:02 +01:00
Luis Ramos
1e0e3a063d
Add margin right to product modal title so that the close button doesnt overlap with it
2020-05-25 19:14:40 +01:00
Luis Ramos
50e4896e32
Fix error in dependency requirement specification
...
~> 11.0 allows upgrades to 11.1, we want to lock it to 11.0.x so we need ~> 11.0.0
Same logic for pry-byebug
2020-05-25 18:15:47 +01:00
Matt-Yorkley
8095371304
Ensure OpenStreetMap credits are aways shown with map and fix credits positioning for /groups page map display
2020-05-25 14:12:12 +02:00
Matt-Yorkley
a3660dfe37
Extract repeated map code to new partial
2020-05-25 14:04:11 +02:00
Matt-Yorkley
e03005ed56
Fix maps and tidy up groups map code
2020-05-25 13:52:56 +02:00
Luis Ramos
cb0aefd29c
Make display on attribute available for managers to change, not just super admins
2020-05-25 11:21:53 +01:00
Transifex-Openfoodnetwork
d9d221e4f3
Updating translations for config/locales/en_GB.yml
2020-05-25 18:46:48 +10:00
François Turbelin
b2b988b063
Apply new order directly inside service
2020-05-24 14:41:40 +02:00
Transifex-Openfoodnetwork
5de3fbba1e
Updating translations for config/locales/en_FR.yml
2020-05-24 19:14:49 +10:00
Transifex-Openfoodnetwork
ed27ed32e3
Updating translations for config/locales/fr.yml
2020-05-24 19:14:25 +10:00
Luis Ramos
f34f693620
Merge branch 'master' into 3-0-stable-may-22
2020-05-22 22:54:19 +01:00
Pau Pérez Fabregat
ff0e85c877
Merge pull request #5486 from Matt-Yorkley/3-0-caching
...
[Spree 2.1] Caching
2020-05-22 18:36:49 +02:00
Pau Perez
a33396984f
Fix and DRY specs
2020-05-22 18:17:59 +02:00
Pau Perez
d9686d6982
Enable spec files to be executed alone
...
This fixes the annoying error
```
NameError:
uninitialized constant OrderManagement::Subscriptions::Whatever
```
and let's you execute the spec file in isolation. It slows down way too
much having the run the entire engine test suite while developing. And
it makes me nervous too.
2020-05-22 15:55:03 +02:00
Matt-Yorkley
f0525b861c
Update expectations in API caching test
...
Rails 4 has slightly changed the format of the cache keys here, so the response was not as expected.
2020-05-22 14:54:57 +02:00
Pau Perez
e339a37cd5
Allow to create order_cycle_schedules
...
There's no way we can create an order_cycle_schedules if the schedule
doesn't have an id, which we can't get if to persist it we need an OC
first, which in turn, will create an order_cycle_schedules.
2020-05-22 14:22:31 +02:00
Matt-Yorkley
b00fbd69ae
Update Fragment Caching for Rails 4
...
Rails 4 introduced "automagically" modified cache keys, that included a digest in the key on any cache entry related to views. This is not what we want at all, fixed here with the `skip_digest: true` option.
2020-05-22 14:22:21 +02:00
Matt-Yorkley
a9abe48ede
Re-enable cache invalidation test
2020-05-22 14:21:45 +02:00
Matt-Yorkley
04ffe7b911
Fix expectations and mock responses for Rails 4
...
Mocking these objects is working differently in Rails 4, and utilising `#and_call_original` helped to ensure different parts of the codebase were receiving the correct response objects (AR relations, etc).
2020-05-22 14:21:45 +02:00
Pau Perez
be9fbae300
Fix misplaced paragraph in shop tab
...
By wrapping message within 12-cols container, keeping consistency with
the other shop tabs.
2020-05-22 12:37:07 +02:00
Robin Klaus
53c0bcbd0b
Changed translation syntax to normal t() call and lazy lookup.
2020-05-22 13:22:57 +10:00
Luis Ramos
12f512e98b
Move arrow on OC selector a little to the right so it's centered between end of text (if max length reached) and border
2020-05-21 15:41:26 +01:00
Luis Ramos
b2bfe28173
Remove extra right pad on OC selector, in mobile, this will make the selector be centered
2020-05-21 15:38:55 +01:00
Luis Ramos
ed14a92c7b
Make OC selector wider to support more characters where there is enough space for that
2020-05-21 15:38:54 +01:00
Pau Pérez Fabregat
16dfb3c7e9
Merge pull request #5408 from luisramos0/columns_params
...
[Spree 2.1] Add missing permitted param to Column Preferences controller
2020-05-21 14:41:30 +02:00
Pau Pérez Fabregat
050e83e745
Merge pull request #5410 from luisramos0/fix_some_specs
...
[Spree 2] Make api/enterprises_controller_spec less flaky
2020-05-21 13:45:39 +02:00
Robin Klaus
16f2c8d7d1
Removed 5 keys per change request in ref to pr5425
2020-05-20 14:13:42 +10:00
Luis Ramos
942dde62ff
Merge pull request #5464 from luisramos0/3-0-action-caching
...
[Spree 2.1] Add actionpack-action_caching gem to make action caching work again
2020-05-19 18:47:55 +01:00
Luis Ramos
758d7c82f7
Rename order_cycle.accessible_by to remove name clash with active record
2020-05-19 12:21:13 +01:00
Luis Ramos
2e9f911958
Add actionpack-action_caching to support action caching
2020-05-19 12:06:24 +01:00
Steve Roberts
446bf268b4
scroll_to substitute
2020-05-19 14:16:45 +10:00
Lucas Hiago
ba30f55e70
Add fullwidth for table selector
2020-05-18 22:53:46 -03:00
Steve Roberts
049b1b4638
Split up the specs again.
...
This now shows that there is one consistent spec erroring, spec/features/admin/order_cycles/complex_editing_multiple_product_pages_spec.rb:
Capybara::ElementNotFound:
Unable to find visible checkbox "order_cycle_incoming_exchange_0_select_all_variants" that is not disabled
2020-05-19 10:14:26 +10:00
Eduardo
6d54e6d4a0
change SearchOrders service to search with shipping_method
2020-05-18 20:44:55 -03:00
Eduardo
473b4c63c4
add Shipping Method filter to admin/orders page
2020-05-18 18:46:43 -03:00
Luis Ramos
d1eebe1cfb
Merge pull request #5426 from luisramos0/3-0-stable-may13
...
[Spree 2.1] Merge master into 3-0-stable
2020-05-18 15:05:11 +01:00
Robin Klaus
a0334953db
Added 14 translation keys for PayPalExpress payment method
2020-05-18 17:06:07 +10:00
Luis Ramos
6c81c19f00
Remove dead code
2020-05-16 14:19:25 +01:00
Lucas Hiago
98cbe5762a
Remove unused css property from image in cancel email
2020-05-15 10:59:56 -03:00
Lucas Hiago
0caa91d057
Remove second table of cancel email instructions style
2020-05-15 10:59:56 -03:00
Lucas Hiago
5d286cb349
Add i18n of new symbols for order cancel mail
2020-05-15 10:59:56 -03:00
Lucas Hiago
f72da4abe4
Add refund message in order cancel mail
2020-05-15 10:59:56 -03:00
Lucas Hiago
42012d6482
Add new order sumary in order cancel mail
2020-05-15 10:59:55 -03:00
Lucas Hiago
1c1200125e
Add new greeting table in order cancel mail
2020-05-15 10:59:55 -03:00
Lucas Hiago
0b8771ccf0
Add connect with us footer to order cancelation email
2020-05-15 10:59:55 -03:00
Luis Ramos
4292636e70
Remove close button in product modal
2020-05-15 12:16:19 +01:00
Luis Ramos
d95b2a97b0
Add missing padding above close button (both for the case with image or just with text) and also below product image
2020-05-15 12:12:07 +01:00
Luis Ramos
edaf419857
Use rem instead of px or em
2020-05-15 12:12:07 +01:00
Luis Ramos
38bc076c77
Fix some rubocop issues
2020-05-15 12:12:07 +01:00
Luis Ramos
30fe457cb5
Make product modal show close button only if there is a scroll bar
2020-05-15 12:12:07 +01:00
Luis Ramos
e239bb33f8
Improve the mobile cookies policy modal with a margin
2020-05-15 12:12:07 +01:00
Luis Ramos
3a512478d6
Improve the mobile registration modal by adding a margin
2020-05-15 12:12:07 +01:00
Luis Ramos
4b63893377
Improve mobile login modal with a margin
2020-05-15 12:12:07 +01:00
Luis Ramos
50fa4fc93d
Make modal close buttons a little darker with a white cross
2020-05-15 12:12:07 +01:00
Luis Ramos
cbfaaabb72
Adjust different paddings and margins to make mobile product modal contents similar to required design
2020-05-15 12:12:07 +01:00
Luis Ramos
0a378a897b
Add close button to mobile product modal, this can be reused in other modals later
2020-05-15 12:12:07 +01:00
Luis Ramos
35ac5e093e
Adapt modal size to leave some background visible so that user can click to disable modal
2020-05-15 12:12:07 +01:00
Luis Ramos
39e47998e1
Make the background opacity of modals in the FrontOffice a litle darker according to design
2020-05-15 12:12:07 +01:00
Luis Ramos
14478d5fc1
Style producer name and properties in product modal
2020-05-15 12:12:07 +01:00
Luis Ramos
59a13adff0
Remove product category from product modal
2020-05-15 12:11:32 +01:00
Luis Ramos
fee03c4101
Remove repeated/dead rules
2020-05-15 12:11:32 +01:00
Luis Ramos
9a115afd4f
Remove bars around product description and border around product image
2020-05-15 12:11:32 +01:00
dependabot-preview[bot]
3bc09cfcfb
Bump stripe from 5.15.0 to 5.22.0
...
Bumps [stripe](https://github.com/stripe/stripe-ruby ) from 5.15.0 to 5.22.0.
- [Release notes](https://github.com/stripe/stripe-ruby/releases )
- [Changelog](https://github.com/stripe/stripe-ruby/blob/master/CHANGELOG.md )
- [Commits](https://github.com/stripe/stripe-ruby/compare/v5.15.0...v5.22.0 )
Signed-off-by: dependabot-preview[bot] <support@dependabot.com >
2020-05-14 21:03:21 +00:00
Luis Ramos
19da0ba969
Merge branch 'master' into 3-0-stable-may13
2020-05-13 19:50:32 +01:00
Luis Ramos
1671184a3c
Merge pull request #5375 from luisramos0/sample_order
...
[Spree 2.1] OFN v3 comes with sample orders :-)
2020-05-13 12:04:06 +01:00
Luis Ramos
be998a9eb6
Reload shipment before shipping otherwise shipment stays as ready but with shipped_at defined
2020-05-13 12:00:37 +01:00
Luis Ramos
58842e8cab
Show order edit link on mobile instead of the cancel link
2020-05-12 19:44:01 +01:00
Luis Ramos
83381ad439
Relax array order verification
2020-05-12 15:46:23 +01:00
Luis Ramos
7804c740a6
Add name to permitted params and fix some rubocop issues
2020-05-12 15:35:47 +01:00
Luis Ramos
cb7a261594
Merge pull request #5374 from luisramos0/3-0-stable-may-5
...
[Spree 2.1] Merge master into 3-0-stable
2020-05-06 13:57:11 +01:00
Luis Ramos
825f747dff
Add more types of sample orders
2020-05-05 23:16:54 +01:00
Luis Ramos
4083da5b16
Add sample order to sample data
2020-05-05 23:16:54 +01:00
Luis Ramos
be4104d6f3
Merge branch 'master' into 3-0-stable-may-5
2020-05-05 19:11:20 +01:00
Steve Roberts
3b8acdc577
Split order_cycle_spec into smaller specs grouped by theme
2020-05-05 15:39:45 +10:00
Pau Pérez Fabregat
e04b18c70e
Merge pull request #5320 from luisramos0/vo_fix
...
[Spree 2.1] Fix variant overrides specs and shops spec
2020-04-29 17:30:41 +02:00
Luis Ramos
f3e651b1f7
present? calls length which is breaking it with error ActiveModel::MissingAttributeError Exception: missing attribute: address_id
...
any? works well as it just verifies the size of the collection
2020-04-28 16:31:11 +01:00
Luis Ramos
7e12142f91
Collection must be after load_data so that hubs are already available for VOs calculation
2020-04-28 16:31:11 +01:00
Luis Ramos
794fb5dfc5
Merge pull request #5316 from luisramos0/3-0-stable-Apr28
...
[Spree 2.1] Merge master into 3-0-stable
2020-04-28 16:30:29 +01:00
Luis Ramos
8c29797b23
Remove empty space on ffi line that was breaking bundle install
...
fuubar gets automatically moved to 2-5-0 as it is in master
2020-04-28 14:57:21 +01:00
Luis Ramos
839b373b5c
Merge branch 'master' into 3-0-stable-Apr28
2020-04-28 13:40:19 +01:00
Pau Pérez Fabregat
e1c2b7e105
Merge pull request #5159 from luisramos0/timezone_mixup
...
[Spree 2.1] Fix timezone mix up in a spec
2020-04-16 11:38:54 +02:00
Pau Pérez Fabregat
7b656fa0a4
Merge pull request #5194 from luisramos0/update_spree_rev
...
[Spree 2.1] Update spree revision to a version that doesnt depend on deface
2020-04-16 11:37:42 +02:00
Luis Ramos
7d82c6a014
Update spre revision to a version that doesnt depend on deface, this allows us to upgrade nokogiri (done in another PR in the master branch)
2020-04-09 13:51:54 +01:00
Luis Ramos
4b23adca2d
Merge pull request #5157 from luisramos0/email_validation
...
[Spree 2.1] Add order.email regexp validation
2020-04-09 10:14:11 +01:00
Luis Ramos
ff799b6e82
Avoid using update_attributes that save the order to the DB
2020-04-07 11:13:29 +01:00
Maikel
45c0209971
Merge pull request #5154 from luisramos0/prop_strong
...
[Spree 2.1] Add strong params implementation to properties controller
2020-04-07 16:27:22 +10:00
Luis Ramos
fa4b8832e9
Fix timezone mix up in specs
...
Converting times with timezones to string was ignoring the timezone and this making the start date be before the end date in this spec
2020-04-04 16:39:23 +01:00
Luis Ramos
4ceaebf098
Add order.email regexp validation and add some tests for it
2020-04-04 16:13:23 +01:00
Luis Ramos
451cfac788
Merge pull request #4995 from Matt-Yorkley/3-0-eager-load
...
[Spree 2.1] Staging and Production configs
2020-04-04 12:21:25 +01:00
Luis Ramos
4f08ed1801
Remove commented turbo-sprockets-rails3 from Gemfile, thsi gem is not needed in rails 4
2020-04-04 11:39:20 +01:00
Luis Ramos
1364a5e367
Merge pull request #4916 from luisramos0/adj_eli
...
[Spree 2.1] Make Spree::OrderUpdater update all adjustments, not just promotions and shipment adjustments
2020-04-04 11:28:13 +01:00
Luis Ramos
07264446e6
Add strong params implementation to properties controller
2020-04-04 11:17:38 +01:00
Luis Ramos
b8f8db0341
Merge pull request #5147 from luisramos0/3-0-stable-apr3
...
[Spree 2.1] Merge master into 3-0-stable
2020-04-04 11:03:14 +01:00
Luis Ramos
04986dd0d5
Merge pull request #4932 from luisramos0/line_items_fix
...
Adapt line items scope to include deleted variants and deleted products - rails 4 fix
2020-04-03 20:09:35 +01:00
Luis Ramos
384ca310a3
Merge branch 'master' into 3-0-stable-apr3
2020-04-03 18:26:26 +01:00
Luis Ramos
154e223a12
Make sure test data is available before the test
2020-04-01 10:00:07 +01:00
Luis Ramos
80b9800b36
Replace SQL with equivalent AR code
2020-04-01 10:00:07 +01:00
Luis Ramos
6e095bdb68
Fix bug in line_item_decorator, should be inner join not left outer join
2020-04-01 10:00:07 +01:00
Luis Ramos
a4c7dbc329
Add comment to better explain why this scope is not simply using joins(:product)
2020-04-01 10:00:07 +01:00
Luis Ramos
8909238682
Fix one rubocop issue
2020-04-01 10:00:07 +01:00
Luis Ramos
7c367da904
Adapt LineItem#supplied_by_any scope to include deleted variants and deleted products (both not included in the respective default scopes) and use it in Permissions::Order so that variants of deleted products are seen in reports
2020-04-01 10:00:07 +01:00
Luis Ramos
0c8635403a
Delete dead scope in line_items
...
This is not used anywhere in the code
2020-04-01 10:00:07 +01:00
Pau Pérez Fabregat
a218eab67b
Merge pull request #4987 from Matt-Yorkley/3-0-packing
...
[Spree 2.1] Packing report deleted variants
2020-03-26 17:37:51 +01:00
Pau Pérez Fabregat
1c2815d2b7
Merge pull request #5044 from luisramos0/add_spring
...
[Spree 2.1] Re-add spring and newrelic_rpm
2020-03-26 11:23:19 +01:00
Pau Pérez Fabregat
b3a47964e6
Merge pull request #5033 from luisramos0/ent_fees_data_types
...
[Spree 2.1] Adapt EnterpriseFeeSummary report code to slightly different rails 4 sql data conversion
2020-03-26 11:20:44 +01:00
Pau Pérez Fabregat
9daa40ce9c
Merge pull request #5028 from luisramos0/remove_attr_accessible
...
[Spree 2.1] Remove new attr_accessible entries
2020-03-26 11:20:08 +01:00
Pau Pérez Fabregat
f70e155b1b
Merge pull request #4984 from Matt-Yorkley/3-0-cart-route
...
[Spree 2.1] Fix cart page update
2020-03-26 11:14:07 +01:00
Matt-Yorkley
0cc5cbd1d2
Merge pull request #5038 from luisramos0/strong_params_prod
...
[Spree 2.1] Implement strong params in products, variants, variant_overrides and resource controllers
2020-03-25 18:35:58 +01:00
Luis Ramos
f1b64e90e0
Add comment to explain unusual fix
2020-03-25 11:49:14 +00:00
Matt-Yorkley
433ae00818
Ensure :info log level is set in Rails 4
2020-03-25 11:39:03 +00:00
Matt-Yorkley
7bc29769cd
Fix logging configs for Rails 4
2020-03-25 11:38:29 +00:00
Matt-Yorkley
483d847b00
Use eager_load = true in production and staging environments
2020-03-25 11:36:33 +00:00
Luis Ramos
95ba1fd02c
Merge pull request #5065 from luisramos0/upgrade_deps
...
[Spree 2.1] Upgrade some dependencies... rails 4.0.13 :-)
2020-03-25 11:04:13 +00:00
Luis Ramos
c02a3e919f
Merge pull request #5039 from luisramos0/strong_params_ent
...
[Spree 2.1] Implement strong params in enterprises, enterprise_groups, enterprise_roles, customers and customer_details controllers
2020-03-25 10:53:01 +00:00
Luis Ramos
b7cb95ae3e
Fix problem in PermittedAttributes::Address namespace
2020-03-25 10:52:23 +00:00
Luis Ramos
6b62c8aafd
Extract permitted attributes to separate service
2020-03-25 10:52:23 +00:00
Luis Ramos
e5f56c19c0
Switch to using PermittedAttributes::Address instead of spree version of it that will be removed later
2020-03-25 10:52:23 +00:00
Luis Ramos
22a005df47
Add needed permitted attributes to admin/customers_controller
2020-03-25 10:51:55 +00:00
Luis Ramos
9b0d7b9604
Ammend customers_controller to include ship address attributes permit and also permit #update with specific attributes
2020-03-25 10:51:55 +00:00
Luis Ramos
4fd3026bd8
Add strong parameters permits to some controllers
2020-03-25 10:51:55 +00:00
Luis Ramos
a9a92e11e2
Bring some strong parameters code from spree to our Spree controllers
...
This code comes from spree commit fbc2d150f6
2020-03-25 10:51:55 +00:00
Luis Ramos
fec5e1d84e
Add needed param to enterprises controller
2020-03-25 10:51:55 +00:00
Luis Ramos
ad9e5d979a
Permit extra needed params in enterprises controller
2020-03-25 10:51:55 +00:00
Luis Ramos
29a457575b
Permit specific params in enterprise_groups controller
2020-03-25 10:51:55 +00:00
Luis Ramos
5c179a0932
Ammend strong params on enterprise controller to cover create action
2020-03-25 10:51:55 +00:00
Luis Ramos
34488e5f63
Handle strong params in enterprise_roles controller
2020-03-25 10:51:55 +00:00
Luis Ramos
6ba3a3c373
Handle strong params in admin/enterprises_controller
2020-03-25 10:51:55 +00:00
Pau Pérez Fabregat
a72957e3c3
Merge pull request #4827 from luisramos0/strong_params
...
[Spree 2.1] Implement Strong Parameters in various controllers
2020-03-25 10:07:14 +01:00
Pau Pérez Fabregat
2c487c2592
Merge pull request #5037 from luisramos0/strong_params_subs
...
[Spree 2.1] Implement strong params in subscriptions controller
2020-03-25 09:50:47 +01:00
Pau Pérez Fabregat
d4d72c6c6c
Merge pull request #5036 from luisramos0/strong_params_users
...
[Spree 2.1] Implement strong params in user related controllers
2020-03-25 09:48:52 +01:00
Pau Pérez Fabregat
b323364661
Merge pull request #5035 from luisramos0/strong_params_checkout
...
[Spree 2.1] Implement strong params in checkout controller
2020-03-25 09:46:53 +01:00
Pau Pérez Fabregat
e70daf53ce
Merge pull request #4876 from luisramos0/strong_params_oc
...
[Spree 2.1] Implement strong params in admin Order cycles controller
2020-03-25 09:44:13 +01:00
Pau Pérez Fabregat
c5621b7740
Merge pull request #5034 from luisramos0/strong_params_payment_methods
...
[Spree 2.1] Implement strong params in admin payment methods controller
2020-03-25 09:39:34 +01:00
Luis Ramos
570fb4bfb4
Upgrade factory_bot_rails
2020-03-24 19:18:13 +00:00
Luis Ramos
89cc7ee5ef
Upgrade devise-encryptable to 0.2.0 which is latest
2020-03-24 19:18:13 +00:00
Luis Ramos
5453fed716
Upgrade rails from 4.0.0 to 4.0.13
2020-03-24 19:18:13 +00:00
Luis Ramos
b8ed28b38c
Remove deface, it's already included in spree_core, it's not a direct dependency of OFN anymore
2020-03-24 19:18:13 +00:00
Luis Ramos
2de550eac1
Make OrderUpdate update all adjustments again as in v2.0.4 otherwise adjustments that are not shipment adjustments will not be calculated correctly
2020-03-22 20:08:11 +00:00
Luis Ramos
85f6be756e
Merge pull request #5027 from luisramos0/3-0-stable-mar20
...
[Spree 2.1] Merge master into 3-0-stable - 20 March 2020
2020-03-22 13:20:54 +00:00
Luis Ramos
658a024272
Re-add spring and newrelix_rpm
2020-03-22 10:41:59 +00:00
Luis Ramos
478f885b26
Fix problem in PermittedAttributes::Variant namespace
2020-03-21 22:36:31 +00:00
Luis Ramos
5e6a210632
Fix problem in PermittedAttributes::User namespace
2020-03-21 22:36:03 +00:00
Luis Ramos
4e43535512
Add missing permitted attribute
2020-03-21 20:30:34 +00:00
Luis Ramos
3ccd58d50b
Fix a problem in the permit list to allow a list to be taken
2020-03-21 20:30:34 +00:00
Luis Ramos
5a0319213f
Fix schedules controller permitted attributes
2020-03-21 20:30:34 +00:00
Luis Ramos
b7c0caf883
Add needed permitted attributes to orders_controller
2020-03-21 20:30:34 +00:00
Luis Ramos
8c5dfea92f
Fix strong params in order_cycles and schedules controllers
2020-03-21 20:30:34 +00:00
Luis Ramos
2a426d29b6
Permit specific params in zones controller
2020-03-21 20:30:34 +00:00
Luis Ramos
47505b6225
Permit specific params in shipping methods controller
2020-03-21 20:30:34 +00:00
Luis Ramos
6fa1ed03dc
Permit specific params in adjustments controller
2020-03-21 20:30:34 +00:00
Luis Ramos
d73d851095
Permit specific params in tax_rates controller
2020-03-21 20:30:34 +00:00
Luis Ramos
495c3a3810
Permit specific params in images controller
2020-03-21 20:30:34 +00:00
Luis Ramos
9681437fba
Permit specific params in tax_categories controller
2020-03-21 20:30:34 +00:00
Luis Ramos
b99d4ab627
Permit specific params in schedules controller
2020-03-21 20:30:34 +00:00
Luis Ramos
423d5a5a61
Permit specific params in taxonomies controller
2020-03-21 20:30:34 +00:00
Luis Ramos
e5a214da78
Permit specific params in states controller
2020-03-21 20:30:34 +00:00
Luis Ramos
46025915d5
Make taxons_controller strong_params method consistent with all other controllers
2020-03-21 20:30:34 +00:00
Luis Ramos
e23267156d
Improve inventory_items_controller strong params by not using permit!
2020-03-21 20:30:34 +00:00
Luis Ramos
6bd72f44de
Handle strong params in credit card controllers
2020-03-21 20:30:34 +00:00
Luis Ramos
913ea5b883
Handle strong parameters in bulk_line_items controller
2020-03-21 20:30:34 +00:00
Luis Ramos
d496a4bdc8
Add strong parameters permits to some controllers
2020-03-21 20:30:34 +00:00
Luis Ramos
c15433af3f
Bring some strong parameters code from spree to our Spree controllers
...
This code comes from spree commit fbc2d150f6
2020-03-21 20:30:34 +00:00
Luis Ramos
244499a27d
Extract permitted atttributes to specific classes
2020-03-21 19:41:05 +00:00
Luis Ramos
58a2805bc9
Make resource controller raise error if permitted_resource_params is not overriden
2020-03-21 19:41:05 +00:00
Luis Ramos
49a2522305
Permit specific params in variants controller
2020-03-21 19:27:55 +00:00
Luis Ramos
527f6cb624
Add necessary attributes to admin/products_controller and handle empty params case
2020-03-21 19:14:26 +00:00
Luis Ramos
36389b7bed
Add missing permitted attributes to bulk_product_update controller
2020-03-21 19:14:26 +00:00
Luis Ramos
b5cdcdf8cb
Permit specific params in products controller
2020-03-21 19:14:26 +00:00
Luis Ramos
a261ae118d
Add missing permitted attributes to variant overrides controller
2020-03-21 19:14:26 +00:00
Luis Ramos
7320b38b93
Add missing attributes to variant override controller
2020-03-21 19:14:26 +00:00
Luis Ramos
0151b5ee9a
Permit extra needed params in variant overrides controller
2020-03-21 19:14:26 +00:00
Luis Ramos
5b37e89738
Handle strong params in variant_overrides_controller
...
We use a simpler way to permit on array within params here and change products_controller to the same style
2020-03-21 19:14:26 +00:00
Luis Ramos
c110f4832d
Handle strong params in spree/admin/products_controller
2020-03-21 19:14:26 +00:00
Luis Ramos
79b0867507
Extract permitted attributes to separate service
2020-03-21 19:08:37 +00:00
Luis Ramos
58c83d056d
Add missing permitted attributes to subscriptions controller
2020-03-21 18:57:37 +00:00
Luis Ramos
c3897b2f1c
Handle strong params in subscriptions controller
2020-03-21 18:57:37 +00:00
Luis Ramos
d7cfda8385
Handle strong params in subscription_line_items controller
2020-03-21 18:57:37 +00:00
Luis Ramos
0ee562c718
Add test coverage for PermittedAttributes::User
2020-03-21 18:48:04 +00:00
Luis Ramos
aec7f12f5a
Extract common user permitted attributes to a separate class
2020-03-21 18:31:14 +00:00
Luis Ramos
6ed93da3f1
Fix case with empty spree_user in user_registrations controller
2020-03-21 18:00:04 +00:00
Luis Ramos
d0bd2818c2
Handle strong params on users_controller
2020-03-21 18:00:04 +00:00
Luis Ramos
431076fc6d
Add strong parameters permits to admin users_controller
2020-03-21 17:59:46 +00:00
Luis Ramos
980fdd65a1
Replace hash rockets syntax
2020-03-21 17:00:08 +00:00
Luis Ramos
2c453359c1
Extract permitted params into a specific service
2020-03-21 16:49:26 +00:00
Luis Ramos
1d9a6edefb
Permit params in checkout controller before we adapt them to bypass issues with permitting added attributes like source CreditCard
2020-03-21 16:30:20 +00:00
Luis Ramos
5ae2e6865c
Add one more needed permitted attribute to checkout controller
2020-03-21 16:30:20 +00:00
Luis Ramos
d7cccd4143
Add guard clause in checkout_controller for empty params[:order]
2020-03-21 16:30:20 +00:00
Luis Ramos
5af27bb14e
Make checkout controller handle strong parameters
2020-03-21 16:30:20 +00:00
Luis Ramos
fd2cf7295e
Extract permitted_attributes from order_cycle_controller into a specific service
2020-03-21 16:17:20 +00:00
Luis Ramos
20c7a0d3ef
Adapt to latest changes in order cycles controller strong params changes
2020-03-21 14:45:51 +00:00
Luis Ramos
57f8fa26ab
Fix strong params in order_cycles
2020-03-21 14:45:51 +00:00
Luis Ramos
1a46e7b7ee
Improve strong params implementation on order_cycle controller and fix corresponding specs
2020-03-21 14:45:51 +00:00
Luis Ramos
905811ccb3
Handle strong params in admin order_cycles controller
2020-03-21 14:45:51 +00:00
Luis Ramos
df799340df
Add missing permitted attributes to payment_methods controller
2020-03-21 14:21:52 +00:00
Luis Ramos
eac0da9812
Fix payment method controllers by removing unnecessary param that only exists in stripe connect payment method preferred_enterprise_id
2020-03-21 14:09:33 +00:00
Luis Ramos
e5ebf45765
Improve strong params implementation on payment methods controller by specifying specific list of permitted attributes
2020-03-21 14:09:33 +00:00
Luis Ramos
38849f5589
Extract method in payments_method_controller to make it readable
2020-03-21 14:09:32 +00:00
Luis Ramos
86d09ff21e
Bring strong parameters code from spree to payment_methods_controller
...
This code comes from spree commit fbc2d150f6
2020-03-21 14:08:03 +00:00
Luis Ramos
337b2ab592
Merge pull request #4979 from Matt-Yorkley/3-0-customer-details
...
[Spree 2.1] Customer details
2020-03-21 13:26:09 +00:00
Matt-Yorkley
80e5608436
Ensure customer selection dropdown is visible when adding customer details
...
For some reason the order is in address state here instead of cart state when adding customer details. There's a conditional in the view which loads the customer selection dropdown that was not being triggered, so a customer could not be selected (and their existing address was not being filled out in the form).
Fixes:
1)
As an administrator
I want to manage orders
filling customer details
Failure/Error: expect(order.ship_address.lastname).to eq @customer.ship_address.lastname
NoMethodError:
undefined method `lastname' for nil:NilClass
# ./spec/features/admin/orders_spec.rb:179:in `block (2 levels) in <top (required)>'
2020-03-21 14:14:31 +01:00
Luis Ramos
b33969e6a2
Adapt to slightly different rails 4 sql data conversion
2020-03-21 12:50:34 +00:00
Luis Ramos
9915717a22
Remove attr_accessible from paper_trail config
...
In rails 4 this is not needed
2020-03-20 15:09:18 +00:00
Luis Ramos
03cdaf589e
Remove newly added attr_accessible from stripe_sca
...
This is not needed, in rails 4 the params are permitted at controller level
2020-03-20 15:08:29 +00:00
Luis Ramos
1c628fac58
Merge branch 'master' into 3-0-stable-mar20
2020-03-20 11:28:06 +00:00
Luis Ramos
eccaaca907
Merge pull request #4930 from luisramos0/3-0-stable-mar6
...
Merge master into 3-0-stable
2020-03-17 14:49:30 +00:00
Pau Pérez Fabregat
42d5b5d8a3
Merge pull request #4974 from luisramos0/payment_rollback
...
[Spree 2.1] Bypass broken payment rollback callback
2020-03-17 15:37:28 +01:00
Pau Pérez Fabregat
b2d015416a
Merge pull request #4934 from luisramos0/latest_spree_branch
...
[Spree 2.1] Use latest version of spree which includes PR openfoodfoundation/spree#39
2020-03-17 09:55:49 +01:00
Matt-Yorkley
1cba129432
Ensure line_item.variant succeeds when variant is soft-deleted
2020-03-12 19:13:15 +01:00
Matt-Yorkley
e24d8d7104
Fix deprecation warning for use of #includes without #references
2020-03-12 19:12:36 +01:00
Matt-Yorkley
40b8eb470c
Fix deprecation warning for use of #all
2020-03-12 19:12:06 +01:00
Luis Ramos
f2089af0c4
Merge pull request #4907 from luisramos0/3-0-enterprise_relationship
...
[Spree 2.1] Uncomment Enterprise#relate_to_owners_enterprises
2020-03-12 17:18:47 +00:00
Luis Ramos
949b6ebad1
Merge pull request #4982 from Matt-Yorkley/3-0-orders-form
...
[Spree 2.1] Fix order index search form
2020-03-12 16:58:08 +00:00
Luis Ramos
32a4355f09
Merge branch 'master' into 3-0-stable-mar6
2020-03-12 16:54:09 +00:00
Luis Ramos
a209b3e860
Merge pull request #4980 from Matt-Yorkley/3-0-destroy-route
...
[Spree 2.1] Line items destroy route
2020-03-12 16:28:05 +00:00
Matt-Yorkley
9a5452cd52
Delete test in line_items_controller_spec.rb
...
Rails 4 does not recognise requests to destroy routes without ids as valid routes.
Fixes:
14) LineItemsController destroying a line item on a completed order without a line item id fails and raises an error
Failure/Error: delete :destroy
ActionController::UrlGenerationError:
No route matches {:action=>"destroy", :controller=>"line_items"}
# ./spec/controllers/line_items_controller_spec.rb:46:in `block (5 levels) in <top (required)>'
2020-03-12 14:15:00 +01:00
Matt-Yorkley
a207c6c45f
Fix cart page update
...
Fixes:
2) full-page cart viewing the cart updating quantities with insufficient stock available shows the quantities saved, not those submitted
Failure/Error: @app.call(env)
AbstractController::ActionNotFound:
The action 'update' could not be found for CartController
# ./lib/open_food_network/rack_request_blocker.rb:36:in `call'
# ------------------
# --- Caused by: ---
# Capybara::ExpectationNotMet:
# expected to find text "Insufficient stock available, only 2 remaining" in "Internal Server Error The action 'update' could not be found for CartController WEBrick/1.3.1 (Ruby/2.3.7/2018-03-28) at 127.0.0.1:38614"
# ./spec/features/consumer/shopping/cart_spec.rb:182:in `block (5 levels) in <top (required)>'
2020-03-12 13:34:55 +01:00
Pau Pérez Fabregat
62c35ef6e5
Merge pull request #4953 from luisramos0/sch_fix
...
Fix problem with misssing params in schedules controller
2020-03-12 12:57:27 +01:00
Pau Pérez Fabregat
f6a1a101a8
Merge pull request #4955 from luisramos0/auth_fix
...
[Spree 2.1] Fix authentication spec by adapting code to devise 3
2020-03-12 12:51:46 +01:00
Pau Pérez Fabregat
1cba857bdb
Merge pull request #4933 from luisramos0/ent_helper
...
[Spree 2.1] Fix tag rule applicator in enterprises_helper
2020-03-12 12:49:18 +01:00
Pau Pérez Fabregat
568cba9d58
Merge pull request #4941 from luisramos0/devise
...
[Spree 2.1] Delete unnecessary code related to spree_auth_devise
2020-03-12 12:48:59 +01:00
Pau Pérez Fabregat
4b12d0be46
Merge pull request #4940 from luisramos0/ent_ctrl
...
[Spree 2.1] Fix issues in EnterprisesController
2020-03-12 12:48:28 +01:00
Pau Pérez Fabregat
de06eed945
Merge pull request #4911 from luisramos0/3-0-manifest
...
[Spree 2.1] Change Shipment#manifest to include deleted variants (again, this time in rails 4)
2020-03-12 12:47:45 +01:00
Matt-Yorkley
d62d89aa4f
Fix order index search form
...
Fixes:
2) spree/admin/orders/index.html.haml print invoices button displays button when invoices are enabled
Failure/Error: = form_tag nil, {name: "orders_form", "ng-submit" => "fetchResults()"} do
ActionView::Template::Error:
No route matches {:action=>"index", :controller=>"spree/admin/orders"}
# ./app/views/spree/admin/orders/_filters.html.haml:2:in `_a1b716152f19bf3f4772a98e56adf411'
# ./app/views/spree/admin/orders/index.html.haml:20:in `block in _d04c5552f7a480bf5f02e5fad0c10de6'
# ./app/views/spree/admin/orders/index.html.haml:19:in `_d04c5552f7a480bf5f02e5fad0c10de6'
# ./spec/views/spree/admin/orders/index.html.haml_spec.rb:26:in `block (3 levels) in <top (required)>'
# ./spec/views/spree/admin/orders/index.html.haml_spec.rb:8:in `block (2 levels) in <top (required)>'
# ------------------
# --- Caused by: ---
# ActionController::UrlGenerationError:
# No route matches {:action=>"index", :controller=>"spree/admin/orders"}
# ./app/views/spree/admin/orders/_filters.html.haml:2:in `_a1b716152f19bf3f4772a98e56adf411'
3) spree/admin/orders/index.html.haml print invoices button does not display button when invoices are disabled
Failure/Error: = form_tag nil, {name: "orders_form", "ng-submit" => "fetchResults()"} do
ActionView::Template::Error:
No route matches {:action=>"index", :controller=>"spree/admin/orders"}
# ./app/views/spree/admin/orders/_filters.html.haml:2:in `_a1b716152f19bf3f4772a98e56adf411'
# ./app/views/spree/admin/orders/index.html.haml:20:in `block in _d04c5552f7a480bf5f02e5fad0c10de6'
# ./app/views/spree/admin/orders/index.html.haml:19:in `_d04c5552f7a480bf5f02e5fad0c10de6'
# ./spec/views/spree/admin/orders/index.html.haml_spec.rb:34:in `block (3 levels) in <top (required)>'
# ./spec/views/spree/admin/orders/index.html.haml_spec.rb:8:in `block (2 levels) in <top (required)>'
# ------------------
# --- Caused by: ---
# ActionController::UrlGenerationError:
# No route matches {:action=>"index", :controller=>"spree/admin/orders"}
# ./app/views/spree/admin/orders/_filters.html.haml:2:in `_a1b716152f19bf3f4772a98e56adf411'
2020-03-12 12:44:50 +01:00
Luis Ramos
5506cc86d2
Bypass after_rollback callback in payment
...
This is causing several specs to fail on rollback and, according to some manual testing (and all the auto tests we have), we dont need this fix in OFN
2020-03-11 14:33:54 +00:00
Luis Ramos
de2c6a8717
Revert previous commits and use collection_singular_ids=(ids) setter instead
2020-03-10 11:21:36 +00:00
Luis Ramos
014e22a7ad
Fix problem with misssing params in schedules controller and adapt its spec
2020-03-09 16:04:59 +00:00
Luis Ramos
5ce4fec68b
Add html to show the errors reported by devise 3 about password and password confirmation mismatch on user.errors[:password_confirmation] (instead of user.errors[:password])
2020-03-09 15:48:45 +00:00
Luis Ramos
8c367a2ed2
Merge branch 'master' into 3-0-stable-mar6
2020-03-09 13:20:19 +00:00
Luis Ramos
e5f05b4db9
Handle case where no user_ids are sent to the controller. Fixes original spec.
2020-03-07 20:10:32 +00:00
Luis Ramos
7daa68a8bc
Adapt consumer/shopping/orders_spec to use an order with a nil user to test a guest checkout, which is what happens in the real checkout
2020-03-07 20:00:40 +00:00
Luis Ramos
56c29ab472
Make EnterprisesController inherit from Api::BaseController to fix authentication and fix problem with saving params[:user_ids]
2020-03-07 18:14:36 +00:00
Luis Ramos
ffe814ce6b
Adapt specs to not use anonymous user
...
Api::BaseController#authenticate is already taking care of the unauthenticated use case when spree_current_user is nil
2020-03-07 18:14:07 +00:00
Luis Ramos
e4d307fe5e
Remove dead code related to user management
2020-03-07 18:14:07 +00:00
Luis Ramos
855d08957c
Adjust completed_order_with_totals to have a distributor so that logic around refreshing shipping rates doesnt fail badly
2020-03-06 16:01:29 +00:00
Luis Ramos
063f733dda
Use latest version of spree which includes PR openfoodfoundation/spree#39
2020-03-06 15:05:29 +00:00
Luis Ramos
4b21aaafc4
Convert ActiveRecord_Associations_CollectionProxy to array so that the modifications the tagRuleApplicator does on it will remain
...
Reject on ActiveRecord_Associations_CollectionProxy wont work
2020-03-06 13:51:53 +00:00
Luis Ramos
0815068050
Merge branch 'master' into 3-0-stable-mar6
2020-03-06 09:20:03 +00:00
Pau Pérez Fabregat
4ea891ee2f
Merge pull request #4834 from Matt-Yorkley/3-0-bind-message
...
[Spree 2.1] PG::ProtocolViolation: ERROR: bind message supplies 0 parameters...
2020-03-05 19:31:06 +01:00
Pau Pérez Fabregat
2a010a3233
Merge pull request #4919 from luisramos0/ship_ctrl
...
[Spree 2.1] Fix shipments admin/enterprises and api/enterprises controller specs
2020-03-05 19:07:10 +01:00
Luis Ramos
cfbec2accc
Fix controller specs after PR #4675 adapted code to rails 4 finders
2020-03-05 11:55:13 +00:00
Luis Ramos
0f726c207f
Fix shipments controller after PR #4675 adapted code to rails 4 finders
2020-03-04 14:47:32 +00:00
Luis Ramos
3b37fa8872
Move variant scoping from alias method into manifest method
2020-03-03 16:40:31 +00:00
Luis Ramos
5688de4936
Make Shipment#manifest work with deleted variants again
...
This makes the default variant scope in inventory_unit being used which includes deleted variants
2020-03-03 16:38:03 +00:00
Luis Ramos
d8a92eec4b
Bring Shipment#manifest from spree as is
2020-03-03 16:37:00 +00:00
Luis Ramos
e584233c9e
Uncomment code that was commented earlier on in the upgrade, we are now ready to handle this
2020-03-03 14:25:17 +00:00
Luis Ramos
cf777b3fa2
Merge pull request #4882 from Matt-Yorkley/3-0-api-with-nil-id
...
[Spree 2.1] Fix spec setup in api/orders_controller_spec
2020-03-03 10:00:32 +00:00
Pau Pérez Fabregat
9980723060
Merge pull request #4880 from Matt-Yorkley/3-0-duplicate-validations
...
[Spree 2.1] Fix duplicate validations in payment methods and shipping methods
2020-03-03 10:44:40 +01:00
Pau Pérez Fabregat
f5ec11047a
Merge pull request #4896 from Matt-Yorkley/3-0-cc_type
...
[Spree 2.1] Add missing :cc_type attribute to default :credit_card factory
2020-03-03 10:43:18 +01:00
Pau Pérez Fabregat
f50593ced8
Merge pull request #4883 from Matt-Yorkley/3-0-helper
...
[Spree 2.1] Fix expectation in helper spec
2020-03-03 10:42:28 +01:00
Pau Pérez Fabregat
b0962f65da
Merge pull request #4872 from Matt-Yorkley/3-0-touch
...
[Spree 2.1] Touch on new records
2020-03-03 10:39:58 +01:00
Pau Pérez Fabregat
a47745c1cc
Merge pull request #4866 from Matt-Yorkley/3-0-ofn-qz
...
[Spree 2.1] ofn-qz
2020-03-03 10:37:10 +01:00
Pau Pérez Fabregat
a60974c760
Merge pull request #4864 from luisramos0/oc_times
...
[Spree 2.1] Fix OrderCycle.earliest_closing_times spec
2020-03-03 10:36:35 +01:00
Pau Pérez Fabregat
e733761e8a
Merge pull request #4847 from Matt-Yorkley/3-0-enterprise-user
...
[Spree 2.1] Fix #create_enterprise_user
2020-03-03 10:30:25 +01:00
Pau Pérez Fabregat
390cc61273
Merge pull request #4867 from Matt-Yorkley/3-0-gmappable
...
[Spree 2.1] acts_as_gmappable
2020-03-03 10:28:12 +01:00
Pau Pérez Fabregat
1e2e50ce54
Merge pull request #4888 from Matt-Yorkley/3-0-shipping-method-id
...
[Spree 2.1] Remove assignment of invalid attribute in test setup
2020-03-03 10:26:15 +01:00
Pau Pérez Fabregat
c48bd12440
Merge pull request #4843 from Matt-Yorkley/3-0-product-import
...
[Spree 2.1] Product import specs
2020-03-03 10:21:48 +01:00
Matt-Yorkley
6cb8bb2fc3
Add missing :cc_type attribute to default :credit_card factory
...
Fixes 5 specs, including:
11) Spree::CreditCardsController#update when the specified credit card is found and the card is owned by the user when the update completes successfully renders a serialized copy of the updated card
Failure/Error: object.cc_type.capitalize
NoMethodError:
undefined method `capitalize' for nil:NilClass
# ./app/serializers/api/credit_card_serializer.rb:6:in `brand'
# (eval):4:in `_fast_attributes'
# ./app/controllers/spree/credit_cards_controller.rb:26:in `update'
# ./spec/controllers/spree/credit_cards_controller_spec.rb:100:in `block (7 levels) in <top (required)>'
# ./spec/controllers/spree/credit_cards_controller_spec.rb:100:in `block (6 levels) in <top (required)>'
# ------------------
# --- Caused by: ---
# NoMethodError:
# undefined method `capitalize' for nil:NilClass
# ./app/serializers/api/credit_card_serializer.rb:6:in `brand'
2020-03-02 22:07:20 +01:00
Matt-Yorkley
13b8cfcd9c
Remove assignment of invalid attribute in test setup
...
Fixes:
11) Order Management viewing a completed order when checking out as an anonymous guest allows the user to see the details
Failure/Error:
order.update_attributes!(
bill_address: bill_address,
ship_address: ship_address,
shipping_method_id: shipping_method.id
)
ActiveRecord::UnknownAttributeError:
unknown attribute: shipping_method_id
# ./spec/features/consumer/shopping/orders_spec.rb:30:in `block (3 levels) in <top (required)>'
# ------------------
# --- Caused by: ---
# NoMethodError:
# undefined method `shipping_method_id=' for #<Spree::Order:0x00007f5e82a94d80>
# ./spec/features/consumer/shopping/orders_spec.rb:30:in `block (3 levels) in <top (required)>'
12) Order Management viewing a completed order when logged in as the customer allows the user to see order details
Failure/Error:
order.update_attributes!(
bill_address: bill_address,
ship_address: ship_address,
shipping_method_id: shipping_method.id
)
ActiveRecord::UnknownAttributeError:
unknown attribute: shipping_method_id
# ./spec/features/consumer/shopping/orders_spec.rb:30:in `block (3 levels) in <top (required)>'
# ------------------
# --- Caused by: ---
# NoMethodError:
# undefined method `shipping_method_id=' for #<Spree::Order:0x00007f5e83382708>
# ./spec/features/consumer/shopping/orders_spec.rb:30:in `block (3 levels) in <top (required)>'
13) Order Management viewing a completed order when not logged in allows the user to see order details after login
Failure/Error:
order.update_attributes!(
bill_address: bill_address,
ship_address: ship_address,
shipping_method_id: shipping_method.id
)
ActiveRecord::UnknownAttributeError:
unknown attribute: shipping_method_id
# ./spec/features/consumer/shopping/orders_spec.rb:30:in `block (3 levels) in <top (required)>'
# ------------------
# --- Caused by: ---
# NoMethodError:
# undefined method `shipping_method_id=' for #<Spree::Order:0x00007f5e8bafe6f0>
# ./spec/features/consumer/shopping/orders_spec.rb:30:in `block (3 levels) in <top (required)>'
2020-03-02 20:37:58 +01:00
Matt-Yorkley
0d08ad7d2e
Fix expectation in helper spec
...
The output of this link helper has changed slightly. All the attributes are the same as before, but they seem to be in alphabetical order now.
Fixes:
69) Spree::BaseHelper#link_to_remove_fields returns an `a` tag followed by a hidden `input` tag
Failure/Error: expect(subject).to eq("<a href=\"#\" class=\"remove_fields icon_link with-tip icon-trash\" data-action=\"remove\" title=\"Remove\"><span class='text'>Hola</span></a><input type="hidden" name="_method" value="destroy">")
expected: "<a href=\"#\" class=\"remove_fields icon_link with-tip icon-trash\" data-action=\"remove\" title=\"...</span></a><input type="hidden" name="_method" value="destroy">"
got: "<a class=\"remove_fields icon_link with-tip icon-trash\" data-action=\"remove\" href=\"#\" title=\"...</span></a><input type="hidden" name="_method" value="destroy">"
(compared using ==)
# ./spec/helpers/spree/admin/base_helper_spec.rb:12:in `block (3 levels) in <top (required)>'
2020-03-02 19:41:47 +01:00
Matt-Yorkley
0784b8263a
Fix spec setup in api/orders_controller_spec
...
ActionController doesn't accept nil values for :id as a valid route request in Rails 4.
Fixes:
2) Api::OrdersController#show Resource not found when no order number is given
Failure/Error: get :show, id: nil
ActionController::UrlGenerationError:
No route matches {:action=>"show", :controller=>"api/orders", :id=>nil}
# ./spec/controllers/api/orders_controller_spec.rb:168:in `block (4 levels) in <module:Api>'
2020-03-02 19:22:43 +01:00
Matt-Yorkley
ed5351d23e
Fix duplicate validations in payment methods and shipping methods
...
Updates the syntax to follow the recommended usage in the code comments of the ActiveModel #validates_with method.
Fixes:
12) Spree::PaymentMethod raises errors when required fields are missing
Failure/Error: expect(pm.errors.to_a).to eq(["Name can't be blank", "At least one hub must be selected"])
expected: ["Name can't be blank", "At least one hub must be selected"]
got: ["Name can't be blank", "At least one hub must be selected", "At least one hub must be selected"]
(compared using ==)
# ./spec/models/spree/payment_method_spec.rb:16:in `block (2 levels) in <module:Spree>'
2020-03-02 19:02:07 +01:00
Matt-Yorkley
a0999f5d58
Check #persisted? before calling #touch
...
Rails 4 now throws a fatal error if calling #touch on an object that hasn't been saved yet: c63cfc8722/activerecord/lib/active_record/persistence.rb (L957)
Fixes:
41) Stock::Package#shipping_methods does not return shipping methods not used by the package's order distributor
Failure/Error: distributors.each(&:touch)
ActiveRecord::ActiveRecordError:
can not touch on a new record object
# ./app/models/spree/shipping_method_decorator.rb:81:in `touch_distributors'
# ./spec/models/stock/package_spec.rb:39:in `block (2 levels) in <module:Stock>'
# ./spec/models/stock/package_spec.rb:17:in `block (2 levels) in <module:Stock>'
# ./spec/models/stock/package_spec.rb:32:in `block (2 levels) in <module:Stock>'
# ./spec/models/stock/package_spec.rb:7:in `block (2 levels) in <module:Stock>'
# ./spec/models/stock/package_spec.rb:44:in `block (3 levels) in <module:Stock>'
42) Stock::Package#shipping_categories returns shipping categories that are not shipping categories of the order's products
Failure/Error: distributors.each(&:touch)
ActiveRecord::ActiveRecordError:
can not touch on a new record object
# ./app/models/spree/shipping_method_decorator.rb:81:in `touch_distributors'
# ./spec/models/stock/package_spec.rb:39:in `block (2 levels) in <module:Stock>'
# ./spec/models/stock/package_spec.rb:17:in `block (2 levels) in <module:Stock>'
# ./spec/models/stock/package_spec.rb:32:in `block (2 levels) in <module:Stock>'
# ./spec/models/stock/package_spec.rb:7:in `block (2 levels) in <module:Stock>'
# ./spec/models/stock/package_spec.rb:50:in `block (3 levels) in <module:Stock>'
2020-03-01 18:40:15 +01:00
Matt-Yorkley
847a349a6b
Fix geocoding in Rails 4 🎉
...
Responsibility for geocoding has moved from the `gmaps4rails` gem using `acts_as_taggable`, to the `geocoding` gem using `geocoded_by`. We already use this in the Address model.
2020-02-29 13:08:06 +01:00
Matt-Yorkley
7dbe0bd6b9
Use new branch of ofn-qz gem
2020-02-28 15:27:11 +01:00
Matt-Yorkley
5c526bf5cc
Move definitions into constants and use #freeze
2020-02-28 13:08:17 +01:00
Matt-Yorkley
a9e6622639
Update product import #assign_attributes calls
...
For some reason some of the superfluous attributes being assigned here have started throwing fatal errors in Rails 4 instead of being silently ignored...
2020-02-28 13:08:17 +01:00
Luis Ramos
ddebd47e32
Merge pull request #4862 from Matt-Yorkley/3-0-paypal
...
[Spree 2.1] Re-introduce Paypal gem
2020-02-28 11:25:16 +00:00
Luis Ramos
56c6ef5e79
Merge pull request #4853 from Matt-Yorkley/3-0-base-controller
...
[Spree 2.1] Fix headers in api/base_controller spec
2020-02-28 11:21:16 +00:00
Matt-Yorkley
eef1574ebe
Call #permit on params object in before_filter
2020-02-28 12:18:53 +01:00
Matt-Yorkley
95996dc11e
Un-comment :credit_cards in routes
2020-02-28 12:18:53 +01:00
Matt-Yorkley
78e430613f
Fix spree routes helper in payment_redirect.rb
2020-02-28 12:18:53 +01:00
Luis Ramos
4f3247c196
Revert "Temporarily remove broken specs"
...
This reverts commit c0dc999e0d .
2020-02-28 12:18:53 +01:00
Luis Ramos
427d3b7641
Revert "Comment spree_paypal express in all.js temporarily"
...
This reverts commit 26ba37b117 .
2020-02-28 12:18:53 +01:00
Luis Ramos
61439a3750
Re-add paypal controller code
2020-02-28 12:18:53 +01:00
Matt-Yorkley
1f59bc8bef
Use new branch of the paypal gem
...
This uses the `2-1-0-stable` branch of the paypal gem. It's the 2-1-stable branch plus a couple of old OFN modifications (from 2-0-stable), plus a minor modification to the gemspec to allow it to be used with spree_core `2.1.0`
2020-02-28 12:18:53 +01:00
Matt-Yorkley
cc1ab1bdb9
Fix issue with broken SQL fragments in scopes and nested subqueries
...
It looks like there are some issues with prepared statements here, where the resulting SQL contains something like: `WHERE "enterprise_roles"."user_id" = $1` in a subquery. The "$1" part is being lost somehow and isn't present if it's used in nested subqueries.
Example fixed spec (there are lots like this one):
59) Spree::Admin::ReportsController Supplier Bulk Coop where I have granted P-OC to the distributor only shows product line items that I am supplying
Failure/Error: produced_line_items.select("spree_line_items.id"))
ActiveRecord::StatementInvalid:
PG::ProtocolViolation: ERROR: bind message supplies 0 parameters, but prepared statement "" requires 1
: SELECT id FROM "spree_line_items" WHERE "spree_line_items"."order_id" IN (SELECT id FROM "spree_orders" WHERE (("spree_orders"."distributor_id" IN (SELECT enterprises.id FROM "enterprises" INNER JOIN "enterprise_roles" ON "enterprise_roles"."enterprise_id" = "enterprises"."id" WHERE (enterprise_roles.user_id = 947)) OR "spree_orders"."order_cycle_id" IN (SELECT id FROM "order_cycles" WHERE "order_cycles"."coordinator_id" IN (SELECT "enterprises"."id" FROM "enterprises" INNER JOIN "enterprise_roles" ON "enterprises"."id" = "enterprise_roles"."enterprise_id" WHERE "enterprise_roles"."user_id" = $1)))))
# ./app/services/permissions/order.rb:28:in `visible_line_items'
# ./lib/open_food_network/reports/line_items.rb:16:in `list'
# ./lib/open_food_network/bulk_coop_report.rb:54:in `table_items'
# ./app/controllers/spree/admin/reports_controller.rb:264:in `order_grouper_table'
# ./app/controllers/spree/admin/reports_controller.rb:101:in `bulk_coop'
# ./spec/controllers/spree/admin/reports_controller_spec.rb:168:in `block (5 levels) in <top (required)>'
# ------------------
# --- Caused by: ---
# PG::ProtocolViolation:
# ERROR: bind message supplies 0 parameters, but prepared statement "" requires 1
# ./app/services/permissions/order.rb:28:in `visible_line_items'
2020-02-28 12:16:55 +01:00
Luis Ramos
e5232c6b79
Merge pull request #4863 from Matt-Yorkley/3-0-gemfile-lock-fixed
...
[Spree 2.1] Fix Gemfile.lock
2020-02-28 11:15:24 +00:00
Luis Ramos
95415cddbd
Merge pull request #4858 from Matt-Yorkley/3-0-embedded
...
[Spree 2.1] Fix embedded response headers
2020-02-28 11:12:16 +00:00
Matt-Yorkley
789ecbb8f2
Fix Gemfile.lock
2020-02-28 12:06:44 +01:00
Luis Ramos
0e607003c1
Fix problem in order cycles spec where miliseconds were not matching
2020-02-28 11:06:16 +00:00
Luis Ramos
1d0392e644
Merge pull request #4848 from Matt-Yorkley/3-0-test-locales
...
[Spree 2.1] Add 'pt' to list of available locales in test suite
2020-02-28 11:04:23 +00:00
Matt-Yorkley
02d46b7c51
Fix test in order_cycles_spec.rb
2020-02-28 11:51:09 +01:00
Matt-Yorkley
8672124f31
Add 'pt' to list of available locales in test suite
...
Fixes several errors such as:
104) UserRegistrationsController via ajax sets user.locale from cookie on create
Failure/Error: I18n.locale = spree_current_user.andand.locale || cookies[:locale] || I18n.default_locale
I18n::InvalidLocale:
"pt" is not a valid locale
# ./app/helpers/i18n_helper.rb:14:in `set_locale'
# ./spec/controllers/user_registrations_controller_spec.rb:56:in `block (3 levels) in <top (required)>'
2020-02-28 11:44:33 +01:00
Pau Pérez Fabregat
9440363a22
Merge pull request #4821 from luisramos0/3_val_issues
...
[Spree 2.1] Fix creation of enterprises/users in specs
2020-02-28 11:34:27 +01:00
Pau Pérez Fabregat
19dd5d0af3
Merge pull request #4855 from luisramos0/3-0-ship-method-services
...
[Spree 2.1] Fix ShippingMethod.services query
2020-02-28 11:31:53 +01:00
Pau Pérez Fabregat
3d453eb5ee
Merge pull request #4861 from luisramos0/3-0-merge-master
...
[Spree 2.1] Merge master into 3-0-stable
2020-02-28 11:28:56 +01:00
Pau Pérez Fabregat
de9f14c305
Merge pull request #4849 from Matt-Yorkley/3-0-private-method
...
[Spree 2.1] Make private method #total_units public
2020-02-28 11:00:12 +01:00
Pau Pérez Fabregat
455241859e
Merge pull request #4850 from Matt-Yorkley/3-0-shipments
...
[Spree 2.1] Fix expectations in order_shipment_spec
2020-02-28 10:58:58 +01:00
Luis Ramos
bf47d34434
Merge branch 'master' into 3-0-merge-master
2020-02-28 09:52:01 +00:00
Pau Pérez Fabregat
d03d1c0cc2
Merge pull request #4832 from Matt-Yorkley/3-0-more_strong_params
...
[Spree 2.1] Add strong_params to admin/enterprise_relationships_controller.rb
2020-02-28 10:41:05 +01:00
Pau Pérez Fabregat
6854301025
Merge pull request #4828 from Matt-Yorkley/3-0-strong_params
...
[Spree 2.1] Strong params
2020-02-28 10:40:52 +01:00
Pau Pérez Fabregat
dc9ebd620a
Merge pull request #4833 from Matt-Yorkley/3-0-moar_params
...
[Spree 2.1] Add strong params to admin/column_preferences_controller.rb
2020-02-28 10:30:54 +01:00
Matt-Yorkley
7124c6bb73
Fix embedded response headers
...
Rails 4 adds an extra layer of "default" headers that override any that are missing (or deleted). This was breaking embedded shopfront responses.
Fixes:
74) setting response headers for embedded shopfronts with embedded shopfronts enabled with a valid whitelist allows iframes on certain pages when enabled in configuration
Failure/Error: expect(response.headers['X-Frame-Options']).to be_nil
expected: nil
got: "SAMEORIGIN"
# ./spec/requests/embedded_shopfronts_headers_spec.rb:54:in `block (4 levels) in <top (required)>'
75) setting response headers for embedded shopfronts with embedded shopfronts enabled with www prefix matches the URL structure in the header
Failure/Error: expect(response.headers['X-Frame-Options']).to be_nil
expected: nil
got: "SAMEORIGIN"
# ./spec/requests/embedded_shopfronts_headers_spec.rb:75:in `block (4 levels) in <top (required)>'
2020-02-27 20:53:09 +01:00
Luis Ramos
bd0245229b
Move enterprise logo and promo_image setting in factories to after creation fixes a problem with enterprise_roles in the enterprise validation
2020-02-27 18:32:56 +00:00
Luis Ramos
b1714d73ef
Capitalize 'Method' in the flash message to make specs pass
2020-02-27 18:01:24 +00:00
Luis Ramos
ade52dd07d
Make ship methods services query work with rails 4, bool_or now results in a boolean and not in t or f as before
2020-02-27 17:40:32 +00:00
Matt-Yorkley
0fa73e3b02
Fix headers in api/base_controller spec
...
The headers in the request were not being populated correctly in the test, so the #api_key method was not functioning as intended.
Fixes:
48) Api::BaseController cannot make a request to the API with an invalid API key
Failure/Error: expect(json_response).to eq( "error" => "Invalid API key (fake_key) specified." )
expected: {"error"=>"Invalid API key (fake_key) specified."}
got: {"products"=>[]}
(compared using ==)
Diff:
@@ -1,2 +1,2 @@
-"error" => "Invalid API key (fake_key) specified.",
+"products" => [],
# ./spec/controllers/api/base_controller_spec.rb:40:in `block (3 levels) in <top (required)>'
2020-02-27 11:23:48 +01:00
Matt-Yorkley
c758cb5508
Fix expectations in order_shipment_spec
...
Some depreacted calls to #find_by_* had previously been updated elsewhere, but the spec was still checking if the object received the #find_by_shipping_method_id message.
Fixes:
85) OrderShipment#select_shipping_method when order has a shipment when shipping_method_id is not valid for the order returns nil
Failure/Error: expect(shipment.shipping_rates).to receive(:find_by_shipping_method_id).with(invalid_shipping_method_id) { nil }
(#<ActiveRecord::Associations::CollectionProxy []>).find_by_shipping_method_id(1312)
expected: 1 time with arguments: (1312)
received: 0 times
# ./spec/models/concerns/order_shipment_spec.rb:53:in `block (5 levels) in <top (required)>'
2020-02-26 11:50:51 +01:00
Matt-Yorkley
82052e459d
Make private method #total_units public
...
This method is being called by another class as if it were public.
Fixes 2 instances of this error:
90) OpenFoodNetwork::OrdersAndFulfillmentsReport::SupplierTotalsReport generates the report
Failure/Error: delegate :supplier_name, :product_name, :line_items_name, :total_units, to: :context
NoMethodError:
private method `total_units' called for #<OpenFoodNetwork::OrdersAndFulfillmentsReport:0x00007f8db5d67168>
# ./lib/open_food_network/orders_and_fulfillments_report/supplier_totals_report.rb:8:in `total_units'
# ./lib/open_food_network/orders_and_fulfillments_report/supplier_totals_report.rb:48:in `block in columns'
# ./lib/open_food_network/order_grouper.rb:41:in `block in build_table'
# ./lib/open_food_network/order_grouper.rb:41:in `map'
# ./lib/open_food_network/order_grouper.rb:41:in `build_table'
# ./lib/open_food_network/order_grouper.rb:47:in `block in build_table'
# ./lib/open_food_network/order_grouper.rb:43:in `each'
# ./lib/open_food_network/order_grouper.rb:43:in `build_table'
# ./lib/open_food_network/order_grouper.rb:47:in `block in build_table'
# ./lib/open_food_network/order_grouper.rb:43:in `each'
# ./lib/open_food_network/order_grouper.rb:43:in `build_table'
# ./lib/open_food_network/order_grouper.rb:47:in `block in build_table'
# ./lib/open_food_network/order_grouper.rb:43:in `each'
# ./lib/open_food_network/order_grouper.rb:43:in `build_table'
# ./lib/open_food_network/order_grouper.rb:56:in `table'
# ./spec/lib/open_food_network/orders_and_fulfillments_report/supplier_totals_report_spec.rb:19:in `block (2 levels) in <top (required)>'
# ./spec/lib/open_food_network/orders_and_fulfillments_report/supplier_totals_report_spec.rb:23:in `block (2 levels) in <top (required)>'
2020-02-26 11:35:00 +01:00
Matt-Yorkley
167c8543aa
Fix #create_enterprise_user
...
User was not being associated to enterprises passed in attributes
2020-02-26 10:46:41 +01:00
Matt-Yorkley
90d1a5e605
Add strong params to admin/column_preferences_controller.rb
...
Fixes failures such as:
119) Admin::ColumnPreferencesController bulk_update json where I don't own the preferences submitted prevents me from updating the column preferences
Failure/Error: raise ActiveModel::ForbiddenAttributesError, params.to_s
ActiveModel::ForbiddenAttributesError:
{"action_name"=>"enterprises_index", "column_preferences"=>[{"id"=>1, "user_id"=>2716, "action_name"=>"enterprises_index", "column_name"=>"name", "visible"=>false}, {"id"=>nil, "user_id"=>2716, "action_name"=>"enterprises_index", "column_name"=>"producer", "visible"=>true}, {"id"=>nil, "user_id"=>2716, "action_name"=>"enterprises_index", "column_name"=>"status", "visible"=>true}], "format"=>"json", "controller"=>"admin/column_preferences", "action"=>"bulk_update"}
# ./app/controllers/application_controller.rb:16:in `print_params'
# ./spec/controllers/admin/column_preferences_controller_spec.rb:28:in `block (5 levels) in <top (required)>'
# ------------------
# --- Caused by: ---
# ActiveModel::ForbiddenAttributesError:
# ActiveModel::ForbiddenAttributesError
# ./app/models/model_set.rb:24:in `block in collection_attributes='
2020-02-23 14:52:06 +01:00
Matt-Yorkley
fccc8037f0
Add strong_params to enterprise_relationships_controller.rb
...
Fixes specs such as:
31)
As an Administrator
I want to manage relationships between enterprises
as a site administrator creating a relationship
Failure/Error: raise ActiveModel::ForbiddenAttributesError, params.to_s
ActiveModel::ForbiddenAttributesError:
{"enterprise_relationship"=>{"parent_id"=>284, "child_id"=>285, "permissions_list"=>["add_to_order_cycle", "edit_profile", "create_variant_overrides"]}, "action"=>"create", "controller"=>"admin/enterprise_relationships"}
# ./app/controllers/application_controller.rb:16:in `print_params'
# ./lib/open_food_network/rack_request_blocker.rb:36:in `call'
# ------------------
# --- Caused by: ---
# ActiveModel::ForbiddenAttributesError:
# ActiveModel::ForbiddenAttributesError
# ./app/controllers/admin/enterprise_relationships_controller.rb:10:in `create'
2020-02-23 14:26:14 +01:00
Matt-Yorkley
f77194875c
Add strong parameters to orders_controller.rb
2020-02-22 16:24:30 +01:00
Matt-Yorkley
bd6b409708
Add strong parameters to user_registrations_controller.rb
2020-02-22 16:24:30 +01:00
Matt-Yorkley
eae7dc7f2e
Temporarily add extra debugging to help with strong parameters
2020-02-22 16:24:28 +01:00
Matt-Yorkley
f5bc120fb1
Remove without_protection: true argument from #create calls
...
Needed for using Strong Parameters in Rails 4
2020-02-22 14:35:10 +01:00
Matt-Yorkley
3896644a04
Remove attr_accessible declarations
...
Needed for using Strong Parameters in Rails 4
2020-02-22 14:03:16 +01:00
Matt-Yorkley
74965eb635
Remove protected_attributes gem
...
This starts the move towards using Strong Parameters in Rails 4
2020-02-22 13:41:46 +01:00
Luis Ramos
e64d21d81d
Move user enterprises definition in tests to after creation fixes a problem with enterprise_roles
2020-02-22 12:06:33 +00:00
Luis Ramos
1dc7b5793c
Fix Enterprise validation and factory so that users are added after the validation
2020-02-22 12:06:33 +00:00
Luis Ramos
10c6e5ad9b
Rebuild rubocop todo after latest rebase
2020-02-22 11:53:35 +00:00
Luis Ramos
f06a4fb1d0
This configuration is no longer needed as this is the default behaviour in rails 4
2020-02-22 11:53:35 +00:00
Luis Ramos
7640c7b606
Make all timestamps nullable, non-nullable timestamps are not required to maketimestamps work properly
2020-02-22 11:53:35 +00:00
Luis Ramos
26ba37b117
Comment spree_paypal express in all.js temporarily
2020-02-22 11:53:35 +00:00
Matt-Yorkley
8b54281254
Add missing method removed from order model in Spree 2.1
...
It looks like #price_adjustments was refactored/removed in Spree 2.1, but we still call the method in order_decorator
2020-02-22 11:06:51 +00:00
Matt-Yorkley
e9e4168102
Fix ActiveModel::MissingAttributeError in order_cycle_permissions.rb
...
Failure/Error: variant_ids_by_supplier_id(permissions.all_incoming_editable_variants.all)
ActiveModel::MissingAttributeError:
missing attribute: product_id
# ./app/serializers/api/admin/order_cycle_serializer.rb:36:in `editable_variants_for_incoming_exchanges'
# (eval):9:in `_fast_attributes'
# ./app/controllers/spree/admin/base_controller_decorator.rb:98:in `render_as_json'
# ./app/controllers/admin/order_cycles_controller.rb:28:in `block (2 levels) in show'
# ./app/controllers/admin/order_cycles_controller.rb:25:in `show'
# ./lib/open_food_network/rack_request_blocker.rb:36:in `call'
# ------------------
# --- Caused by: ---
# ActiveModel::MissingAttributeError:
# missing attribute: product_id
# ./app/serializers/api/admin/order_cycle_serializer.rb:36:in `editable_variants_for_incoming_exchanges'
2020-02-22 11:06:51 +00:00
Matt-Yorkley
b9db6df952
Replace current_spree_user method
...
This wasn't working before but now seems to be fixed
2020-02-22 11:06:51 +00:00
Matt-Yorkley
b3e7f9a07e
Fix deprecated arguments used in #sanitize
...
Failure/Error: d = sanitize(object.description, tags: "p, b, strong, em, i, a, u", attributes: "href, target")
ArgumentError:
You should pass :tags as an Enumerable
# ./app/serializers/api/product_serializer.rb:26:in `description_html'
# (eval):10:in `_fast_attributes'
# ./app/services/products_renderer.rb:24:in `products_json'
# ./app/controllers/api/order_cycles_controller.rb:14:in `products'
# ./lib/open_food_network/rack_request_blocker.rb:36:in `call'
2020-02-22 11:06:51 +00:00
Matt-Yorkley
04b760f221
Fix changed order of arguments for TestCase#process
...
DEPRECATION WARNING: TestCase#process now expects the HTTP method as second argument: process(action, http_method, params, session, flash). (called from api_process at /home/user/Github/openfoodnetwork/spec/support/controller_hacks.rb:22)
2020-02-22 11:06:51 +00:00
Matt-Yorkley
df1299b290
Fix user not yet loaded in prepend_before_filter
2020-02-22 11:06:51 +00:00
Matt-Yorkley
c0ecdb9e3a
Fix deprecated #includes in join without reference
...
DEPRECATION WARNING: It looks like you are eager loading table(s) (one of: variant_overrides, enterprises, enterprise_roles) that are referenced in a string SQL snippet. For example:
Post.includes(:comments).where("comments.title = 'foo'")
Currently, Active Record recognizes the table in the string, and knows to JOIN the comments table to the query, rather than loading comments in a separate query. However, doing this without writing a full-blown SQL parser is inherently flawed. Since we don't want to write an SQL parser, we are removing this functionality. From now on, you must explicitly tell Active Record when you are referencing a table from a string:
Post.includes(:comments).where("comments.title = 'foo'").references(:comments)
If you don't rely on implicit join references you can disable the feature entirely by setting `config.active_record.disable_implicit_join_references = true`. (called from collection at /home/user/Github/openfoodnetwork/app/controllers/admin/variant_overrides_controller.rb:77)
2020-02-22 11:06:51 +00:00
Matt-Yorkley
6c22d655c8
Temporarily remove additional email validation on Spree::Order
...
A new custom email validator class was added to Spree::Order, and currently it fails every time (breaking lots of specs). This ugly hack disables it for now.
2020-02-22 11:06:51 +00:00
Matt-Yorkley
3615dcd355
Fix namespacing in cookies test
...
NameError: uninitialized constant ActionDispatch::Cookies::SignedCookieJar::MAX_COOKIE_SIZE
# ./spec/requests/large_request_spec.rb:8
2020-02-22 11:06:51 +00:00
Matt-Yorkley
49bc3308be
Fix broken references to url_helpers
2020-02-22 11:06:51 +00:00
Matt-Yorkley
fd11d4fa5b
Fix broken html encoding of injected json in admin views
2020-02-22 11:06:51 +00:00
Matt-Yorkley
e3ae65fad5
Update rubocop_todo
2020-02-22 11:06:51 +00:00
Matt-Yorkley
ae11cb9954
Update rubocop_styleguide for Rails 4
2020-02-22 11:06:51 +00:00
Matt-Yorkley
98fdbb8621
Update deprecated #scoped calls
...
Model#scoped is now deprecated. Model#where(nil) in Rails 4 returns the same result as Model#scoped in Rails 3
2020-02-22 11:06:51 +00:00
Matt-Yorkley
241e9221b4
Replace deprecated :restrict option in address_decorator.rb
...
DEPRECATION WARNING: The :restrict option is deprecated. Please use :restrict_with_exception instead, which provides the same functionality. (called from block in <top (required)> at /home/user/Github/openfoodnetwork/app/models/spree/address_decorator.rb:4)
2020-02-22 11:06:51 +00:00
Matt-Yorkley
9fd4863a13
Fix missing roles error in spec workflow #create_enterprise_user
2020-02-22 11:06:51 +00:00
Matt-Yorkley
6541b55872
Fix default_association deprecated syntax in OrderCycle and Schedule
2020-02-22 11:06:51 +00:00
Matt-Yorkley
aff8911309
Update database_cleaner to >1.0 to fix transaction issues in test suite
2020-02-22 11:06:51 +00:00
Matt-Yorkley
39459b4a63
Temporarily fix root_path missing error in many controllers
2020-02-22 11:06:51 +00:00
Matt-Yorkley
7aef5af71c
Fix report types spec
2020-02-22 11:06:51 +00:00
Matt-Yorkley
0a9d63dd2f
Fix deprecated syntax on associations
2020-02-22 11:06:51 +00:00
Matt-Yorkley
5cc223e8f4
Update routes loading for Rails 4
2020-02-22 11:06:51 +00:00
Matt-Yorkley
a12ec8c5d9
Move filter parameters to initializer for Rails 4
2020-02-22 11:06:51 +00:00
Matt-Yorkley
046d8e0b87
Explicitly configure shoulda-matchers in spec_helper
...
This is apparently needed in Rails 4
2020-02-22 11:06:51 +00:00
Matt-Yorkley
16d56a8cbd
Routes using match must specify a request method in Rails 4
2020-02-22 11:06:51 +00:00
Matt-Yorkley
a8b8269a02
Update deprecated #find_or_initialize_by_* methods
2020-02-22 11:06:51 +00:00
Matt-Yorkley
b80929022b
Remove unused newrelic_rpm gem
2020-02-22 11:06:51 +00:00
Matt-Yorkley
53645517af
Update deprecated #find_by_* methods
2020-02-22 11:06:51 +00:00
Matt-Yorkley
30558485de
Update deprecated #find_or_create_by_* methods
2020-02-22 11:06:51 +00:00
Matt-Yorkley
6dd982532c
Update deprecated/changed Rubocop rule names
2020-02-22 11:06:51 +00:00
Matt-Yorkley
418e9d23fe
Add missing eager_load setting in test.rb
2020-02-22 11:06:51 +00:00
Matt-Yorkley
b8e608abc2
Remove deprecated #whiny_nils
2020-02-22 11:06:51 +00:00
Matt-Yorkley
6eef2555f1
Fix i18n available locales error
...
For some reason the locales defined in config/environment/test.rb are `en,es`, but in the used application.yml they were `en,es,en-GB`. This seems to trigger errors in the test environment ("en-GB is not a valid locale") if the two lists don't match..?
2020-02-22 11:06:51 +00:00
Matt-Yorkley
5603b70379
Fix html escaping of characters in AMS rendering causing fatal javascript errors
2020-02-22 11:06:51 +00:00
Matt-Yorkley
b03ef140f7
Enable sass
2020-02-22 11:06:51 +00:00
Matt-Yorkley
8ffdd806bf
Enable rails-i18n 4.0
2020-02-22 11:06:25 +00:00
Matt-Yorkley
1fa2d05c0c
Fix AuthenticationHelper#spree_current_user
...
`current_spree_user`: method not found...
2020-02-22 11:05:24 +00:00
Matt-Yorkley
5b8552f541
Fix ActiveRecord::Relation issue
...
Rails 4 now returns an Activerecord::Relation when using `SomeModel.all`. To get all the objects (as in Rails 3) we need to use `SomeModel.all.to_a`. See: https://stackoverflow.com/a/22417170
2020-02-22 11:05:24 +00:00
Matt-Yorkley
6f62c2cf27
Fix SCSS import file error
2020-02-22 11:05:24 +00:00
luisramos0
2f69d2a7e7
Remove devise email validation regexp, it's not needed in this version
2020-02-22 11:05:24 +00:00
luisramos0
2dc78d2f50
Remove spree_auth_devise and upgrade devise to 3.0.1
2020-02-22 11:05:24 +00:00
luisramos0
d1248c76fa
Temporarily disable caching classes in test to fix karma rake task
...
This fixes the error "can't modify immutable index" in lib/tasks/karma.rake:25:in `application_spec_files'
2020-02-22 11:05:24 +00:00
luisramos0
4ca9a19d8a
Temporarily remove broken fontawesome and spree_paypal_express assets
2020-02-22 11:05:24 +00:00
luisramos0
1ef9209183
Remove the assets group in the gemfile (removed in rails 4)
2020-02-22 11:05:24 +00:00
luisramos0
9c369caae2
Temporarily comment part of Enterprise.relate_to_owners_enterprises so that most specs pass for now
2020-02-22 11:05:24 +00:00
luisramos0
b23ce20488
Add migration from spree v2.1
2020-02-22 11:05:24 +00:00
luisramos0
18ca9980c9
Temporarily comment Enterprise.ensure_owner_is_manager otherwise all specs erroring out with 'Enterprise roles invalid'
2020-02-22 11:05:24 +00:00
luisramos0
c0dc999e0d
Temporarily remove broken specs
2020-02-22 11:05:24 +00:00
luisramos0
8fe3a8aea4
Setting secret key in config/initializers/secret_token.rb
2020-02-22 11:05:24 +00:00
luisramos0
8e747e54a0
Setting eager loading on each env configuration
2020-02-22 11:05:24 +00:00
luisramos0
8dbe4accaf
Upgrade to the spree branch 2-1-0-stable and rails 4.0.0
2020-02-22 11:05:24 +00:00