David Cook
b6a50d85b2
Remove unused code
...
We don't use poltergeist, and it just works fine now.
2024-05-30 14:16:16 +10:00
Gaetan Craig-Riou
0e9b753bbc
Merge pull request #12514 from cyrillefr/RedundantPresenceValidationOnBelongs_part_V
...
Fix redundant presence validation on belongs part V
2024-05-29 09:57:22 +10:00
Maikel
5f137c19a3
Merge pull request #12527 from openfoodfoundation/dependabot/bundler/view_component-3.12.1
...
chore(deps): bump view_component from 3.12.0 to 3.12.1
2024-05-29 08:47:53 +10:00
dependabot[bot]
8421c35e42
chore(deps): bump view_component from 3.12.0 to 3.12.1
...
Bumps [view_component](https://github.com/viewcomponent/view_component ) from 3.12.0 to 3.12.1.
- [Release notes](https://github.com/viewcomponent/view_component/releases )
- [Changelog](https://github.com/ViewComponent/view_component/blob/main/docs/CHANGELOG.md )
- [Commits](https://github.com/viewcomponent/view_component/compare/v3.12.0...v3.12.1 )
---
updated-dependencies:
- dependency-name: view_component
dependency-type: direct:production
update-type: version-update:semver-patch
...
Signed-off-by: dependabot[bot] <support@github.com >
2024-05-28 09:59:37 +00:00
Gaetan Craig-Riou
6f5bcda421
Merge pull request #12523 from openfoodfoundation/dependabot/bundler/webmock-3.23.1
...
chore(deps-dev): bump webmock from 3.23.0 to 3.23.1
2024-05-28 10:02:00 +10:00
dependabot[bot]
f70dd58533
chore(deps-dev): bump webmock from 3.23.0 to 3.23.1
...
Bumps [webmock](https://github.com/bblimke/webmock ) from 3.23.0 to 3.23.1.
- [Changelog](https://github.com/bblimke/webmock/blob/master/CHANGELOG.md )
- [Commits](https://github.com/bblimke/webmock/compare/v3.23.0...v3.23.1 )
---
updated-dependencies:
- dependency-name: webmock
dependency-type: direct:development
update-type: version-update:semver-patch
...
Signed-off-by: dependabot[bot] <support@github.com >
2024-05-27 09:50:38 +00:00
David Cook
a405f17c0b
Merge pull request #12510 from cyrillefr/RemoveUnusedMissingIdsCheckRakeTasks
...
Deleting some used one-shot rake tasks
2024-05-27 13:53:21 +10:00
Gaetan Craig-Riou
a64bf68d54
Merge pull request #12517 from openfoodfoundation/dependabot/bundler/knapsack_pro-7.4.0
...
chore(deps-dev): bump knapsack_pro from 7.2.0 to 7.4.0
2024-05-27 10:11:25 +10:00
dependabot[bot]
9cd5f5aaa0
chore(deps-dev): bump knapsack_pro from 7.2.0 to 7.4.0
...
Bumps [knapsack_pro](https://github.com/KnapsackPro/knapsack_pro-ruby ) from 7.2.0 to 7.4.0.
- [Changelog](https://github.com/KnapsackPro/knapsack_pro-ruby/blob/master/CHANGELOG.md )
- [Commits](https://github.com/KnapsackPro/knapsack_pro-ruby/compare/v7.2.0...v7.4.0 )
---
updated-dependencies:
- dependency-name: knapsack_pro
dependency-type: direct:development
update-type: version-update:semver-minor
...
Signed-off-by: dependabot[bot] <support@github.com >
2024-05-24 09:13:06 +00:00
filipefurtad0
0bfa9fe997
Update all locales with the latest Transifex translations
v4.4.47
2024-05-24 09:49:20 +01:00
cyrillefr
5bc6d16f34
Fix redundant presence validation on belongs part V
...
- presence: true is redundant since Rails 5.0 BUT applies
with new default config of
belongs_to_required_by_default to true.
Lots of files with belongs_to_required_by_default = false
(backward compatibility).
So: deleting this setting implies to adding optional: true
2024-05-24 08:22:20 +02:00
Maikel
5b9e83745d
Merge pull request #12508 from mkllnk/schema
...
Fixup db schema.rb
2024-05-24 08:19:31 +10:00
Filipe
88baf5e1de
Merge pull request #12496 from isidzukuri/12355-remove-unused-discourse
...
[12355] Remove discourse
2024-05-23 19:28:02 +01:00
Filipe
5455e68620
Merge pull request #12504 from dacook/buu/edit-image-turbo-12399
...
[BUU] Remove StimulusReflex from Image Edit
2024-05-23 17:32:59 +01:00
Rachel Arnould
163e9b6c9c
Merge pull request #12505 from filipefurtad0/remove_deprecation
...
Removes deprecation strikethrough from tax report (by producer)
2024-05-23 17:14:28 +02:00
cyrillefr
8c0e20f53c
Deleting some used one-shot rake taskes
...
- in relation to: Fix Rubocop Rails issues #11482
- and Require belongs_to associations by default #11297
- and #12407 #12428 etc.
2024-05-23 10:07:56 +02:00
Maikel Linke
b9c7be769a
Fixup db schema.rb
...
It looks like I forgot to commit the latest change in 46cd60aa3c .
2024-05-23 15:51:43 +10:00
David Cook
e30dc1b05c
Merge pull request #12493 from cyrillefr/RedundantPresenceValidationOnBelongs_part_IV
...
Fix RedundantPresenceValidationOnBelongs on two files
2024-05-23 13:09:17 +10:00
David Cook
55ac816a87
Show success message
...
Phew, that was really easy.
And now the existing feature spec still Just Works.
2024-05-23 11:55:56 +10:00
David Cook
05f0b93548
Use requestSubmit to allow JS events
...
Yay, now it works. Not sure the best way to show loading yet.
- currently the Turbo loading indicator shows which is better than nothing (blue bar at top)
- ideally we could show a small spinner over the image thumbnail. need to write some stimulus to hook into turbo lifecycle I guess.
- or we could activate the frame-level loading overlay. refactor loading_controller a bit so that it's applied on the container, then hopefully we can just call change->loading#showLoading. the turbo_stream response could dectivate it.
2024-05-23 11:55:33 +10:00
David Cook
665273ce2f
[wip] Attempt to update edited image in-place with Turbo Stream
...
but it doesn't quite work.
Maybe we can force it with JS (https://www.writesoftwarewell.com/process-turbo-stream-javascript/ )
2024-05-23 11:55:29 +10:00
David Cook
0e03ff6925
Merge branch 'master' into RedundantPresenceValidationOnBelongs_part_IV
2024-05-23 09:22:36 +10:00
Maikel
74cfa42461
Merge pull request #12465 from MrBowmanXD/mirror
...
Allow return or line break in shipping / payment method description
2024-05-23 09:10:13 +10:00
Maikel Linke
be79016753
Remove now obsolete check in rake task
2024-05-23 09:08:05 +10:00
Maikel
32eb9c5504
Merge pull request #12481 from mkllnk/connect-app-once
...
Avoid submitting duplicate Connected Apps
2024-05-23 08:50:34 +10:00
Manuel Gonçalves
2bc481d8f7
Merge branch 'master' into mirror
2024-05-22 23:35:45 +01:00
Filipe
b4d4faafc0
Merge pull request #12472 from dacook/bump-stimulus_reflex-cable_ready
...
Bump stimulus_reflex to 3.5.1 and cable_ready to 5.0.5
2024-05-22 21:37:42 +01:00
Filipe
1130bf8cbe
Merge pull request #12460 from mkllnk/enterprise-data-purge
...
Clean up unused enterprise fields
2024-05-22 21:29:54 +01:00
filipefurtad0
5ccecdd9ff
Removes deprecation strikethrough from enterprise fees with tax report by producer
2024-05-22 20:38:11 +01:00
cyrillefr
46cd60aa3c
Fix RedundantPresenceValidationOnBelongs on two files
...
- presence: true is redundant since Rails 5.0 BUT applies
with new default config of
belongs_to_required_by_default to true.
Lots of files with belongs_to_required_by_default = false
(backward compatibility).
So: deleting this setting implies to adding optional: true
- added 'NOT NULL' constraints so model constraints match
with contraints on DB tables.
- corresponding migration files to match AR Models &
DB tables
- rake tasks to check corrupt data (ie: NULL/nil in id fields)
- updated the todo
2024-05-22 09:44:02 +02:00
David Cook
536b5608ab
Show edit form for new images
2024-05-22 17:08:58 +10:00
David Cook
3fcc9ac1fa
Move product image edit modal to Turbo Stream
...
Didn't even need to touch the controller, because data-turbo-stream tells it to render the turbo_stream format ✨
But you might notice that it doesn't redirect to the right return_url yet. Let's see if we can use more Turbo to avoid the page refresh..
TODO: also handle empty images
2024-05-22 17:08:58 +10:00
David Cook
b935a0e8d9
[add gem] turbo-rails
...
This gives us a nice 'turbo_stream' format helper.
2024-05-22 15:07:16 +10:00
Maikel
5fdea3e8ce
Merge pull request #12502 from dacook/specs
...
Break up large spec files
2024-05-22 13:39:02 +10:00
Maikel
9d453a82ed
Merge pull request #12494 from filipefurtad0/update_paypal_checkout_system_spec
...
Updates Paypal system spec to consider new checkout flow
2024-05-22 10:17:06 +10:00
David Cook
b2078c2906
Merge pull request #12486 from dacook/tax-category-12480
...
Show any tax category errors when saving variants
2024-05-22 09:30:19 +10:00
David Cook
5ff89a279c
Use Spring to save on subsequent boot times
...
Each time we run a rails command, it can take some time to load up (I think it was 20s). We run two commands (db setup, then rspec), so the second one should be faster now.
2024-05-22 09:21:51 +10:00
David Cook
85165369a2
Move run runner over for system specs.
2024-05-22 09:21:51 +10:00
David Cook
1d11f8c85b
Move bulk order actions to separate spec file
...
This was another large file, potentially causing a bottleneck.
All the order setup is duplicated from the other file which is a bit of shame, but I think it makes sense.
2024-05-22 09:20:12 +10:00
filipefurtad0
818c0306c6
Updates paypal feature spec to consider new checkout flow
2024-05-21 17:37:22 +01:00
David Cook
39f1eac300
Move invoices specs to the invoices spec file
...
The orders file is too big and causes a bottleneck for parallelising specs.
Maybe they should be merged with the above specs, but I'm not familiar enough to know for sure.
2024-05-21 16:43:17 +10:00
David Cook
f0f2754d7f
Remove now-unused translation
2024-05-21 16:33:49 +10:00
isidzukuri
b9a0c1cc87
revert feature-template.md
2024-05-21 09:02:45 +03:00
isidzukuri
621a95e913
Remove discourse
2024-05-21 09:02:45 +03:00
David Cook
b729ec3428
Merge pull request #12490 from mkllnk/mail-spec
...
Move mail spec precompile assets for it
2024-05-21 09:51:46 +10:00
David Cook
c960e55437
Bump cable_ready and stimulus_reflex again
...
Sigh.
2024-05-20 15:51:55 +10:00
Manuel Gonçalves
1c0a6d1229
Reintroduced changes made in commit #12443
2024-05-20 03:55:49 +01:00
Gaetan Craig-Riou
943c21be34
Merge pull request #12487 from openfoodfoundation/dependabot/bundler/rexml-3.2.8
...
chore(deps): bump rexml from 3.2.6 to 3.2.8
2024-05-20 10:38:28 +10:00
David Cook
a9bf25244e
Merge pull request #12457 from abdellani/12452-log-who-deleted-products
2024-05-20 10:37:32 +10:00
Mohamed ABDELLANI
8ccb59a9ab
included LogDestroyPerformer on product model
2024-05-17 15:42:39 +01:00