Filipe
e99b072442
Merge pull request #12313 from anthonyms/11482-fix-rubocop-rails-issue-has_many
...
Fix Rubocop Rails: Rails/HasManyOrHasOneDependent
2024-04-25 16:24:47 +01:00
Filipe
f269584b8e
Merge pull request #12392 from sergioosouzaa/fix/duplicate-language-display-new-user-11513
...
Fix Duplicate Language Display on New User Creation
2024-04-25 16:01:09 +01:00
Filipe
4ff832693c
Merge pull request #12412 from dacook/buu/fix-non-admin-saving-12403
...
[BUU] Fix non-admin saving
2024-04-25 12:37:11 +01:00
Rachel Arnould
09354dc80e
Merge pull request #12219 from abdellani/prevent-generate-invoices-when-distributor-cannot-invoice
...
prevent generating invoices when order's distributor can't generate invoices
2024-04-25 12:18:49 +02:00
David Cook
b846d0f517
Add ability to bulk update products for product managers
...
I forgot to do this in #12328 [BUU] Remove Stimulus Reflex from Products screen
2024-04-24 10:56:30 +10:00
filipefurtad0
85e3d5d858
Tests payment intents for offline payments
...
Moves let variables outside shared examples block
Re-records cassettes
2024-04-23 12:24:00 +01:00
Anthony Musyoki
0d03cdf815
Fix Rubocop: Delete dependent stock_movements
2024-04-23 13:13:26 +03:00
Anthony Musyoki
434afb73cd
Fix Rubocop: Update handling of enterprise associations
2024-04-23 12:40:58 +03:00
Anthony Musyoki
c2cbe4f0bf
Fix Rubocop: Hard delete paranoid associations
...
As much as the associated models act_as_paranoid, it
doesnt make sense to keep them around after deleting the enterprise
2024-04-23 12:31:43 +03:00
Anthony Musyoki
4f851bbe1f
Fix Rubocop: Do not delete dependent stock_movements
2024-04-23 12:31:43 +03:00
Anthony Musyoki
1ec453df4d
Fix Rubocop issue: Do not delete addresses having shipments
...
The reasoning is that we should not delete an address that has
ever received a shipment
2024-04-23 12:31:43 +03:00
Anthony Musyoki
5559816e12
Fix Rubocop Rails issue: Rails/HasManyOrHasOneDependent
2024-04-23 12:31:41 +03:00
David Cook
d0f683d279
Spec for bug
2024-04-23 16:48:52 +10:00
Mohamed ABDELLANI
7aad4ad652
Merge branch 'master' into prevent-generate-invoices-when-distributor-cannot-invoice
2024-04-22 10:55:39 +01:00
Konrad
8e7f4ff908
Merge pull request #12333 from chahmedejaz/bugfix/12294-wrong-tax-category-display
...
[BUU] Fix Wrong Tax Category Display
2024-04-20 11:42:05 +02:00
Rachel Arnould
2090bc1b47
Merge pull request #12361 from dacook/buu/add-turbo
...
[BUU] Add Turbo for dynamic page loading
2024-04-19 11:44:17 +02:00
David Cook
5b043574a2
Merge pull request #12384 from mkllnk/mail-config
...
Don't pass invalid auth method "None" to net-smtp
2024-04-18 16:51:51 +10:00
Maikel
09ece4b35e
Merge pull request #12396 from openfoodfoundation/dependabot/bundler/stripe-11.1.0
...
chore(deps): bump stripe from 11.0.0 to 11.1.0
2024-04-18 08:56:59 +10:00
Sergio Souza
9477725997
update tests for avaliable_locales
2024-04-18 08:55:34 +10:00
Maikel Linke
c4e92e7d8f
Update Stripe API recordings for new version
2024-04-18 08:42:18 +10:00
filipefurtad0
574297343f
Sets pending examples for order edit page subsections
2024-04-17 14:27:56 +01:00
David Cook
acc72514de
Fix spec
2024-04-17 15:17:59 +10:00
David Cook
72ce3a01a9
Ensure search terms and filters are retained when saving
2024-04-17 15:17:59 +10:00
Maikel Linke
5cd53e0b5e
Clean up MailSettings spec
...
Best viewed with whitespace ignored.
2024-04-16 16:44:12 +10:00
Maikel Linke
a41019fbff
Simplify SMTP auth method selection
...
Instead of using the auth method name, let's just not supply username
and password when we don't want to authenticate. The two affected
servers AU and CA don't have credentials set anyway. This is compatible.
The specs needed changing though.
2024-04-16 16:40:50 +10:00
Maikel Linke
d6f2a531aa
Don't pass invalid auth method "None" to net-smtp
...
It's our magic word to not pass username and password on.
2024-04-16 16:18:49 +10:00
Gaetan Craig-Riou
5371258125
Merge pull request #12366 from openfoodfoundation/dependabot/bundler/stripe-11.0.0
...
chore(deps): bump stripe from 10.15.0 to 11.0.0
2024-04-16 09:16:49 +10:00
Maikel Linke
d9ec6e2ca3
Wait longer for real Stripe responses
...
When we re-record cassettes, I noticed this spec failing. We may need to
add this parameter to more specs.
2024-04-12 12:34:26 +10:00
Maikel Linke
e5323c8e82
Update Stripe API recordings for new version
2024-04-12 12:27:02 +10:00
filipefurtad0
fd54a12bcb
Moves methods to end of the file
2024-04-11 12:36:47 +01:00
filipefurtad0
f4108e97c7
Improves regression spec after reviewer feedback
...
Removes shared_examples, defines a separate method
Removes pending to bring spec to green
2024-04-11 12:33:10 +01:00
Konrad
27a4202fa4
Merge pull request #12357 from mkllnk/fix-invoice-order
...
Fix: preserve order of invoices in bulk print
2024-04-11 12:47:14 +02:00
Maikel
dc7c5b6748
Merge pull request #12359 from openfoodfoundation/dependabot/bundler/stripe-10.15.0
...
chore(deps): bump stripe from 10.14.0 to 10.15.0
2024-04-11 16:25:04 +10:00
Maikel
082b1f9411
Merge pull request #12300 from mkllnk/checkout-steps
...
Remove unnecessary method checkout_steps
2024-04-11 15:57:45 +10:00
Maikel Linke
2f3b8c8573
Re-record Stripe cassettes
2024-04-11 11:45:14 +10:00
David Cook
3318c2622d
Merge pull request #12352 from cyrillefr/FixRailsWhereEquals
...
Fix rubocop FixRailsWhereEquals
2024-04-11 11:43:03 +10:00
Maikel Linke
c2c7910357
Reset I18n.local for each spec
...
This avoids a locale setting leaking from one spec to another. It also
means that we don't have to reset the locale in individual specs.
Also:
- `cookies` is reset automatically and we don't need to do that.
- Removed some unused code (German number format and helper methods).
2024-04-11 10:14:52 +10:00
Ahmed Ejaz
b2172ef8d8
12332 - Add around block to apply default_locale on specs
2024-04-11 10:14:52 +10:00
Ahmed Ejaz
693b9bd171
12332 - Fix rubocop Rails/I18nLocaleAssignment errors
...
- use I18n.with_locale method rather than direct locale assignment
2024-04-11 10:13:50 +10:00
Filipe
89033579bd
Merge pull request #12328 from dacook/buu/remove-stimulus-reflex
...
[BUU] Remove Stimulus Reflex from Products screen
2024-04-10 21:59:59 +01:00
Konrad
b3dd51d6ac
Merge pull request #12324 from chahmedejaz/task/12314-fix-rubocop-rails-helper-instance-variable
...
Fix Rubocop Rails/HelperInstanceVariable
2024-04-10 20:17:06 +02:00
David Cook
e78ef120f4
TODO
...
I think this case got lost.
2024-04-10 15:17:56 +01:00
David Cook
129ceb01f0
Fix spec
2024-04-10 15:17:56 +01:00
David Cook
fd8be37a62
Use shared page controls on products screen
...
This has an auto submit and can potentially work with Turbo, like on the Orders screen.
2024-04-10 15:17:56 +01:00
David Cook
0fc3d39106
Ensure pagination is retained when saving or discarding
...
But we have more work to do.
2024-04-10 15:17:56 +01:00
David Cook
2c71f7f1ed
Discard changes by reloading the page
...
Now there's a popup asking to confirm, which I think is worth keeping!
2024-04-10 15:17:56 +01:00
David Cook
8696e05e66
Move index and bulk_update actions to good ol' HTTP requests
...
We've found that we just can't rely in StimulusReflex (and the underlying WebSockets stack) to guarantee a response to a request.
Because of this, there was intermittent issues when the server was overloaded with large requests, and the response never arrived, leaving an infinite loader, and a poor user wondering if anything was still happening.
2024-04-10 15:17:56 +01:00
Maikel Linke
3382a62eb5
Simplify BulkInvoiceJob by removing checks
...
The check for invoiceability is already done by the reflex triggering
the job. Let's DRY the code, save time and be more flexible in the
future.
Also checking the order of actually generated PDF pages.
2024-04-10 16:52:56 +10:00
Maikel Linke
48b447500f
Move selection of invoicable orders to database
...
It's more efficient and should allow for further optimisations.
2024-04-10 16:52:56 +10:00
Maikel Linke
16c877f7cb
Fix: preserve order of invoices in bulk print
2024-04-10 14:51:55 +10:00
