Skip to content

[FIX] currency_rate_update_boi: fix test#3848

Merged
OCA-git-bot merged 1 commit into
OCA:16.0from
Borruso:16.0-fix-currency_rate_update_boi-fix_test
Jan 26, 2024
Merged

[FIX] currency_rate_update_boi: fix test#3848
OCA-git-bot merged 1 commit into
OCA:16.0from
Borruso:16.0-fix-currency_rate_update_boi-fix_test

Conversation

@Borruso

@Borruso Borruso commented Jan 15, 2024

Copy link
Copy Markdown
Contributor

No description provided.

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hi @eLBati,
some modules you are maintaining are being modified, check this out!

@Borruso Borruso force-pushed the 16.0-fix-currency_rate_update_boi-fix_test branch from a10a4c5 to 8e23f8e Compare January 15, 2024 08:00
Comment thread currency_rate_update_boi/tests/test_currency_rate_update_boi.py
@francesco-ooops francesco-ooops linked an issue Jan 15, 2024 that may be closed by this pull request
@Borruso Borruso force-pushed the 16.0-fix-currency_rate_update_boi-fix_test branch from 8e23f8e to 8b1caa3 Compare January 16, 2024 11:25

@eLBati eLBati left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Grazie

@francesco-ooops

Copy link
Copy Markdown
Contributor

@SirAionTech se puoi fare una review vediamo di sbloccare la situazione sulla 16, grazie!

@SirAionTech

Copy link
Copy Markdown
Contributor

@SirAionTech se puoi fare una review vediamo di sbloccare la situazione sulla 16, grazie!

Non so quale vantaggio possa avere ritornare Giovedì piuttosto che Venerdì piuttosto che qualsiasi altro giorno perché non ho seguito questi sviluppi, quindi non saprei come revisionare.

@francesco-ooops

Copy link
Copy Markdown
Contributor

@tafaRU mergiamo?

@francesco-ooops

Copy link
Copy Markdown
Contributor

@OCA/local-italy-maintainers possiamo mergiare questa che blocca tutte le PR v16?

@Borruso

Borruso commented Jan 26, 2024

Copy link
Copy Markdown
Contributor Author

@SirAionTech se puoi fare una review vediamo di sbloccare la situazione sulla 16, grazie!

Non so quale vantaggio possa avere ritornare Giovedì piuttosto che Venerdì piuttosto che qualsiasi altro giorno perché non ho seguito questi sviluppi, quindi non saprei come revisionare.

in pratica la banca d'italia non ti risponde nulla se quando fai la chiamata è sabato o domenica
quindi se parte il test sabato c'è la funzione che calcola da data e ritornare a venerdì

@sergiocorato

Copy link
Copy Markdown
Contributor

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-3848-by-sergiocorato-bump-patch, awaiting test results.

@SirAionTech

Copy link
Copy Markdown
Contributor

@SirAionTech se puoi fare una review vediamo di sbloccare la situazione sulla 16, grazie!

Non so quale vantaggio possa avere ritornare Giovedì piuttosto che Venerdì piuttosto che qualsiasi altro giorno perché non ho seguito questi sviluppi, quindi non saprei come revisionare.

in pratica la banca d'italia non ti risponde nulla se quando fai la chiamata è sabato o domenica quindi se parte il test sabato c'è la funzione che calcola da data e ritornare a venerdì

Capito grazie, ma io non ho mai fatto partire dei test di Sabato o Domenica; tutte le volte che è capitato era tra Lunedì e Venerdì.
Ad esempio l'errore https://github.com/OCA/l10n-italy/actions/runs/7337497663/job/19978466159#step:8:829 è successo il 27 Dicembre che è Mercoledì.

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@OCA-git-bot OCA-git-bot merged commit d5bf2f4 into OCA:16.0 Jan 26, 2024
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 163a4ac. Thanks a lot for contributing to OCA. ❤️

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.

Errore nei test di currency_rate_update_boi

6 participants