Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BO - Nouveau formulaire] Timeout upload multiple avec de nombreux fichiers #2259

Merged
merged 1 commit into from
Feb 19, 2024

Conversation

sfinx13
Copy link
Collaborator

@sfinx13 sfinx13 commented Feb 19, 2024

Ticket

#2255

Description

Le timeout de 30 sec est assez limité pour de l'upload multiple car la même instance axios est utilisé pour les différents upload

Changements apportés

  • Ajout d'une méthode dédiée pour gérer l'uplad de fichier avec une création d'une nouvelle instance avec un timeout de 120s

Pré-requis

make npm-build

Tests

  • Déposer au moins 30 photos sur un désordre et déposer votre signalement

Copy link

sonarcloud bot commented Feb 19, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@hmeneuvrier
Copy link
Collaborator

hmeneuvrier commented Feb 19, 2024

J'ai jouté plein de photos sur un désordre, je n'ai aucune erreur, ni à l'ajout de photos, ni à la validation du signalement, mais je ne vois pas les photos une fois le signalement dans le BO

EDIT : en fait il faut accepter le signalement pour qu'elles apparaissent. tout va bien (peut-être un pb de worker)

Copy link
Collaborator

@hmeneuvrier hmeneuvrier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK

Copy link
Collaborator

@emilschn emilschn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK lecture et tests

@emilschn emilschn merged commit b60c0fb into main Feb 19, 2024
3 checks passed
@sfinx13
Copy link
Collaborator Author

sfinx13 commented Feb 19, 2024

@hmeneuvrier comme tu as mis plein de photos, faut laisser le temps au worker de bosser (pour ma part j'ai tester avec 200 photos) :-D

@sfinx13 sfinx13 deleted the hotfix/2255-timeout-upload-multiple-file branch February 21, 2024 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants