- 01 Mar, 2024 5 commits
-
-
Léo-Paul Géneau authored
Rename triggerParachute function into land in DroneManager for compliance with multicopters APIs.
-
Nicolas Wavrant authored
See merge request !1880
-
Nicolas Wavrant authored
I investigated a case where activities calling an external API were not retried after the initial failure, when it was caused by TimeoutReachedError: https://lab.nexedi.com/nexedi/erp5/-/blob/f41b33cd91348a800db3b95b30efd857f583bcde/bt5/erp5_web_service/MixinTemplateItem/portal_components/mixin.erp5.RESTAPIClientConnectorMixin.py#L142 The interesting thing is that these activities are written in a way that allow them to be retried (automatically, or manually) without causing un-intended side effects, like creating twice a document in the external API. So not having SQLBase retrying them automatically is a pity. The bug is that the use of TimeoutReachedError here was intended only for the activities that are aborted by CMFActivity itself, and not for the userland code that can also use TimeoutReachedError. This commit removes this special processing, as there is no need to especially not retry the activities that trigger the CMFActivity timeout. The max_retry decision should be taken by the calling code, and not by CMFActivity.
-
Rafael Monnerat authored
See merge request !1889
-
Rafael Monnerat authored
See merge request !1887
-
- 29 Feb, 2024 20 commits
-
-
Rafael Monnerat authored
-
Rafael Monnerat authored
Move Simulation to priority 3 helps to not block/delay immediateReindex on a scenario where everything is expanding. We use priority 3 to match what is used in SimulableMixin.updateSimulation and Delivery.localBuild
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Jérome Perrin authored
This fixes compatibility for python3, especially with hash randomization (PYTHONHASHSEED) turned on. This also enables coding style test that was not running for this bt, after fixing everything flagged by coding style test. See merge request nexedi/erp5!1892
-
Jérome Perrin authored
Make sure we see *all* warnings, both in the tests and in the logfile in production. Fix many warnings by updating imports and calls to deprecated APIs See merge request !1879
-
Jérome Perrin authored
for consistency and because it's probably good to let user choose sending mode.
-
Jérome Perrin authored
This matches the form title
-
Jérome Perrin authored
-
Jérome Perrin authored
-
Jérome Perrin authored
-
Jérome Perrin authored
When producing DSN report, there was a few places where we depended on dict iteration order, so the test did not pass when running with PYTHONHASHSEED (erp5_payroll_l10n_fr:testDSNSocialDeclarationReport fails with PYTHONHASHSEED 323). In these changes we add sorts to some places where we were iterating on dicts so that the order of the Lines is deterministic even with hash randomization enabled. Co-authored-by: Nicolas Wavrant <nicolas.wavrant@nexedi.com>
-
Jérome Perrin authored
test was simplified not to use difflib, unittest does a good job of explaining the differences when using splitlines on long texts, no need to compute a diff ourselves.
-
Jérome Perrin authored
-
- 28 Feb, 2024 8 commits
-
-
Rafael Monnerat authored
Without this the redirection places value unquoted causing parser to fail later on when resolve {n.me}. came_from cannot be None so we simplify the block a bit.
-
Rafael Monnerat authored
See merge request nexedi/erp5!1888
-
Rafael Monnerat authored
-
Jérome Perrin authored
-
Jérome Perrin authored
This makes it easier to find the problematic listbox and also report once per listbox.
-
Léo-Paul Géneau authored
See merge request !1870
-
Léo-Paul Géneau authored
Set speed with setTargetCoordinates instead of a separate function because some drones API have an unique call for setting both coordinates and speed and the rate of this call can be curbed.
-
Léo-Paul Géneau authored
fix expected timestamp as its value is now floored to milliseconds
-
- 27 Feb, 2024 5 commits
-
-
Roque authored
-
Roque authored
-
Julien Muchembled authored
-
Georgios Dagkakis authored
-
- 26 Feb, 2024 1 commit
-
-
Jérome Perrin authored
It was using "Reference" instead of the dedicated term "Currency Code". Attribute Unicity constraint message was not correctly including the reference, the error message was always "Another currency with reference None already exists".
-
- 23 Feb, 2024 1 commit
-
-
Rafael Monnerat authored
See merge request nexedi/erp5!1877
-