Pau Pérez Fabregat
ef84a165c7
Merge pull request #5663 from luisramos0/constraints
...
Make db timestamps nullable
2020-06-25 13:16:10 +02:00
Pau Perez
970750281d
Update all locales with the latest Transifex translations
2020-06-23 18:08:21 +02:00
Pau Pérez Fabregat
ec3899a4d4
Merge pull request #5660 from coopdevs/fix-spree-preferences
...
Migrate all spree pref. keys to the new format
2020-06-23 18:01:16 +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
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
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
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
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
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]
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
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
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
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
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
Pau Perez
a287f370b5
Remove unused home photos
2020-06-05 18:05:58 +02: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
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
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
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
Kristina Lim
cf534237cd
Update all locales with the latest Transifex translations
2020-05-23 01:51:23 +08:00
Pau Pérez Fabregat
da88399ca6
Merge pull request #5411 from Matt-Yorkley/orders-closed-fullwidth
...
Fix closed shop message not displaying full-width
2020-05-22 19:32:08 +02:00
Pau Pérez Fabregat
a5b4c5316c
Merge pull request #5458 from Matt-Yorkley/oc-form-ui-bug
...
Fix display issue on order cycle edit form
2020-05-22 19:31:41 +02:00
Pau Pérez Fabregat
f1dbad1007
Merge pull request #5466 from Matt-Yorkley/shop-search-fix
...
Fix search bar in /shops and /groups
2020-05-22 18:40:27 +02:00
Pau Pérez Fabregat
17e77df14f
Merge pull request #5451 from Matt-Yorkley/oc-selector-fix
...
Fix tab-buttons element overlapping order cycle selector
2020-05-22 18:39:10 +02:00
Pau Pérez Fabregat
3f593d6246
Merge pull request #5440 from luisramos0/grumpy_cat
...
Fix Grumpy Cat on shop page (cart with items from a closed OC)
2020-05-22 18:38:41 +02:00
Pau Pérez Fabregat
72399cb6df
Merge pull request #5485 from luisramos0/byeby_phantom_js
...
Rename method refering to phantomjs
2020-05-22 18:37:28 +02: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 Pérez Fabregat
b804036822
Merge pull request #5469 from openfoodfoundation/transifex
...
Transifex
2020-05-22 18:24:14 +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
Transifex-Openfoodnetwork
88ab5c2dd7
Updating translations for config/locales/fr.yml
2020-05-22 23:31:48 +10:00
Transifex-Openfoodnetwork
466bbe52ae
Updating translations for config/locales/en_FR.yml
2020-05-22 23:29:08 +10:00
Luis Ramos
9a71ffc093
Merge pull request #5438 from Matt-Yorkley/cart-stock-ux
...
Cart stock UX
2020-05-22 14:21:16 +01: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
Luis Ramos
b7c601abcd
Merge pull request #5424 from luisramos0/ship_method_warning
...
Add warning to enterprise Shipping Method list and to Shipping method edit page
2020-05-22 11:24:12 +01:00
Luis Ramos
639c1e1ccc
Rename method, it's not phantomjs running underneath any more
2020-05-22 10:57:51 +01:00
Robin Klaus
53c0bcbd0b
Changed translation syntax to normal t() call and lazy lookup.
2020-05-22 13:22:57 +10:00
Luis Ramos
6fcca0197c
Merge pull request #5450 from rmklaus12/5095-ordercycle-amtqty-translations
...
Changed report_header fields for amount and quantity to reflect corre…
2020-05-21 21:21:14 +01: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
Transifex-Openfoodnetwork
adb3c9e56f
Updating translations for config/locales/en_NZ.yml
2020-05-22 00:22:33 +10:00
Pau Pérez Fabregat
c6c9dcc345
Merge pull request #5478 from luisramos0/bundler1-17-3
...
Upgrade bundler, byebug and pry-byebug
2020-05-21 16:10:38 +02: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
Luis Ramos
6e6dab8f72
Merge pull request #5474 from openfoodfoundation/dependabot/bundler/activerecord-import-1.0.5
...
Bump activerecord-import from 1.0.4 to 1.0.5
2020-05-21 13:17:51 +01:00
Luis Ramos
574e4a02c0
Force pry to 0.12 so that it works well with pry-byebug 3.7
2020-05-21 13:10:59 +01:00
Luis Ramos
01eda90a00
Upgrade bundler, byebug and pry-byebug
2020-05-21 12:59:57 +01: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
Pau Pérez Fabregat
6851921e9d
Merge pull request #5443 from openfoodfoundation/dependabot/bundler/mini_racer-0.2.14
...
Bump mini_racer from 0.2.11 to 0.2.14
2020-05-21 13:32:30 +02:00
Luis Ramos
8cdc9b82df
Merge pull request #5427 from luisramos0/jquery-ui-rails
...
Upgrade jquery-ui-rails to v4.2.1
2020-05-21 10:57:10 +01:00
Luis Ramos
2ae34d8df3
Merge pull request #5355 from jeduardo824/enhancement/make-shop-names-links-on-transactions-page
...
change shop name to be link to shop on transactions page
2020-05-21 10:56:19 +01:00
Luis Ramos
35b249f40d
Merge pull request #5390 from rmklaus12/fix-enyml-keys
...
Fix translation keys in en.yml file from previous conflicting PRs
2020-05-21 10:56:01 +01:00
Luis Ramos
a6ea79d3a5
Add warning message to enterprise ship methods list and ship method edit page
2020-05-21 10:47:22 +01:00
Luis Ramos
d017c0b701
Change base translation from applies to active so the naming is consistent: ship methods can be active or inactive for an enterprises which means they can be linked or not to that enterprise
2020-05-21 10:45:58 +01:00
Luis Ramos
cf0ad179a1
Move hubs_sidebar to the last 4 columns at the top of the page
2020-05-21 10:45:58 +01:00
Transifex-Openfoodnetwork
8bb47ef467
Updating translations for config/locales/en_NZ.yml
2020-05-21 16:43:58 +10:00
Maikel
da55956a6e
Merge pull request #5392 from luisramos0/ship_method_bo
...
Make "backoffice only" ship methods work and remove option "frontoffice only"
2020-05-21 10:16:15 +10:00
dependabot-preview[bot]
0e1ae03454
Bump activerecord-import from 1.0.4 to 1.0.5
...
Bumps [activerecord-import](https://github.com/zdennis/activerecord-import ) from 1.0.4 to 1.0.5.
- [Release notes](https://github.com/zdennis/activerecord-import/releases )
- [Changelog](https://github.com/zdennis/activerecord-import/blob/master/CHANGELOG.md )
- [Commits](https://github.com/zdennis/activerecord-import/compare/v1.0.4...v1.0.5 )
Signed-off-by: dependabot-preview[bot] <support@dependabot.com >
2020-05-20 20:02:29 +00:00
Luis Ramos
e287680c1b
Merge pull request #5429 from mkllnk/5400-orders-distributors-report
...
5400 List each order only once in Orders And Distributors report
2020-05-20 20:36:58 +01:00
Matt-Yorkley
6b3a7228ce
Update flash message with instruction to update the quantities
2020-05-20 09:43:29 +02:00
Transifex-Openfoodnetwork
8387c71b6e
Updating translations for config/locales/tr.yml
2020-05-20 15:55:52 +10:00
Robin Klaus
16f2c8d7d1
Removed 5 keys per change request in ref to pr5425
2020-05-20 14:13:42 +10:00
Luis Ramos
76b721b663
Merge pull request #5457 from Matt-Yorkley/cart-fix
...
Fix cart fields sharing same scope
2020-05-19 19:31:49 +01: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
Matt-Yorkley
3df2d4721b
Fix search bar in /shops and /groups
2020-05-19 14:00:20 +02: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
Matt-Yorkley
4a837e8c07
Fix display issue on order cycle form
...
Some of the form elements on the first OC edit page were covered up by the save bar in some cases. this ensures they are always visible and clickable.
2020-05-19 11:56:41 +02:00
Matt-Yorkley
05e6f5792d
Fix cart fields sharing same scope
...
All cart page quantity fields were displaying a single max quantity instead of a different value for each one.
2020-05-19 10:23:12 +02:00
Matt-Yorkley
e54fad9bfd
Add spec for maximum quantities with multiple items in cart
...
This currently fails
2020-05-19 10:21:24 +02: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
Maikel Linke
2d95e9ebc0
List each order only once in report
2020-05-19 10:38:28 +10:00
Maikel Linke
19f7f04816
Add spec for report bug
2020-05-19 10:38:28 +10:00
Maikel Linke
359fd1ac98
Improve readability of report spec
2020-05-19 10:38:28 +10: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
Matt-Yorkley
e3c337ef7e
Fix mixed error display issue
2020-05-18 16:41:57 +02:00
Matt-Yorkley
d70174becc
Use fresh scope for each quantity field
2020-05-18 16:28:07 +02:00
Matt-Yorkley
f9f76f9082
Move reused available stock logic to service
2020-05-18 16:19:49 +02:00
Matt-Yorkley
111e700799
Update invalid input feedback
2020-05-18 16:14:14 +02: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
Matt-Yorkley
f7b4717f29
Improve validation code
2020-05-18 14:51:11 +02:00
Matt-Yorkley
fe27c8466e
Add feature specs for full cart page stock correction process
2020-05-18 14:51:11 +02:00
Matt-Yorkley
63e4430ea4
Update cart page links to be disabled when changes are required and form is unsaved
...
See: https://stackoverflow.com/questions/23425254/enable-disable-anchor-tags-using-angularjs
2020-05-18 14:51:08 +02:00
Matt-Yorkley
5dd801b432
Fix tab-buttons element overlapping order cycle selector
2020-05-18 14:27:05 +02:00
Matt-Yorkley
3a46f32aa3
Merge pull request #5435 from openfoodfoundation/transifex
...
Transifex
2020-05-18 13:44:32 +02:00
Luis Ramos
ea8157a5c6
Merge pull request #5304 from dacook/5299-orders-filter-several-tags
...
5299 Impossible to filter orders with several tags
2020-05-18 11:25:54 +01:00
Luis Ramos
37350fcbb0
Add table name to condition so it doesnt cause trouble in the future if mixed with other tables in the same query
2020-05-18 10:55:03 +01:00
Luis Ramos
05136b51d1
Merge pull request #5356 from rmklaus12/4534-mobile-searchkey
...
Changed search input to inputmode to enable 'go' button on dynamic mo…
2020-05-18 10:30:56 +01:00
Robin Klaus
311bbd2aa2
Changed report_header fields for amount and quantity to reflect correct translation to French
2020-05-18 18:41:04 +10:00
Robin Klaus
a0334953db
Added 14 translation keys for PayPalExpress payment method
2020-05-18 17:06:07 +10:00
Transifex-Openfoodnetwork
d2b8b5e888
Updating translations for config/locales/pt_BR.yml
2020-05-18 05:14:19 +10:00
Luis Ramos
6c81c19f00
Remove dead code
2020-05-16 14:19:25 +01:00
Robin Klaus
438c0ecde9
delete extra lines
2020-05-16 20:12:54 +10:00
Robin Klaus
ba8ecf19ac
Added input code to two additional pages to cover shop tab search boxes
2020-05-16 12:56:06 +10:00
dependabot-preview[bot]
f4afc8ffcd
Bump mini_racer from 0.2.11 to 0.2.14
...
Bumps [mini_racer](https://github.com/discourse/mini_racer ) from 0.2.11 to 0.2.14.
- [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.11...v0.2.14 )
Signed-off-by: dependabot-preview[bot] <support@dependabot.com >
2020-05-15 19:20:08 +00:00
Luis Ramos
ace73be4e2
Add unit tests to order_cart_reset
2020-05-15 20:05:45 +01:00
Luis Ramos
1c749a8029
Extract order_cycle_not_listed? method and rename order_cycles to listed_order_cycles
2020-05-15 19:36:59 +01:00
Luis Ramos
f22eae752d
Adapt spec to validate issue #5340
2020-05-15 19:30:01 +01:00
Luis Ramos
a438317d69
Extract order_cart_reset service from enterprises_controller
2020-05-15 19:30:00 +01:00
Luis Ramos
35824c7aa1
Rename restartCheckout to order_checkout_restart to make it follow service naming convention
2020-05-15 19:29:37 +01:00
Luis Ramos
ba585064e1
Rename ResetOrderService to OrderCompletionReset to follow new service naming convention and also to make it more specific to completed orders
2020-05-15 19:29:37 +01:00
Luis Ramos
21d1a7bc04
Remove dead code
2020-05-15 19:29:37 +01:00
Luis Ramos
6827ce5c7f
Refactor order.set_order_cycle, return early and remove indentation
2020-05-15 19:29:37 +01:00
Luis Ramos
493adc8b1f
Fix problem in spec where wrong enterprise was being used
2020-05-15 19:29:37 +01:00
Luis Ramos
94bb95861b
If selected OC is not in the available OCs, empty the order
2020-05-15 19:29:35 +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
52e7ca2417
Select first default OC only if no OC is already selected
2020-05-15 14:26:56 +01:00
Luis Ramos
c3f0c0ed7f
Extract method to prepare work ahead
2020-05-15 14:25:20 +01:00
Transifex-Openfoodnetwork
4f14d7910a
Updating translations for config/locales/en_IE.yml
2020-05-15 23:04:12 +10:00
Pau Pérez Fabregat
5c7541e63d
Merge pull request #5415 from openfoodfoundation/dependabot/bundler/bugsnag-6.13.1
...
Bump bugsnag from 6.13.0 to 6.13.1
2020-05-15 13:47:46 +02:00
Matt-Yorkley
c2efa901c2
Don't highlight update button when form is invalid
2020-05-15 13:38:17 +02: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
Matt-Yorkley
3ba1182661
Dynamically mark quantity field as invalid when stock is incorrect
2020-05-15 12:59:29 +02:00
Matt-Yorkley
42195b76d3
Highlight update button when cart page form is "dirty"
2020-05-15 12:59:29 +02:00
Matt-Yorkley
73b533690e
Refactor cart update button syntax
2020-05-15 12:59:29 +02:00
Matt-Yorkley
78f736db45
Add red border on quantity field when it must be updated
2020-05-15 12:59:28 +02:00
Luis Ramos
367de87c61
Upgrade jquery-ui-rails to v4.2
2020-05-15 08:58:40 +01:00
Luis Ramos
4fe1135a33
Merge pull request #5376 from rmklaus12/5104-mobile-keyboard-inputs
...
Added inputmode attribute to show appropriate mobile keyboard for sel…
2020-05-15 08:49:42 +01:00
Transifex-Openfoodnetwork
78db7e0e5f
Updating translations for config/locales/en_GB.yml
2020-05-15 17:36:21 +10:00
David Cook
b4228e8cdd
5299 Remove unnecessary comments
2020-05-15 15:11:26 +10:00
David Cook
f5629d59b0
5299: Add trailing square brackets for array parameters (distributor and order_cycle)
2020-05-15 15:04:01 +10:00
David Cook
8a2c65b6bb
Remove duplicate line
2020-05-15 15:04:01 +10:00
David Cook
79df4868de
5299: Unit and feature test for filtering by multiple order cycles
2020-05-15 15:04:01 +10:00
Maikel Linke
51feef83ca
Update translations from Transifex
2020-05-15 14:10:36 +10:00
Maikel
90baec0e2c
Merge pull request #5393 from openfoodfoundation/transifex
...
Transifex
2020-05-15 13:53:56 +10:00
Maikel
d566051e7d
Merge pull request #5432 from openfoodfoundation/dependabot/bundler/mini_racer-0.2.11
...
Bump mini_racer from 0.2.10 to 0.2.11
2020-05-15 11:20:46 +10:00
Maikel
6c409d6c70
Merge pull request #5380 from ckhung5/5201-fix-missing-jquery-ui
...
Update jquery-ui-rails version to 4.1.1
2020-05-15 11:11:36 +10:00
dependabot-preview[bot]
4613502707
Bump bugsnag from 6.13.0 to 6.13.1
...
Bumps [bugsnag](https://github.com/bugsnag/bugsnag-ruby ) from 6.13.0 to 6.13.1.
- [Release notes](https://github.com/bugsnag/bugsnag-ruby/releases )
- [Changelog](https://github.com/bugsnag/bugsnag-ruby/blob/master/CHANGELOG.md )
- [Commits](https://github.com/bugsnag/bugsnag-ruby/compare/v6.13.0...v6.13.1 )
Signed-off-by: dependabot-preview[bot] <support@dependabot.com >
2020-05-14 21:04:00 +00: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
dependabot-preview[bot]
99ab7688b6
Bump mini_racer from 0.2.10 to 0.2.11
...
Bumps [mini_racer](https://github.com/discourse/mini_racer ) from 0.2.10 to 0.2.11.
- [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.10...v0.2.11 )
Signed-off-by: dependabot-preview[bot] <support@dependabot.com >
2020-05-14 20:36:44 +00:00
Luis Ramos
644c65b124
Merge pull request #5430 from coopdevs/make-logger-spec-more-reliable
...
Make JobLogger spec more reliable
2020-05-14 14:33:52 +01:00
Pau Perez
403e1d317d
Make JobLogger spec more reliable
...
This will hopefully fix our bild. I believe that the underlying issue is
that the logger's test double gets leaked into other examples, as RSpec
tells when running `spec/jobs/` specs.
```
5) SubscriptionPlacementJob performing the job when unplaced proxy_orders exist processes placeable proxy_orders
Failure/Error: JobLogger.logger.info("Placing Order for Proxy Order #{proxy_order.id}")
#<InstanceDouble(Logger) (anonymous)> was originally created in one example but has leaked into another example and can no longer be used. rspec-mocks' doubles are designed to only last for one example, and you need to create a new one in each example you wish to use it for.
# ./app/jobs/subscription_placement_job.rb:31:in `place_order_for'
```
Read more: https://relishapp.com/rspec/rspec-mocks/v/3-4/docs/basics/scope#doubles-cannot-be-reused-in-another-example
For whatever reason the JobLogger keeps its `.logger` being stubbed
after this spec.
2020-05-14 13:51:57 +02:00
Transifex-Openfoodnetwork
bd8b3f11d1
Updating translations for config/locales/en_FR.yml
2020-05-14 05:42:38 +10:00
Transifex-Openfoodnetwork
937c981dd8
Updating translations for config/locales/fr.yml
2020-05-14 05:41:55 +10:00
Luis Ramos
19da0ba969
Merge branch 'master' into 3-0-stable-may13
2020-05-13 19:50:32 +01:00
Luis Ramos
5f7c1474a2
Merge pull request #5377 from manvil/1782_enterprise_at_same_address
...
Handling multiple enterprises at the same location
2020-05-13 19:31:14 +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
Transifex-Openfoodnetwork
9b7de5ee6e
Updating translations for config/locales/en_NZ.yml
2020-05-13 19:27:55 +10:00
Luis Ramos
ef2d7f9485
Merge pull request #5330 from openfoodfoundation/mobile-ux-filters
...
[mobile ux] Merge search and filter epic to master
2020-05-13 08:59:32 +01:00
Luis Ramos
58842e8cab
Show order edit link on mobile instead of the cancel link
2020-05-12 19:44:01 +01:00
Matt-Yorkley
598071fa78
Remove conflicting "clear search" buttons added by Chrome
2020-05-12 18:59:15 +02:00
Matt-Yorkley
70e3f0f1f1
Fix closed shop message not displaying full-width
2020-05-12 18:29:48 +02:00
Matt-Yorkley
265a8ed573
Move messages and products form outside of row and columns to enable full-width
2020-05-12 18:24:10 +02: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
41645e8351
Merge pull request #5334 from coopdevs/output-logs-in-djs-log-file
...
Output logs in Delayed Job's log file
2020-05-12 10:05:26 +01:00
Luis Ramos
5f4882fc32
Merge pull request #5403 from openfoodfoundation/dependabot/bundler/ddtrace-0.35.2
...
Bump ddtrace from 0.35.1 to 0.35.2
2020-05-12 08:39:26 +01:00
dependabot-preview[bot]
1114e062b5
Bump ddtrace from 0.35.1 to 0.35.2
...
Bumps [ddtrace](https://github.com/DataDog/dd-trace-rb ) from 0.35.1 to 0.35.2.
- [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.1...v0.35.2 )
Signed-off-by: dependabot-preview[bot] <support@dependabot.com >
2020-05-11 22:07:52 +00:00
Luis Ramos
dcd097a22e
Merge pull request #5231 from pacodelaluna/add-dfc-provider-engine
...
Add DFC Provider engine
2020-05-11 20:27:21 +01:00
Matt-Yorkley
ab143ceaed
Merge pull request #5352 from pacodelaluna/change-date-displayed-on-sales-tax-report
...
Change date displayed on sales_tax reports
2020-05-11 09:56:38 +02:00
Matt-Yorkley
0cd4ad0a1a
Ensure large amount of search text doesn't break layout with clear all button
2020-05-10 09:44:12 +02:00
Matt-Yorkley
9f882a8674
Ensure search results feedback wraps nicely onto next line
2020-05-10 09:44:12 +02:00
Matt-Yorkley
cb9c56d8aa
Ensure correct left-alignment on text in search results feedback
2020-05-10 09:44:12 +02:00
Matt-Yorkley
dee793a392
Change join word between the two filter sets to be "and", to match the actual logic
2020-05-10 09:44:12 +02:00
Matt-Yorkley
7e6683581a
Remove full stop from translation
...
This doesn't fit with the new syntax structure
2020-05-10 09:44:12 +02:00
Matt-Yorkley
b78976fcd7
Ensure consistent styling in both "no results" and "searching for" text
2020-05-10 09:44:12 +02:00
Matt-Yorkley
82b11d6947
Ensure join words are not bold
2020-05-10 09:44:12 +02:00
Matt-Yorkley
a75d3276a4
Include applied filters list in "no results" feedback
2020-05-10 09:44:12 +02:00
Matt-Yorkley
a1435e78c9
Extract applied filters feedback to partial
2020-05-10 09:44:12 +02:00
Matt-Yorkley
7be85d1dab
Use bold font with teal colour for both applied taxons and applied properties
2020-05-10 09:44:12 +02:00
Matt-Yorkley
483a654d31
Add join word "or" when both categories and properties are present
...
Previously for example with "Organic" property and "Fruit" and "Nuts" categories it rendered as: "Fruit or Nuts Organic" instead of: "Fruit or Nuts or Organic"
2020-05-10 09:44:12 +02:00
Matt-Yorkley
f6b910b38a
Fix clear search (x) button position on some layouts
...
Hopefully this will work on iPhones as well...?
2020-05-10 09:44:12 +02:00
Matt-Yorkley
ea06ec7ede
Remove smaller (less than 1em) scaling in search bar on mobile
2020-05-10 09:44:12 +02:00
Matt-Yorkley
07ac7dd9c7
Add explicit 'search' input type to search field.
...
In theory this should improve some of the custom-keyboard functionality added by mobiles, but the implementations will be vary...
2020-05-10 09:44:12 +02:00
Matt-Yorkley
71fb3b1845
Focus search field after it's cleared, ready for immediate typing of new search term on mobile
2020-05-10 09:44:12 +02:00
Matt-Yorkley
e0cb514cad
Add hover state to filters button with darker background
2020-05-10 09:44:12 +02:00
Matt-Yorkley
5ceaff24e9
Darken overlay background colour to reduce distraction
2020-05-10 09:44:11 +02:00
Matt-Yorkley
a7771a20c4
Adjust border-radius on filters button
2020-05-10 09:44:11 +02:00
Matt-Yorkley
03afae01b3
Decrease sidebar width on mobile for more tapping space
2020-05-10 09:44:11 +02:00
Matt-Yorkley
96c6da304a
Increase margin between filter selectors
2020-05-10 09:44:11 +02:00
Matt-Yorkley
2964ec138d
Update display of join words in filters list and rearrange code a bit for clarity
...
Here we need to build a really dynamic structure on the page like "Results for: <search-term> in <filter1> or <filter2>" and the styling on each of the words needs to be a bit different, specifically the filter terms need to be differently styled to the join words like "in" and "or". We have to generate those bits on the Angular side, but they also have to be wrapped in HTML in a specific way.
$sce.trustAsHtml() marks the injected HTML as "safe" (in relation to security issues like XSS). In this case the injected content doesn't involve any user-defined data, so it should be fine.
2020-05-10 09:44:11 +02:00
Matt-Yorkley
14d928c39d
Fix feature specs where UI has changed
2020-05-10 09:44:11 +02:00
Matt-Yorkley
6b66787004
Update "no results" layout on mobile and tablet
2020-05-10 09:44:11 +02:00
Matt-Yorkley
4b695014e4
Update "no results" styling
2020-05-10 09:44:11 +02:00
Matt-Yorkley
c9bfccf465
Update styling on search feedback bar
2020-05-10 09:44:11 +02:00
Matt-Yorkley
026b98022c
Add clear search button in shop searchbar
2020-05-10 09:44:11 +02:00
Matt-Yorkley
ff69389bb0
Make search bar "sticky" for tablets and mobile
2020-05-10 09:44:11 +02:00
Matt-Yorkley
f24969edb1
Remove gutter padding on shop content to gain a bit more horizontal real-estate
2020-05-10 09:44:11 +02:00
Matt-Yorkley
cbb67e15ed
Extract button styles
2020-05-10 09:44:11 +02:00
Matt-Yorkley
865dcf79cb
Extract some properties to variables
2020-05-10 09:44:11 +02:00
Matt-Yorkley
ebe00bdc31
Update filters clear button functionality
2020-05-10 09:44:11 +02:00
Matt-Yorkley
9eaf55f4c0
Add missing translation key
2020-05-10 09:44:11 +02:00
Matt-Yorkley
d1ea2c5f77
Extract shop searchbar styles to separate file
2020-05-10 09:44:11 +02:00
Matt-Yorkley
035bde9eff
Define breakpoints in mixin and refactor
2020-05-10 09:44:11 +02:00
Matt-Yorkley
cbd77b4941
Improve element sizing and position on small screens
2020-05-10 09:44:11 +02:00
Matt-Yorkley
f337fd49e1
Extract search feedback bar into partial and move into main column
2020-05-10 09:44:11 +02:00
Matt-Yorkley
e25e4e7482
Fix issue with button submitting form instead of opening popup
2020-05-10 09:44:11 +02:00
Matt-Yorkley
ff07b97033
Add sidebar footer and buttons
...
Also includes a minor refactor to resolve an issue with animation timings. Angular was not adding the "shown" class to the different elements at the same time in the digest cycle, and it looked a bit shaky.
2020-05-10 09:44:11 +02:00
Matt-Yorkley
94145d0f7e
Improve layout and sizing on mobile
2020-05-10 09:44:11 +02:00
Matt-Yorkley
da1d8e90e2
Update search placeholder
2020-05-10 09:44:11 +02:00
Matt-Yorkley
7c00d41fc4
Add some animation effects
2020-05-10 09:44:11 +02:00
Matt-Yorkley
8e9833fd06
Implement overlaid sidebar on mobile and tablet
2020-05-10 09:44:11 +02:00
Matt-Yorkley
f43bf3880c
Add filters button on mobile and tablet
2020-05-10 09:44:11 +02:00
Matt-Yorkley
142cef3f3a
Add filters count
2020-05-10 09:44:11 +02:00
Matt-Yorkley
0d9f049d24
Add inner icon and restyle search input element
2020-05-10 09:44:11 +02:00
Matt-Yorkley
f46ca0c595
Move search to new searchbar
...
This required a lot of refactoring, as the search needed to be inside both the form element and the Angular ProductsCtrl element, but to get a full-width row for the searchbar it needed to be outside of the 12 column layout of the other shop page elements...
2020-05-10 09:44:11 +02:00
Matt-Yorkley
532e27b7e1
Move filters into new column on desktop
2020-05-10 09:37:56 +02:00
Matt-Yorkley
3ca491683a
Merge pull request #5317 from Matt-Yorkley/caching-timed-shops
...
[Caching] Timed caching on shops data
2020-05-09 15:33:26 +02:00
Transifex-Openfoodnetwork
644d06fb08
Updating translations for config/locales/en_FR.yml
2020-05-09 18:41:29 +10:00
Transifex-Openfoodnetwork
3044e62d8b
Updating translations for config/locales/fr.yml
2020-05-09 18:06:16 +10:00
Pau Pérez Fabregat
b85e439b0e
Merge pull request #5395 from coopdevs/skip-broken-spec
...
Skip unreliable caching test
2020-05-08 17:11:12 +02:00
Pau Perez
fa8c1270ce
Skip unreliable caching test
...
This spec failed two times in a row and it's blocking the release.
2020-05-08 16:37:30 +02:00
Pau Pérez Fabregat
09caf88b6c
Merge pull request #5396 from coopdevs/fix-brittle-test
...
Make test not depend on response's order
2020-05-08 16:37:06 +02:00
Pau Perez
b99bcc3c12
Make test not depend on response's order
...
I'm assuming this is failing in CI due to the order in which the closed
shops are returned.
2020-05-08 15:56:20 +02:00
Transifex-Openfoodnetwork
296a927900
Updating translations for config/locales/en_FR.yml
2020-05-08 22:49:17 +10:00
Transifex-Openfoodnetwork
32ead7b6b7
Updating translations for config/locales/fr.yml
2020-05-08 22:49:02 +10:00
Luis Ramos
0a6bd1424c
Make ship method display on back_end work correcly by making checkout ignore ship methods configured for backoffice only
...
Adding both unit and feature tests as this is important enough for that
2020-05-08 12:59:17 +01:00
Luis Ramos
34d8b1957e
Improve variable names
2020-05-08 12:42:32 +01:00
Luis Ramos
40dd307a35
Remove shipping method display on front_end option, it is not working and it's not straight forward to make it work correctly
2020-05-08 12:00:58 +01:00
Transifex-Openfoodnetwork
4485dcd7fe
Updating translations for config/locales/en_GB.yml
2020-05-08 20:57:56 +10:00
Pau Perez
ad70a63370
Update all locales with the latest Transifex translations
2020-05-08 12:01:42 +02:00
Pau Pérez Fabregat
60470aaf9f
Merge pull request #5332 from coopdevs/add-timestamps-to-oc-schedule
...
Add timestamps to OC schedule join table
2020-05-08 10:51:43 +02:00
Pau Pérez Fabregat
2cf407ce35
Merge pull request #5349 from openfoodfoundation/transifex
...
Transifex
2020-05-08 10:20:07 +02:00
Pau Pérez Fabregat
c38c59b0b9
Merge pull request #5223 from luisramos0/def_user
...
Change default user from spree@example.com to ofn@example.com ❤️
2020-05-08 09:51:31 +02:00
Luis Ramos
424932d61b
Merge pull request #5262 from luisramos0/closed_shop_msg
...
[Mobile] Improve the design of shop page messages (closed shop, private shop and select OC)
2020-05-08 07:52:58 +01:00
Robin Klaus
aa46aec935
Fix translation keys in en.yml file from previous conflicting PRs
2020-05-08 13:21:47 +10:00
Kok Hung Cheah
19d5bcb150
Upgrade jquery-ui-rails to 4.1.2 instead of 4.1.1
2020-05-08 09:54:10 +10:00
Matt-Yorkley
5518ffa856
Ensure validations are called when updating attributes in test
2020-05-07 19:37:43 +02:00
Luis Ramos
dd5041db65
Remove unnecessary default user email and pwd from docker setup
...
These are now the default values in the app
2020-05-07 18:18:42 +01:00
Luis Ramos
052dbca1ba
Merge pull request #5370 from Matt-Yorkley/dont-render-twice
...
Don't render twice
2020-05-07 18:15:47 +01:00
Matt-Yorkley
86bfd1bebb
Add explanatory comments on usages of Timecop.travel
2020-05-07 17:29:58 +02:00
Pau Perez
ee94101edd
Log subs jobs to DJ's log file
...
There's a bunch of logging bits we need to find out what happened in
`#5294`, the first of which is logging as we are used to for Rails
itself. Turns out we were already logging but the log lines weren't
displayed.
That's because workers have no access to the Rails logger unless we hack
things a bit in the config/initializer. If we want to keep DJ's logs in
a different file, which I think is good for debugging purposes, we need
to diverge from Rails.logger.
However, there's no access to Delayed::Worker.logger in custom job
classes either so there's no way other than mimicking that logger's format
if we want log lines to be consistent.
Now the current logs will see the light like:
```
2020-05-07T14:09:42+0000: [Worker(host:8dc5e7e6f434 pid:1)] Job SubscriptionPlacementJob (id=54) RUNNING
2020-05-07T14:09:42+0000: Placing Order for Proxy Order 1
2020-05-07T14:09:42+0000: Issue in Subscription Order 2: complete
2020-05-07T14:09:42+0000: Placing Order for Proxy Order 7
2020-05-07T14:09:42+0000: Issue in Subscription Order 3: complete
2020-05-07T14:09:43+0000: [Worker(host:8dc5e7e6f434 pid:1)] Job SubscriptionPlacementJob (id=54) COMPLETED after 0.3378
2020-05-07T14:09:43+0000: [Worker(host:8dc5e7e6f434 pid:1)] 1 jobs processed at 2.8054 j/s, 0 failed
```
We'll see if we need to format these messages differently to make them
more greppeable.
2020-05-07 17:21:39 +02:00
Luis Ramos
468576ee8a
Merge pull request #5329 from romale/master
...
Correct translations
2020-05-07 16:04:24 +01:00
Luis Ramos
c9b60d0f62
Merge pull request #5341 from mathewdbutton/5173-show-outstanding-order-balance-on-orders-page
...
Show outstanding balance on orders page
2020-05-07 16:03:58 +01:00
Luis Ramos
52b39d04a4
Merge pull request #5359 from mbudm/issue/5108
...
Add NavigationCheck to order cycles edit form
2020-05-07 14:37:32 +01:00
Matt-Yorkley
01d741509f
Fix test of rendered properties AMS
...
Here the displayed properties that we're testing also rely on shops data (including the list of supplied properties), which is now cached for 15 seconds. We clear that cache entry so we can cleanly test only the caching of the properties AMS array.
2020-05-07 15:20:37 +02:00
Manvil George
63a1a6fefa
More optimsations: removing unwanted code
2020-05-07 22:58:09 +10:00
Luis Ramos
66fe26fcc5
Merge pull request #5295 from Matt-Yorkley/shopfront-optimising
...
Enterprise serializer optimisations
2020-05-07 13:08:14 +01:00
Matt-Yorkley
460ab6cdb4
Use #allow instead of #stub
2020-05-07 13:20:30 +02:00
Matt-Yorkley
7457543c2b
Adds spec for timed caching on shops data
2020-05-07 13:20:30 +02:00
Matt-Yorkley
6b6ab864ef
Improve comment on Fragment Caching
2020-05-07 13:20:30 +02:00
Matt-Yorkley
e73c8232de
Improve unit test in cache_service_spec
2020-05-07 13:20:30 +02:00
Matt-Yorkley
37821beb1b
Extract home stats caching to method in service
2020-05-07 13:20:30 +02:00
Matt-Yorkley
2292cbaae4
Extract timed enterprise AMS fragment caching options to service
2020-05-07 13:20:28 +02:00
Luis Ramos
987347d5ab
Move translation keys to new correct location
2020-05-07 12:18:40 +01:00
Luis Ramos
3c7970f72f
Add select oc message when there are multiple OCs
2020-05-07 12:18:40 +01:00
Luis Ramos
d4980893a1
Refactor darker-background directive, extract method
2020-05-07 12:17:43 +01:00
Luis Ramos
0d6d071db2
Remove dead class .shopfront_hidden_message
2020-05-07 12:17:43 +01:00
Luis Ramos
cba9382230
Replace line-height with margin
2020-05-07 12:17:43 +01:00
Luis Ramos
7fcee0fe6d
Fix problem in warning sign
2020-05-07 12:17:43 +01:00
Luis Ramos
f3347a8e16
Make teal links appear only when the darker background is activated
...
This way links in producers, about, etc, where the background is white will still appear as orange, as per design
2020-05-07 12:17:43 +01:00
Luis Ramos
782988b61c
Fix with-darker-background class by moving it inside page-view selector and using opacity instead of just grey so that other elements in the page (shadow from element above) can be seen
2020-05-07 12:17:43 +01:00
Luis Ramos
21a1ba2fb5
Move translation keys according to new location in the customer_required partial
2020-05-07 12:17:43 +01:00
Luis Ramos
ba25986756
Fix some css rubocop issues
2020-05-07 12:17:43 +01:00
Luis Ramos
098ad6f250
Merge partial messages back to home partial where it is only used
2020-05-07 12:17:43 +01:00
Luis Ramos
319657d2c5
Move closed shop partial to /messages
2020-05-07 12:17:43 +01:00
Luis Ramos
eb773a813e
Extract open shop message to partial
2020-05-07 12:17:43 +01:00
Luis Ramos
5dedbc4231
Extract customer_required message to separate partial
2020-05-07 12:17:43 +01:00
Luis Ramos
84c00fe215
Remove duplicate conditionals already present in ShopHelper#shop_tabs, when these partials are rendered, these conditions will always be true
...
In the shops partial, by adding the else, we avoid rendering empty space with an empty
products list in case the shop is closed
2020-05-07 12:17:43 +01:00
Luis Ramos
77d7d2b804
Make the background darker only in specific cases
2020-05-07 12:17:43 +01:00
Luis Ramos
b34fb79d69
Move content div to messages partial so that it's not used in the closed shop message (no margin according to design)
2020-05-07 12:17:43 +01:00
Luis Ramos
1c41061f9e
If shop is closed show closed shop message partial in the home tab
2020-05-07 12:17:43 +01:00
Luis Ramos
b86081090f
Extract closed shop message to a partial
2020-05-07 12:17:43 +01:00
Luis Ramos
dbdbe01d6a
Add new style to closed shop message
2020-05-07 12:17:43 +01:00
Luis Ramos
0996c28c09
Memoize helper methods that are called several times in the views
2020-05-07 12:17:43 +01:00
Luis Ramos
5f70ef6e75
Improve style of closed shop message and remove border from message
2020-05-07 12:17:43 +01:00
Luis Ramos
e0b3e9a040
Remove border of message and paddings, add background color, change link color and split message in two lines
2020-05-07 12:17:43 +01:00
Matt-Yorkley
2876b98155
Use time-based caching on rendered AMS data for shops
2020-05-07 13:11:53 +02:00
Matt-Yorkley
545ca85644
Extract caching of homepage stats to service
2020-05-07 13:11:28 +02:00
Luis Ramos
c64ba75e8d
Merge pull request #5348 from luisramos0/single_oc_selector
...
Replace select box with text when only one OC is available for selection
2020-05-07 12:07:19 +01:00
Luis Ramos
56b9737676
Merge pull request #5343 from luisramos0/windows
...
Make OC selector options color explicitly gray and not white
2020-05-07 11:44:14 +01:00
Luis Ramos
d967d855fc
Merge pull request #5319 from Matt-Yorkley/caching-by-class
...
[Caching] Caching data by class
2020-05-07 11:43:44 +01:00
Luis Ramos
1e9b4516cb
Merge pull request #5318 from Matt-Yorkley/caching-api
...
[Caching] API Action Caching on shop filters
2020-05-07 11:42:52 +01:00
Transifex-Openfoodnetwork
ffebaee9c7
Updating translations for config/locales/tr.yml
2020-05-07 17:34:22 +10:00
Manvil George
a629e1ecb7
Remove incorrect commenting
2020-05-07 11:46:49 +10:00
Manvil George
d7cf825531
Minimised version of the svg
2020-05-07 11:33:35 +10:00
Manvil George
585a754b18
Missed out on inverting the boolean expression
2020-05-07 11:31:27 +10:00
Kok Hung Cheah
edcdae6b58
Update jquery-ui-rails version to 4.1.1
2020-05-07 10:48:07 +10:00
Manvil George
d3504c177e
A better SVG
2020-05-07 10:40:48 +10:00
Manvil George
7123d8347a
XML Formatted The SVG
2020-05-07 09:50:27 +10:00
Manvil George
7847c4ed9e
Removing unwanted blank lines
2020-05-07 09:37:35 +10:00
Manvil George
5b009f606a
More optimizations around coding standards
2020-05-07 09:07:32 +10:00
Luis Ramos
066f978c59
Update ruby version in GETTING_STARTED
2020-05-06 22:19:43 +01:00
François Turbelin
d5b69ec90c
Remove before call when no factorization
2020-05-06 22:04:56 +02:00
François Turbelin
13f00480ce
Use serialized_data
2020-05-06 22:04:56 +02:00
François Turbelin
f6e06b5658
Use default enterprise_id to get the entreprise
2020-05-06 22:04:56 +02:00
François Turbelin
99798b010f
Use ActionController::Base
2020-05-06 22:04:56 +02:00
François Turbelin
43293f00df
Make the endpoint unrelated to enterprise id
...
Better to avoid to have endpoint specific to an enterprise
as we don't know the enterprise id yet on DFC side
2020-05-06 22:04:56 +02:00
François Turbelin
f0f8224934
Put back former lighter inclusions
2020-05-06 22:04:56 +02:00
François Turbelin
013198b33b
Update routes
2020-05-06 22:04:56 +02:00
François Turbelin
830bf796d0
Update authors and remove emails
2020-05-06 22:04:56 +02:00
François Turbelin
dd9c994f9d
Use spree_api_controller_setup layer
2020-05-06 22:04:56 +02:00
François Turbelin
c1d173d601
Add the access token logic, light version
2020-05-06 22:04:56 +02:00
François Turbelin
a8fb059482
Add basic spec
2020-05-06 22:04:56 +02:00
François Turbelin
496174255f
Add REAME.md file
2020-05-06 22:04:56 +02:00
François Turbelin
80bbd5d513
Adjust namespace and scope for routes
2020-05-06 22:04:56 +02:00
François Turbelin
b152e532d2
Cosmetics
2020-05-06 22:04:56 +02:00
François Turbelin
d9d218f661
Add DFC Provider engine
2020-05-06 22:04:56 +02:00
Luis Ramos
d1392d400a
Merge pull request #5350 from luisramos0/edit_order_snail
...
Improve order edit page in data inconsistency scenario (follow up from S2 #4186 )
2020-05-06 20:33:40 +01:00
Luis Ramos
20984ac008
Merge pull request #5379 from openfoodfoundation/dependabot/bundler/ddtrace-0.35.1
...
Bump ddtrace from 0.35.0 to 0.35.1
2020-05-06 20:31:20 +01:00
dependabot-preview[bot]
7f9dfb6142
Bump ddtrace from 0.35.0 to 0.35.1
...
Bumps [ddtrace](https://github.com/DataDog/dd-trace-rb ) from 0.35.0 to 0.35.1.
- [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.0...v0.35.1 )
Signed-off-by: dependabot-preview[bot] <support@dependabot.com >
2020-05-06 19:20:07 +00:00
Transifex-Openfoodnetwork
6c71259b25
Updating translations for config/locales/en_FR.yml
2020-05-07 03:10:34 +10:00
Transifex-Openfoodnetwork
29a54cbfe4
Updating translations for config/locales/fr.yml
2020-05-07 03:10:22 +10:00
Manvil George
bf3bb4fb63
Review Comments incorporation
2020-05-07 00:54:01 +10:00
Transifex-Openfoodnetwork
05820c7ef0
Updating translations for config/locales/tr.yml
2020-05-07 00:14:33 +10:00
Transifex-Openfoodnetwork
492f96c90f
Updating translations for config/locales/tr.yml
2020-05-07 00:11:28 +10:00
Pau Pérez Fabregat
ace3767eb2
Merge pull request #5333 from coopdevs/add-worker-docker-container
...
Create a container to run a DJ worker in dev
2020-05-06 16:11:21 +02:00
Luis Ramos
c7c8bfe212
Merge pull request #5347 from luisramos0/hide_oc_selector_for_private_shop
...
[Mobile] Hide OC selector if shop requires login/customer
2020-05-06 14:31:52 +01:00
Pau Pérez Fabregat
dadc17398f
Merge pull request #5371 from Matt-Yorkley/cart-add-error
...
Cart add error handling
2020-05-06 15:22:06 +02:00
Pau Pérez Fabregat
194c87e805
Merge pull request #5361 from Matt-Yorkley/cart-with-soft-deletion
...
Cart with soft-deletion
2020-05-06 15:20:35 +02:00
Manvil George
6c51841039
Reverting unrelated changes
2020-05-06 23:05:34 +10: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
Manvil George
d1e03909ec
Minor CSS alterations to make the button visible at the bottom
2020-05-06 22:30:33 +10:00
Pau Pérez Fabregat
ead21a97cc
Merge pull request #5327 from openfoodfoundation/luisramos0-patch-2
...
Add link to docker tips wiki page to Docker.md
2020-05-06 14:10:37 +02:00
Pau Pérez Fabregat
a0012e1c5b
Merge pull request #5326 from openfoodfoundation/luisramos0-patch-1-1
...
Add link to Docker.md from GETTING_STARTED.md
2020-05-06 12:31:00 +02:00
Matt-Yorkley
9c24c1cd05
Ensure #caches_action works in test suite
2020-05-06 12:24:33 +02:00
Matt-Yorkley
4aac97c985
Add feature spec for action caching of taxon and properties endpoints
2020-05-06 11:47:51 +02:00
Matt-Yorkley
7e4c00ba3f
Ensure caching works in cache tests for API controllers
2020-05-06 11:47:45 +02:00
Pau Pérez Fabregat
65433c6ac6
Merge pull request #5351 from luisramos0/improve_sample_data
...
Improve ship methods names on sample data
2020-05-06 11:27:34 +02:00
Pau Pérez Fabregat
b3205c5459
Merge pull request #5309 from luisramos0/better-karma
...
Make dockerfile install node and adapt karma browser config to work inside a docker container
2020-05-06 11:11:00 +02:00
Transifex-Openfoodnetwork
d52db51b29
Updating translations for config/locales/nb.yml
2020-05-06 17:25:07 +10:00
Mathew Button
e088b27a13
Always display the outstanding balance if not zero
2020-05-06 16:45:44 +10:00
Manvil George
f802775ad6
Fixing up old Enterprise Modal references
2020-05-06 15:37:28 +10:00
Manvil George
e9cba32f31
Appeasing the Codeclimate Workers
2020-05-06 13:13:35 +10:00
Manvil George
19b5f6a562
Handling multiple enterprises at the same location
...
Changes:
* Introduced a cluster marker to denote multiple points of interest at
the same location
* Seperated out a plain enterprise modal into 2 parts
* A modal called EnterpriseModal for showing a list of enterprises at
the same location
* A box called EnterpriseBox(which by the way is also a technically a
modal) that shows the details of that particular enterprise selected
* If at a location there exists only a single enterprise then only the
box is shown
2020-05-06 12:54:42 +10:00
Robin Klaus
f43380fce5
Added inputmode attribute to show appropriate mobile keyboard for selected field
2020-05-06 12:52:53 +10:00
Robin Klaus
7dce53a77b
Added inputmode attribute to show appropriate mobile keyboard for selected field
2020-05-06 12:46:37 +10:00
Steve Roberts
4573127c59
Add NavigationCheck to order cycles edit form
...
- Simplify if statements with safe navigation operator
- Simplify order cycle nav check spec
- Rename nav check callback so a comment is not needed and remove unnecessary assignation to $scope
2020-05-06 10:43:21 +10: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
Matt-Yorkley
61d7adaf74
Enable optional caching tag in test metadata
2020-05-05 18:43:09 +02:00
Matt-Yorkley
bdc2d002fa
Update karma tests
2020-05-05 17:22:12 +02:00
Matt-Yorkley
13e4411b89
Add error message on cart populate failure and don't keep re-sending the data to the server.
...
This was triggering many requests to be continuously re-sent in a very problematic way (server-melting), and with no user feedback.
2020-05-05 17:22:10 +02:00
Luis Ramos
c2a9a698f4
Merge pull request #5312 from openfoodfoundation/dependabot/bundler/unicorn-5.5.5
...
Bump unicorn from 5.5.4 to 5.5.5
2020-05-05 15:24:59 +01:00
Matt-Yorkley
9a7807b134
Don't continuously re-post data on cart error
2020-05-05 16:04:05 +02:00
Luis Ramos
24414b83dd
Merge pull request #5325 from oeoeaio/enhancement-5214
...
Add order number and date/time to order cycles customer report
2020-05-05 14:34:47 +01:00
Matt-Yorkley
4f4b3fb206
Don't render twice 🙈
2020-05-05 15:21:05 +02:00
Matt-Yorkley
6afda87baf
Add explanatory comment on soft-deleted variant stock logic
2020-05-05 13:57:50 +02:00
Matt-Yorkley
0a28abbf2d
Add additional feature specs for soft-deleted variants in cart
2020-05-05 13:57:50 +02:00
Matt-Yorkley
26ba76cff9
Fix soft-deletion in CartService and update spec
2020-05-05 13:57:50 +02:00
Matt-Yorkley
a3458aa562
Ensure VariantStockLevels can process soft-deleted variants
2020-05-05 13:57:50 +02:00
Matt-Yorkley
0e429da377
Return zero available stock (total_on_hand) for variants that are soft-deleted
2020-05-05 13:57:50 +02:00
Matt-Yorkley
eb51b87bea
Add spec for current soft deletion behaviour in OrderCycle#variants_distributed_by scope
2020-05-05 13:57:50 +02:00
Matt-Yorkley
d3de1ce47e
Add spec for current soft deletion behaviour in VariantOverride#indexed
2020-05-05 13:57:50 +02:00
Matt-Yorkley
5639b21c77
Add tests for current soft-deleted variant behavior in CartService
2020-05-05 13:57:50 +02:00
Matt-Yorkley
068c8feea7
Merge pull request #5289 from Matt-Yorkley/cart-populate
...
Cart populate
2020-05-05 13:56:17 +02:00
François Turbelin
6e5cbebf4a
Change date value on order_and_distributor_report
2020-05-05 12:27:17 +02:00
François Turbelin
9cfd7db99d
Change date displayed on sales_tax reports
2020-05-05 12:27:17 +02:00
Luis Ramos
bcfa8982ef
Merge pull request #5364 from openfoodfoundation/dependabot/bundler/ddtrace-0.35.0
...
Bump ddtrace from 0.34.2 to 0.35.0
2020-05-05 10:58:13 +01:00
Matt-Yorkley
805f91e838
Remove unnecessary #all call
2020-05-05 09:56:08 +02:00
Matt-Yorkley
a3757992b5
Use #tap to destroy and remove before block
2020-05-05 09:55:14 +02:00
Steve Roberts
3b8acdc577
Split order_cycle_spec into smaller specs grouped by theme
2020-05-05 15:39:45 +10:00
dependabot-preview[bot]
7180af2736
Bump ddtrace from 0.34.2 to 0.35.0
...
Bumps [ddtrace](https://github.com/DataDog/dd-trace-rb ) from 0.34.2 to 0.35.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.34.2...v0.35.0 )
Signed-off-by: dependabot-preview[bot] <support@dependabot.com >
2020-05-04 22:46:59 +00:00
Eduardo
2fca16f02f
change shop name to be link to shop on transactions page
2020-05-04 07:06:07 -03:00
Mathew Button
fe6c810505
Add brackets in view instead of API response
2020-05-04 19:09:18 +10:00
Luis Ramos
44872844ed
Make ship methods in sample data have the name of the enterprise so it's easier to distinguish them
2020-05-03 16:36:12 +01:00
Luis Ramos
bf0b941e1c
Merge pull request #4935 from coopdevs/data-archiving
...
Allow data archiving using the :truncate_data task
2020-05-03 12:58:31 +01:00
Luis Ramos
e2b26b2e9c
Replace break with if
...
What we would need would be a continue, not break, so that other units will be processed without halting the cycle through the units
2020-05-03 12:00:23 +01:00
Luis Ramos
92d0a5b735
Change default user from spree@example.com to ofn@example.com
2020-05-03 10:12:43 +01:00
Luis Ramos
8e27ace563
Adapt specs where there is only one OC available to the absence of the OC selector
2020-05-02 20:16:41 +01:00
Transifex-Openfoodnetwork
93881a742e
Updating translations for config/locales/en_GB.yml
2020-05-03 05:09:16 +10:00
Luis Ramos
b92e257e44
Fix codeclimate issue
2020-05-02 19:12:47 +01:00
Luis Ramos
fa908d4024
Extract change_order_cycle partial from main enterprise shop view
2020-05-02 19:01:36 +01:00
Luis Ramos
7fd14cf29b
Adjust line height of select label and select itself
2020-05-02 19:01:36 +01:00
Luis Ramos
ef0e9dc0ca
Replace select box with text when only one OC is available for selection
2020-05-02 19:01:33 +01:00
Luis Ramos
89cb447387
Hide OC selector if shop requires login/customer
2020-05-02 17:56:49 +01:00
Luis Ramos
b568d25446
Update all locales with the latest Transifex translations
2020-05-02 17:08:53 +01:00
Luis Ramos
303464a04e
Merge pull request #5297 from openfoodfoundation/transifex
...
Transifex
2020-05-02 15:40:54 +01:00
Luis Ramos
0662c57d9d
Merge pull request #5344 from luisramos0/debug_assets
...
Move PROFILE debug assets flag to another ENV var
2020-05-02 14:21:15 +01:00
Luis Ramos
acb7e9751b
Move debug assets flag to another env var so we go back to having both profile and debug assets disabled by default
2020-05-02 10:42:49 +01:00
Luis Ramos
9ae3ad9279
Move rule on option out of the select rule to fix rubocop issue
2020-05-01 17:43:52 +01:00
Luis Ramos
240e2a36d3
Make option color explicitly gray and not white otherwise we will have options render white on white on some browsers, namely chrome on windows 10
2020-05-01 17:39:12 +01:00
Mathew Button
8087e6b31b
Show outstanding balance on orders page
...
Prior to this change, there was no way of knowing how much was owed (or
in need of refunding) at a glance for each order on the orders index page,
this information was only available by clicking through to an order
payment page
This change adds the outstanding balance for each order
2020-05-01 21:38:20 +10:00
Transifex-Openfoodnetwork
39875308c2
Updating translations for config/locales/en_FR.yml
2020-05-01 04:55:35 +10:00
Transifex-Openfoodnetwork
c424e7b65e
Updating translations for config/locales/fr.yml
2020-05-01 04:55:21 +10:00
Pau Perez
9d253e1e3e
Create a container to run a DJ worker in dev
2020-04-30 17:52:22 +02:00
Matt-Yorkley
1990417b72
Test values are not fetched from database when cache exists
2020-04-30 17:37:48 +02:00
Matt-Yorkley
7a22f7f783
Move and rename caching spec
2020-04-30 17:37:45 +02:00
Luis Ramos
74c7a01151
Merge pull request #5259 from luchiago/add-extra-social-media-to-emails
...
Add extra social media to order confirmation email
2020-04-30 15:01:09 +01:00
Matt-Yorkley
f724d1b572
Add feature spec for caching properties and taxons AMS
2020-04-30 15:35:39 +02:00
Luis Ramos
8845260979
Merge pull request #5285 from rmklaus12/5246-missing-translation-orders-list
...
5246 missing translation orders list
2020-04-30 14:00:26 +01:00
Pau Perez
1903134e13
Add timestamps to OC schedule join table
...
This is critical to debug bugs related to subscriptions.
Essentially, `has_and_belongs_to_many` doesn't give us the option for
any other column that the foreign keys themselves:
> A has_and_belongs_to_many association creates a direct many-to-many
> connection with another model, with no intervening model.
Source: https://guides.rubyonrails.org/v3.2/association_basics.html#the-has_and_belongs_to_many-association
Note however, that there's no way to update an order_cycle_schedule,
that I can think of but `updated_at` doesn't do any harm.
2020-04-30 15:00:13 +02:00
Pau Pérez Fabregat
4174ea69a1
Merge pull request #5247 from chrishil1/Placeholders_variant
...
Added placeholders in variant edit page
2020-04-30 12:05:59 +02:00
Pau Pérez Fabregat
676add18c3
Merge pull request #5322 from luisramos0/dead_content
...
Delete some dead views
2020-04-30 12:02:14 +02:00
Matt-Yorkley
a35f8cdb02
Move cache keys used in views into the cache service
2020-04-30 09:45:15 +02:00
Luis Ramos
8a107bee98
Merge pull request #5235 from luisramos0/mail_methods
...
Remove smtp config from admin config page
2020-04-29 19:10:02 +01:00
Matt-Yorkley
210029aaff
Enable optional caching tag in test metadata
2020-04-29 20:04:54 +02:00
Transifex-Openfoodnetwork
5bc5ef9a9d
Updating translations for config/locales/pt_BR.yml
2020-04-30 03:52:57 +10:00
Matt-Yorkley
f2cd122ec8
Update variant_stock_spec for unsaved and soft-deleted cases
2020-04-29 18:29:44 +02:00
Matt-Yorkley
4054bdd722
Add Bugsnag call for "variant with no stock item" case
2020-04-29 17:34:48 +02: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
romale
3a4ef2697c
Correct translations
...
Without it, some labels not translated to other language.
And, please check code:
en.admin.subscription.subscriptions - does not exists
but
en.admin.subscriptions.subscriptions - is exist
2020-04-29 18:27:09 +03:00
Matt-Yorkley
2bdda7de04
Change order of methods for easier reading
2020-04-29 17:22:46 +02:00
Matt-Yorkley
92a881c584
Simplify relation used in #exchanges_supplying
2020-04-29 17:19:16 +02:00
Pau Perez
647a7bdddf
Clarify we won't use truncate_date in prod yet
...
We initially aimed at implementing data archiving in production reusing
this rake task but priorities have changed. It'll be just a refactor for
now.
2020-04-29 17:00:53 +02:00
Pau Perez
e0228f66af
Default to archiving data older than 2 years.
...
This is safer than the current 3 months.
2020-04-29 16:49:43 +02:00
Pau Perez
38ea95ea85
Prevent nil input to turn into 0
2020-04-29 16:49:43 +02:00
Pau Perez
6ceeda7d9e
Instrument TruncateData logging start and end
2020-04-29 16:49:43 +02:00
Pau Perez
703706ee0b
Replace one-letter variable with full word
2020-04-29 16:49:43 +02:00
Pau Perez
e2a3dd0c6f
Delete only return auths. of the deleted orders
...
They are associated to order and as such we can't remove them all
blindly.
2020-04-29 16:49:43 +02:00
Pau Perez
a3b8638faf
Delete sessions older than two weeks
...
This affects users that are actively purchasing, so 2 weeks data is more
than enough. Others can afford to log in again.
2020-04-29 16:49:43 +02:00
Pau Perez
4f015320a3
Upper case DELETE statement
...
This keeps it consistent with the rest of the log.
2020-04-29 16:49:43 +02:00
Pau Perez
0f1d57db73
Delete LogEntries older than a month
...
They are useful for troubleshooting but a month data seems enough.
2020-04-29 16:49:43 +02:00
Pau Perez
5f84c51c13
Delete StateChanges older than a month
...
They are useful for troubleshooting but a month data seems enough.
2020-04-29 16:49:43 +02:00
Pau Perez
d215c76bc9
Make it even more explicit the action is dangerous
...
We're yelling at the person to make a backup before proceeding.
2020-04-29 16:49:43 +02:00
Pau Perez
e1a80edb7e
Carefully doc how to archive data from an instance
2020-04-29 16:49:43 +02:00
Pau Perez
a4372e4d31
Fix long lines
2020-04-29 16:49:43 +02:00
Pau Perez
b6d3c3039a
Fix "Method has too many lines" violation
2020-04-29 16:49:43 +02:00
Pau Perez
f199cb1bea
Warn but allow executing :truncate_data in prod
2020-04-29 16:49:43 +02:00
Pau Perez
be123b2a72
Specify how much data to remove in :truncate_data
2020-04-29 16:49:43 +02:00
Pau Perez
60d29d619f
Replace :truncate_data definition with new class
...
And cover it with a test.
2020-04-29 16:49:43 +02:00
Pau Perez
78fd785f0c
Extract TruncateData out of :truncate_data task
2020-04-29 16:49:43 +02:00
Pau Pérez Fabregat
f9f4bdae8f
Merge pull request #5238 from Matt-Yorkley/moar-performance
...
Improve Pain Points
2020-04-29 16:36:15 +02:00
Luis Ramos
0f16c43f21
Add link to docker tips wiki page to Docker.md
2020-04-29 14:16:59 +01:00
Luis Ramos
865024fc1f
Add link to the docker setup guide to the getting started guide.
2020-04-29 14:05:45 +01:00
Luis Ramos
407d890d23
Merge pull request #5315 from Matt-Yorkley/ordercyles-controller-tweaks
...
OrderCylesController tweaks
2020-04-29 13:50:10 +01:00
Luis Ramos
0e5d7c1eb1
Add migration to drop dead spree_mail_methods table and some dead mail_methods preferences
2020-04-29 13:34:50 +01:00
Luis Ramos
a38b18bd0d
Remove smtp config from admin config page
2020-04-29 13:34:19 +01:00
Rob H
6405c34428
Add order number and date/time to order cycles customer report
2020-04-29 21:20:14 +10:00
Pau Pérez Fabregat
e7df9bb58d
Merge pull request #5178 from coopdevs/dont-populate-empty-cart
...
Dont populate empty cart
2020-04-29 13:08:26 +02:00
Matt-Yorkley
d3af3d3f27
Avoid extra query on stock_items every time #on_demand is called on a variant.
...
In the case where the variant has not been saved yet, we can use #new_record? here instead of #stock_items.empty?, to avoid an additional query. This can be called a vast number of times per request, in various N+1s. The other case where we need to return here is when a variant has been deleted, so #stock_items will be empty and #stock_item will be nil. Likewise, we can just check that with #deleted? and avoid #stock_items.empty?
2020-04-28 20:54:04 +02:00
Matt-Yorkley
9682544442
Eager-load and update BasicEnterpriseSerializer
2020-04-28 20:10:40 +02:00
Matt-Yorkley
f4bcf6c6d5
Eager-load line_item associations in order
2020-04-28 20:10:40 +02:00
Luis Ramos
e6fa08edfc
Delete some dead views
2020-04-28 19:07:19 +01: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
Transifex-Openfoodnetwork
18fc4b7c92
Updating translations for config/locales/ca.yml
2020-04-29 01:13:42 +10:00
Transifex-Openfoodnetwork
612cc45ab7
Updating translations for config/locales/en_FR.yml
2020-04-29 01:13:35 +10:00
Transifex-Openfoodnetwork
16e289bf37
Updating translations for config/locales/en_GB.yml
2020-04-29 01:13:29 +10:00
Transifex-Openfoodnetwork
b54c6fcb26
Updating translations for config/locales/nb.yml
2020-04-29 01:13:26 +10:00
Transifex-Openfoodnetwork
8365c66add
Updating translations for config/locales/fil_PH.yml
2020-04-29 01:13:22 +10:00
Transifex-Openfoodnetwork
5590671c23
Updating translations for config/locales/fr.yml
2020-04-29 01:13:19 +10:00
Luis Ramos
9e43661127
Merge pull request #5153 from luisramos0/bye_bye_ga
...
Bye bye Google Analytics
2020-04-28 16:09:16 +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
Matt-Yorkley
ec581dccb8
Use class-based caching for queries in EnterpriseInjectionData
2020-04-28 15:11:59 +02:00
Matt-Yorkley
4d098448f5
Cache rendered AMS arrays in darkswarm layout based on latest timestamps of rendered object classes
2020-04-28 15:11:59 +02:00
Matt-Yorkley
53ebe10483
Fix issue with generic primary taxon in product factory changing which taxons are correctly counted as being in open order cycles.
2020-04-28 15:11:59 +02:00
Matt-Yorkley
eb5f8b85ff
Touch primary taxon when a product's primary_taxon is changed
2020-04-28 15:11:59 +02:00
Matt-Yorkley
1b18808d21
Touch ShippingMethod when it's assigned to a new distributor
2020-04-28 15:11:59 +02:00
Matt-Yorkley
6ef345c5d8
Touch taxon when a taxon is applied to a product
2020-04-28 15:11:59 +02:00
Matt-Yorkley
a25a75bbe8
Ensure action caching includes GET params
...
By default the auto-generated action cache keys looks like this:
`views/0.0.0.0:3000/api/order_cycles/1/properties`
With this change the cache keys now look like this:
`views/0.0.0.0:3000/api/order_cycles/1/properties?distributor=3`
2020-04-28 15:11:14 +02:00
Matt-Yorkley
ea1ec1a1c6
Use ActionCaching in OrderCyclesController (taxons and properties)
2020-04-28 15:11:12 +02:00
Matt-Yorkley
975afb3152
Enable use of Action Caching in the API 🎉
...
See: https://guides.rubyonrails.org/api_app.html#adding-other-modules
2020-04-28 15:11:00 +02:00
Luis Ramos
839b373b5c
Merge branch 'master' into 3-0-stable-Apr28
2020-04-28 13:40:19 +01:00
Kristina Lim
265871932f
Merge pull request #5146 from coopdevs/dev-mode-profiling-settings
...
Add PROFILE var to set production-like settings
2020-04-28 20:31:13 +08:00
Kristina Lim
c34570e96e
Merge pull request #5313 from kristinalim/fix/mirror_db_script_bucket
...
Fix script for syncing public/ in AWS bucket to local
2020-04-28 20:28:05 +08:00
Matt-Yorkley
ac166f3590
Add CacheService
2020-04-28 14:15:16 +02:00
Matt-Yorkley
e4985a9d51
Avoid needlessly fetching the current user records (for authentication and API key checks)
...
These endpoints are absolutely public, and don't need the current user at any point.
2020-04-28 14:06:02 +02:00
Matt-Yorkley
eb7de18298
Return early (before hitting the DB) in complex product list rendering if we already know the order cycle is closed
2020-04-28 13:51:25 +02:00
Matt-Yorkley
67a7140642
Memoize distributor and order_cycle in Api::OrderCyclesController
2020-04-28 13:48:07 +02:00
Luis Ramos
868929eed3
Merge pull request #4993 from luisramos0/move_ent_fees
...
Change engine routes to better work with specs in rails 4
2020-04-28 11:48:41 +01:00
Transifex-Openfoodnetwork
8e6d53f6c6
Updating translations for config/locales/nb.yml
2020-04-28 17:59:10 +10:00
Robin Klaus
f072e9d9c2
Add missing translations on order list page
2020-04-28 16:47:39 +10:00
Kristina Lim
7c985f39ab
Fix script for syncing public/ in AWS bucket to local
2020-04-28 13:22:48 +08:00
Kristina Lim
efb83c2f95
Merge pull request #5250 from mkllnk/mirror-db-script
...
Add cache to mirror_db script
2020-04-28 13:18:29 +08:00
Matt-Yorkley
74e81b078f
Avoid N+1s in Order#cap_quantity_at_stock!
2020-04-28 01:11:28 +02:00
Matt-Yorkley
3fa2b3161f
Avoid N+1s when using OrderCycle#exchanges_supplying
2020-04-28 01:10:07 +02:00
Matt-Yorkley
fe2bf8d531
Eager-load variants and stock items when doing stock checks on multiple variants
2020-04-28 01:05:42 +02:00
Matt-Yorkley
eb858159ce
Eager-load :default_price on variants
2020-04-28 01:04:22 +02:00
Matt-Yorkley
43869fc140
Replace fired events with clearer method call
...
The 'spree.cart.add' event has no listeners in spree_core on in ofn. The 'spree.order.contents_changed' just has a single listener that calls `order.update_distribution_charge`.
2020-04-28 00:56:18 +02:00
Matt-Yorkley
a5c4364f92
Fetch (or create) current_order only once
2020-04-28 00:52:09 +02:00
dependabot-preview[bot]
02775d033d
Bump unicorn from 5.5.4 to 5.5.5
...
Bumps [unicorn](https://yhbt.net/unicorn/ ) from 5.5.4 to 5.5.5.
Signed-off-by: dependabot-preview[bot] <support@dependabot.com >
2020-04-27 20:50:56 +00:00
Luis Ramos
6559b4d3a4
Make dockerfile install node and adapt karma browser config to work inside a docker container
2020-04-27 19:05:11 +01:00
Luis Ramos
d851aa5106
Add model definition to migration to make migration more resilient
2020-04-27 18:40:56 +01:00
Luis Ramos
c5b9727177
Remove ga_cookies_preference from DB
2020-04-27 18:40:56 +01:00
Luis Ramos
248b0016d4
Remove trackers and google analytics
2020-04-27 18:40:33 +01:00
Luis Ramos
d8d6bad11c
Remove config and sections related to google analytics in the cookies banner and cookies page
2020-04-27 18:40:00 +01:00
Luis Ramos
ab16931d70
Merge pull request #5257 from luisramos0/stripe_sca_payments_last
...
StripeSCA - reuse better method to fetch last payment of an order to avoid nasty bugs in the future
2020-04-27 18:34:35 +01:00
Matt-Yorkley
e33de8a20e
Update specs and refactor a bit
2020-04-27 19:27:07 +02:00
Matt-Yorkley
468cb3f57e
Remove obviously private methods from the public interface in CartService
...
These methods are not called from anywhere in the app, only in a couple of tests in cart_service_spec.
2020-04-27 19:27:07 +02:00
Matt-Yorkley
1152f307e2
Eager-load associated line_items data in #update_distribution_charge!
2020-04-27 19:27:07 +02:00
Matt-Yorkley
25525d4f75
Use guard clause in each block
2020-04-27 19:27:07 +02:00
Matt-Yorkley
a759d8c7c7
Avoid N+1s for variants and for line_items of variants
2020-04-27 19:27:07 +02:00
Matt-Yorkley
b3242041e5
Return earlier (before scoping process, if possible)
2020-04-27 19:27:07 +02:00
Matt-Yorkley
106bb7a27f
Rename some variables and add comments for clarity in cart_service
2020-04-27 19:27:07 +02:00
Matt-Yorkley
2334ab6d00
Delete some dead code in cart_service
...
This Bugsnag error was added a year ago and hasn't been seen
2020-04-27 19:27:07 +02:00
Matt-Yorkley
11af5dffdc
Memoize scoper in cart_service
...
Avoids fetching all of the hub's variant overrides from the db every time it's initialized.
2020-04-27 19:27:07 +02:00
Matt-Yorkley
3ce7e96777
Add some debounce and an onwheel hack to product add to basket field
...
Debounce ensures we don't get a million requests if the up/down buttons are clicked rapidly. The onwheel hack adds some protection against scrolling triggering the quantity up/down. See: https://stackoverflow.com/a/51076231
2020-04-27 19:27:07 +02:00
Transifex-Openfoodnetwork
670fff9d36
Updating translations for config/locales/es_CR.yml
2020-04-28 02:38:07 +10:00
Transifex-Openfoodnetwork
171ba09663
Updating translations for config/locales/fr.yml
2020-04-28 01:40:44 +10:00
Transifex-Openfoodnetwork
65c9c287e8
Updating translations for config/locales/en_FR.yml
2020-04-28 01:37:49 +10:00
Transifex-Openfoodnetwork
4332734f27
Updating translations for config/locales/ca.yml
2020-04-28 01:37:44 +10:00
Transifex-Openfoodnetwork
340f669506
Updating translations for config/locales/en_GB.yml
2020-04-28 01:37:36 +10:00
Transifex-Openfoodnetwork
3b7ad0ef4e
Updating translations for config/locales/ca.yml
2020-04-27 23:49:48 +10:00
Luis Ramos
6e23f5bdac
Merge pull request #5233 from luisramos0/stripe_sca_bo
...
[StripeSCA] Fix card payments in the Backoffice
2020-04-27 13:00:24 +01:00
Luis Ramos
8b5378e673
Merge pull request #5251 from luisramos0/Issue4654
...
Change Result of PriceSack Calculation from Integers to Floats, clone #4812
2020-04-27 12:36:18 +01:00
Transifex-Openfoodnetwork
ba32e2d676
Updating translations for config/locales/fil_PH.yml
2020-04-27 18:08:38 +10:00
Pau Pérez Fabregat
74c4722cfe
Merge pull request #5286 from rmklaus12/4966-missing-translation-admin-prod-list
...
4966 missing translation admin prod list
2020-04-27 09:28:57 +02:00
Luis Ramos
b3eda9fecb
Merge pull request #5256 from dacook/5190-fix-order-sorting
...
Fix sorting of orders list
2020-04-26 14:24:00 +01:00
Pau Perez
c224df9b6a
Do not trigger an orderChanged with null quantity
...
When loading the page $watchGroup calls the listener function for every
listed line item but with a set variant and null quantity and
max_quantity. There's no point on computing an order change when there
was none.
This saves an empty request on the second most used endpoint of the app,
specially busy when users are placing orders.
2020-04-24 15:54:46 +02:00
Luis Ramos
b9c86d54b0
Merge pull request #5261 from luisramos0/search-iphone
...
[Mobile] Make search box font size be 16px so that no zoom happens on iphone
2020-04-24 11:30:26 +01:00
Transifex-Openfoodnetwork
cebba4dd43
Updating translations for config/locales/en_GB.yml
2020-04-24 18:22:49 +10:00
Matt-Yorkley
43ba73ac19
Avoid expensive queries for supplied product properties if enterprise is not a supplier
2020-04-23 21:51:39 +02:00
Matt-Yorkley
728326c2a5
Eager-load :properties on supplied products in enterprise_shopfront_serializer
2020-04-23 21:48:41 +02:00
Transifex-Openfoodnetwork
8bcc9456d8
Updating translations for config/locales/en_FR.yml
2020-04-24 04:30:04 +10:00
Transifex-Openfoodnetwork
5f51b21fe9
Updating translations for config/locales/fr.yml
2020-04-24 04:29:52 +10:00
Matt-Yorkley
69cf7dff2c
Memoize :active in enterprise serializers that call it multiple times
2020-04-23 13:17:44 +02:00
Luis Ramos
568e570b4b
Move method to more generic helper to use it in mailers
2020-04-23 16:08:10 +10:00
Luis Ramos
42be6c905f
Added spec for OrderPaymentFinder
2020-04-23 16:08:10 +10:00
Luis Ramos
d482cccefe
Use OrderPaymentFinder methods instead of payments.last and payments.pending.last
2020-04-23 16:08:10 +10:00
Luis Ramos
52becf6abc
Extract payment method logic from OrderPaymentFinder into the orders helper
2020-04-23 16:08:10 +10:00
Robin Klaus
d8f4df4bcc
Fixed missing translation, added keys to en.yml file
2020-04-23 09:58:47 +10:00
Pau Perez
2d9b41729e
Use memory cache-store when profiling
...
Or you want see any change when playing fragment-caching or other caching
strategies.
2020-04-22 17:56:56 +02:00
Pau Perez
de8d8e658c
Add PROFILE var to set production-like settings
...
I took this from a recent newsletter I read. Sometimes replication
performance issues locally is actually slower than production due to dev
mode settings (code reloading, etc.), heavy de-only gems and the asset
pipeline.
The PROFILE env var switches these settings all at the same time,
giving us an environment closer to production, essential for reliable
profiling. Then, rack-mini-profiler is going to be more accurate.
Apparently it's something
[RubyGems](b026df86ae/config/environments/development.rb (L72-L92) )
and
[CodeTriage](a3c957647d )
both use.
2020-04-22 17:53:17 +02:00
Matt-Yorkley
53183b8598
Eager-load properties in inject_enterprise_and_relatives
2020-04-21 11:13:20 +02:00
Luis Ramos
2ab07bc6a9
Make search box font size be 16px so that no zoom happens on iphone
2020-04-18 18:50:25 +01:00
Lucas Hiago
acfe0c540a
Create instagram css property and apply
2020-04-18 10:06:14 -03:00
Lucas Hiago
ac2ab34e11
Add connect with us footer to customer order confirmation email
2020-04-18 10:06:06 -03:00
Lucas Hiago
41c0204cfa
Update instagram and linkedin links
2020-04-18 10:05:59 -03:00
Luis Ramos
910cc99c2f
Add spec to cover orders_controller watch sortOptions
2020-04-18 10:23:02 +01:00
David Cook
a0a361673a
Fix sorting of orders
...
(wrong copied function call)
2020-04-18 15:03:39 +10:00
jeffrey s hill md
ed91c179cd
Changed translation paths
2020-04-17 12:04:27 -05:00
Luis Ramos
9147518422
Remove some unnecessary code
2020-04-17 14:51:06 +01:00
Luis Ramos
d23397f250
Move float test to a separate context
2020-04-17 14:49:09 +01:00
blainebillings
993a684e44
Add Price Sack Spec for Float Amounts
2020-04-17 14:43:28 +01:00
blainebillings
427dc54945
Change Result of PriceSack Calculation from Integers to Floats
2020-04-17 14:43:28 +01:00
Maikel Linke
b5d159e163
Add cache to mirror_db script
...
Also added some better error handling around image syncing.
2020-04-17 16:09:22 +10:00
jeffrey s hill md
d8444dcf3c
Added placeholders
2020-04-16 14:59:41 -05: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
fcc746a1b7
Fix long lines in payments_controller_spec
2020-04-15 16:18:17 +01:00
Luis Ramos
8f7b3df9b5
Make payment controller authorize stripe_sca payments before processing them or advancing order workflow (that also calls payment.process)
2020-04-15 16:07:34 +01: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
Matt-Yorkley
e014e6c1a4
Ensure perform_deliveries is correctly set when testing user confirmation emails
2020-03-24 13:12:20 +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
9994bc75ca
Adapt reports controller to handle routes of reports in the order_management engine differently
2020-03-20 11:07:23 +00:00
Luis Ramos
3f5a964dec
Move enterprise_fee_summaries_spec to order_management engine, moving
...
translation keys for the views and adapting some routes
2020-03-20 11:07:23 +00:00
Luis Ramos
b4befea606
Fix namespace in spec
2020-03-20 11:07:23 +00:00
Luis Ramos
58465c4645
Adapt routes placeholder in the new catalog engine to make it similar to the other engines
2020-03-20 11:07:23 +00:00
Luis Ramos
0b05312f19
Move cookies spec to web engine and adapt routes to the fact they are now normal main apps routes
2020-03-20 11:07:23 +00:00
Luis Ramos
e209452f8b
Make engine routes just prepend to apps routes instead of creating engine routes
...
This makes things a bit simpler in terms of routing, we avoid a problem running specs and we can still have the engine routes separated in specific files
2020-03-20 11:07:23 +00:00
Luis Ramos
a8078b22f8
Move enterprise fees summaries controller and views to ordermanagement engine
2020-03-20 11:07:23 +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