Maikel
d3e00cd4b7
Merge pull request #12633 from openfoodfoundation/dependabot/npm_and_yarn/floating-ui/dom-1.6.7
...
chore(deps): bump @floating-ui/dom from 1.6.6 to 1.6.7
2024-07-02 10:59:29 +10:00
Gaetan Craig-Riou
7a741d92e7
Merge pull request #12630 from dacook/cleanup
...
Remove unused spec code
2024-07-02 10:05:17 +10:00
dependabot[bot]
33b680c67c
chore(deps): bump @floating-ui/dom from 1.6.6 to 1.6.7
...
Bumps [@floating-ui/dom](https://github.com/floating-ui/floating-ui/tree/HEAD/packages/dom ) from 1.6.6 to 1.6.7.
- [Release notes](https://github.com/floating-ui/floating-ui/releases )
- [Changelog](https://github.com/floating-ui/floating-ui/blob/master/packages/dom/CHANGELOG.md )
- [Commits](https://github.com/floating-ui/floating-ui/commits/@floating-ui/dom@1.6.7/packages/dom )
---
updated-dependencies:
- dependency-name: "@floating-ui/dom"
dependency-type: direct:production
update-type: version-update:semver-patch
...
Signed-off-by: dependabot[bot] <support@github.com >
2024-07-01 09:05:34 +00:00
David Cook
c747b2e60c
Remove unused code
...
We don't use poltergeist, and it just works fine now.
2024-07-01 10:43:06 +10:00
Konrad
5bb47823c6
Merge pull request #12619 from rioug/12559-disable-report-button
...
[Reports] Disable GO button once clicked
2024-06-28 17:11:29 +02:00
Konrad
909bc2792c
Merge pull request #12565 from chahmedejaz/task/11987-remove-SR-from-clone
...
11987: Prevent Reloading the whole table upon Product Clone
2024-06-28 15:35:52 +02:00
Konrad
04dd463f8e
Merge pull request #12592 from mkllnk/report-rendering
...
Load large on-screen reports on demand
2024-06-28 11:14:38 +02:00
Maikel Linke
b117fd03da
Simplify spec checking for disabled button
2024-06-28 08:42:24 +10:00
David Cook
d799230440
Update controller to catch more specific error
...
We now know which errors to expect. We still let other unknown errors raise higher up, where they will be logged with BugSnag, and treated as internal_server_error.
2024-06-27 12:51:28 +10:00
David Cook
92bd7a5d37
Revert new product duplicator behaviour
2024-06-27 11:39:49 +10:00
David Cook
898ab08bab
Add specs for invalid records
...
It turns out that the duplicator still raises an exception in some cases. Now I think I see why the the controller was catching the exceptions. At least now we know which exceptions to catch.
2024-06-27 11:39:49 +10:00
Ahmed Ejaz
b1a3bff2ed
11987: add failing spec for product clone
2024-06-26 16:48:14 +05:00
Ahmed Ejaz
4315a05eb8
11987: fix lint issue
2024-06-26 16:32:32 +05:00
Ahmed Ejaz
0aea201d53
11987: update mocking save! to save
2024-06-26 16:31:16 +05:00
Ahmed Ejaz
5a259f1b91
Merge branch 'master' into task/11987-remove-SR-from-clone
2024-06-26 16:18:42 +05:00
Ahmed Ejaz
9f832e6743
11987: update code as per new product duplicator behavior
2024-06-26 16:16:13 +05:00
Ahmed Ejaz
95972c75c6
11987: remove rescue StandardError from controller
2024-06-26 16:10:31 +05:00
Ahmed Ejaz
3f6e5e7d09
11987: duplicate product without raising error
2024-06-26 16:00:59 +05:00
David Cook
0001ffa970
Fix product index
...
The param product_index wasn't present, so it was always choosing 1.
The products on page are indexed 0-14, so of course it would always conflict.
It would be simpler if we just used product IDs as the index, I think I did earlier but don't remember why not.
Anyway, using a negative number seems to work.
If there's an error, there will only be one at a time.
2024-06-26 17:04:49 +10:00
Gaetan Craig-Riou
74fb6c3143
Disable GO button once clicked
...
It is disabled both on client side and server side, so even if the
server takes a while to respond the button is disabled
2024-06-26 16:01:05 +10:00
Gaetan Craig-Riou
ad78210b00
Update all locales with the latest Transifex translations
v4.4.54
2024-06-26 14:18:59 +10:00
David Cook
a22c17a520
Merge pull request #12618 from dacook/buu/page-title
...
[BUU] Add translation for "Products V3" page title
2024-06-26 10:03:21 +10:00
David Cook
0e26df2c18
Add translation for products_v3 page title
2024-06-26 09:43:38 +10:00
David Cook
2269bcee4c
Merge pull request #12616 from chahmedejaz/task/12603-auto-author-assign-workflow
...
Add Auto Author Assign Workflow for Pull Requests
2024-06-26 09:27:58 +10:00
Gaetan Craig-Riou
cb98f2f3d5
Merge pull request #12606 from dacook/buu/enable-admin_style_v3-new_users
...
[BUU] Enable admin_style_v3 for new users from 3/07/2024
2024-06-26 09:23:40 +10:00
Gaetan Craig-Riou
1f018292f7
Merge pull request #12605 from abdellani/12373_12374_fix_bulk_invoice_sending_printing
...
[Invoices] Notify if any order cannot be invoiced on bulk invoice printing (update the behavior only when invoice flag enabled )
2024-06-26 09:16:55 +10:00
Maikel
8dff2e6017
Merge pull request #12614 from cyrillefr/FixRailsUnknownEnv
...
Fixes Rails/UnknownEnv offense
2024-06-26 09:06:00 +10:00
Maikel
2619e12216
Merge pull request #12617 from openfoodfoundation/dependabot/npm_and_yarn/floating-ui/dom-1.6.6
...
chore(deps): bump @floating-ui/dom from 1.6.5 to 1.6.6
2024-06-26 09:03:49 +10:00
dependabot[bot]
dd68ccdf2d
chore(deps): bump @floating-ui/dom from 1.6.5 to 1.6.6
...
Bumps [@floating-ui/dom](https://github.com/floating-ui/floating-ui/tree/HEAD/packages/dom ) from 1.6.5 to 1.6.6.
- [Release notes](https://github.com/floating-ui/floating-ui/releases )
- [Changelog](https://github.com/floating-ui/floating-ui/blob/master/packages/dom/CHANGELOG.md )
- [Commits](https://github.com/floating-ui/floating-ui/commits/@floating-ui/dom@1.6.6/packages/dom )
---
updated-dependencies:
- dependency-name: "@floating-ui/dom"
dependency-type: direct:production
update-type: version-update:semver-patch
...
Signed-off-by: dependabot[bot] <support@github.com >
2024-06-25 09:59:33 +00:00
cyrillefr
b09f41136f
Fixes Rails/UnknownEnv offense
...
- Cop: Rails/UnknownEnv
- Cf. https://docs.rubocop.org/rubocop-rails/cops_rails.html#railsunknownenv
- Rails.env.typo? will always return false and will not return an error
- The way to check this is to add environments in the Environments config parameter
- in the rubocop file.
2024-06-25 10:18:45 +02:00
Ahmed Ejaz
feb1fa1198
12603 - add auto-author-assign workflow
2024-06-25 12:20:24 +05:00
David Cook
93798ed54b
Merge pull request #12599 from chahmedejaz/task/12346-rubocop-rails-pluck-and-pluck-where
...
12346 - Fix Rubocop Rails/Pluck and Rails/PluckInWhere
2024-06-25 17:12:18 +10:00
Ahmed Ejaz
bb6e69ff56
Merge branch 'master' into task/12346-rubocop-rails-pluck-and-pluck-where
2024-06-25 11:53:50 +05:00
David Cook
b4ef83b3a7
Enable admin_style_v3 for new users from 3/07/2024
2024-06-25 13:04:36 +10:00
Maikel
c8e134cef5
Remove redundant wait loading report content
...
Co-authored-by: Gaetan Craig-Riou <40413322+rioug@users.noreply.github.com >
2024-06-25 11:50:01 +10:00
Gaetan Craig-Riou
3afb3f5ba0
Merge pull request #12601 from cyrillefr/FixRailsRelativeDateConstant
...
Fixes Rails/RelativeDateConstant offense
2024-06-25 09:15:29 +10:00
Mohamed ABDELLANI
a3fd7b77c8
notify if any distributors doesn't have an abn
2024-06-24 14:46:01 +01:00
Ahmed Ejaz
72640cfc2a
12346: refactor orders_controller spec
2024-06-24 14:43:17 +05:00
Ahmed Ejaz
ff10a36e4a
12346: refactor bulk_line_items_controller spec
2024-06-24 14:25:34 +05:00
cyrillefr
980ce95ab2
Requested changes
...
- replace direct instance variable use by instance getter use
2024-06-24 10:37:43 +02:00
Gaetan Craig-Riou
c4470f76e3
Merge pull request #12593 from filipefurtad0/regression_spec_12591
...
Adds regression spec for #12591
2024-06-24 14:46:29 +10:00
Gaetan Craig-Riou
cd768a1fe9
Merge pull request #12585 from mkllnk/flaky-caching
...
Ensure all assets are compiled on test page visit
2024-06-24 14:44:11 +10:00
Gaetan Craig-Riou
c756afa1cc
Merge pull request #12571 from filipefurtad0/adds_spec_on_abn_restriction
...
Adds test case to assure acceptance criteria from #12374 and #12373
2024-06-24 14:41:31 +10:00
Ahmed Ejaz
afd09091a2
12346 - fix broken specs
2024-06-22 02:11:39 +05:00
cyrillefr
659111932c
Fixes Rails/RelativeDateConstant offense
...
- Cop: Rails/RelativeDateConstant
- raises offense if Constant is relative data (ie: since, ago)
- Reason: relative data will be evaluated only once
- BUT here, Date should not be evaluated in a class method, and have a different
- value for each call. But the data should be the same for an instance
- Therefore: move the ago in init method
- Cf. https://docs.rubocop.org/rubocop-rails/cops_rails.html#railsrelativedateconstant
- Since there is no constant to be called form a class, but a date from an instance, the
spec has been modified accordingly. The RemoveTransientData.new.call had to be splitted.
2024-06-21 23:07:16 +02:00
Ahmed Ejaz
9dd427ab7f
12346 - fix rubocop rails/pluckInWhere errors
2024-06-22 01:54:54 +05:00
Ahmed Ejaz
98e0b0e703
12346 - fix rubocop rails/pluck errors
2024-06-22 01:45:58 +05:00
Maikel
0dd7f264b9
Merge pull request #12542 from anansilva/12448-sanitise-html-custom-tab
...
Sanitize HTML in custom tab content [read only]
2024-06-21 08:35:26 +10:00
filipefurtad0
1d55c05900
Update all locales with the latest Transifex translations
v4.4.53
2024-06-20 11:17:26 -06:00
Konrad
f9e1ff9238
Merge pull request #12567 from mkllnk/flash-css-fix-v3
...
Error messages don't block bottom of page clicks
2024-06-20 17:22:02 +02:00