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

Fix CI for v16 #52

Merged
merged 6 commits into from
Mar 5, 2024
Merged

Fix CI for v16 #52

merged 6 commits into from
Mar 5, 2024

Conversation

florian-dacosta
Copy link
Member

No description provided.

@florian-dacosta
Copy link
Member Author

@Paul_Goubert @bealdav

J'essaie dans cette PR de corriger la CI, et cela corrige aussi le bug de Paul (incompatibilité avec le module rating).
Cependant, pas de garantie, ca reste super touchy toute cette partie des mails.

Il reste encore un test qui plante, et cela correspond réelement à un bug. Dans l'ERP du client, on peut ajouter des pièces jointes, mais on ne peut pas les télécharger.
J'ai commencé à regarder cette partie, mais je sèche un peu sur la façon de faire pour que ça marche. (La façon de télécharger les pièce jointes n'a plus rien à voir en v16 par rapport aux versions antérieures)

@sebastienbeau Il faudrait qu'on en discute
@bguillot pour info

@PaulGoubert
Copy link

@Kev-Roche c'est pour Voltaire que j'avais le soucis, car la module Rating est installé.
@florian-dacosta Merci en tout cas pour le diagnostique et le fix. J'attend le retour de Kévin pour l'installer.

Copy link

codecov bot commented Feb 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ No coverage uploaded for pull request base (16.0@468d53b). Click here to learn what that means.

Additional details and impacted files
@@           Coverage Diff           @@
##             16.0      #52   +/-   ##
=======================================
  Coverage        ?   59.49%           
=======================================
  Files           ?       12           
  Lines           ?      669           
  Branches        ?       95           
=======================================
  Hits            ?      398           
  Misses          ?      253           
  Partials        ?       18           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@florian-dacosta
Copy link
Member Author

@bealdav @PaulGoubert
J'ai fais des nouveaux correctifs, maintenant le téléchargement de pièce jointe (depuis l'ERP client) est possible avec un bouton "download".
A ma connaissance, il n'y a plus de problème sur ce module en v16, je vais merger cela.
N'hésitez pas à me remonter si vous avez des soucis une fois que vous aurez mis à jour

@PaulGoubert
Copy link

@florian-dacosta Merci bien pour le taff

@florian-dacosta florian-dacosta merged commit cc9a782 into 16.0 Mar 5, 2024
3 checks passed
@github-grap-bot github-grap-bot deleted the 16-fix-ci branch March 5, 2024 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants