entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 34 Releases Activity

misc: allow using schwifty for IBAN validation (#41903) #959

Merged
fpeters merged 1 commits from wip/41903-iban-schwifty into main 2024-01-12 21:19:48 +01:00
Conversation 1 Commits 1 Files Changed 3 +63 -46
fpeters commented 2023-12-23 11:35:34 +01:00
Owner
Copy Link
No description provided.
fpeters force-pushed wip/41903-iban-schwifty from 2e7bb91fe5 to fc69e47a5a 2023-12-23 12:10:31 +01:00 Compare
fpeters changed title from WIP: misc: allow using schwifty for IBAN validation (#41903) to misc: allow using schwifty for IBAN validation (#41903) 2023-12-23 12:19:37 +01:00
fpeters reviewed 2023-12-23 12:21:29 +01:00
wcs/qommon/misc.py
@ -64,0 +64,4 @@
try:
from schwifty import IBAN
except ImportError:
IBAN = None
fpeters commented 2023-12-23 12:21:29 +01:00
Author
Owner
Copy Link

C'est laissé optionnel par flemme de regarder pour le backport du paquet debian vers bullseye; une fois qu'on sera en bookworm on pourra regarder (a priori juste copier le paquet de trixie fera l'affaire).

C'est laissé optionnel par flemme de regarder pour le backport du paquet debian vers bullseye; une fois qu'on sera en bookworm on pourra regarder (a priori juste copier le paquet de trixie fera l'affaire).
tnoel requested changes 2023-12-23 19:17:41 +01:00
tnoel left a comment
Owner
Copy Link

Pourquoi pas, bien que je trouve https://github.com/mdomke/schwifty assez mouvant (mais c'est sans doute juste l'ajout des banques et je sais pas si ça sera gênant que le paquet Debian soit en retard là dessus).

Sinon, peut-être ajouter le paquet dans Recommends sur debian/control ? (pas que ça soit absolument nécessaire, mais bon, en guide de rappel...) ?

Pourquoi pas, bien que je trouve https://github.com/mdomke/schwifty assez mouvant (mais c'est sans doute juste l'ajout des banques et je sais pas si ça sera gênant que le paquet Debian soit en retard là dessus). Sinon, peut-être ajouter le paquet dans Recommends sur debian/control ? (pas que ça soit absolument nécessaire, mais bon, en guide de rappel...) ?
fpeters commented 2023-12-23 19:49:58 +01:00
Author
Owner
Copy Link

Sinon, peut-être ajouter le paquet dans Recommends sur debian/control ? (pas que ça soit absolument nécessaire, mais bon, en guide de rappel...) ?

J'étais plutôt à attendre qu'on soit en bookworm, pour faire le backport et supprimer l'ancien code, et à ce moment alors avoir une vraie dépendance.

> Sinon, peut-être ajouter le paquet dans Recommends sur debian/control ? (pas que ça soit absolument nécessaire, mais bon, en guide de rappel...) ? J'étais plutôt à attendre qu'on soit en bookworm, pour faire le backport et supprimer l'ancien code, et à ce moment alors avoir une vraie dépendance.
tnoel approved these changes 2024-01-09 14:39:50 +01:00
fpeters merged commit 40b20bcd4b into main 2024-01-12 21:19:48 +01:00
fpeters deleted branch wip/41903-iban-schwifty 2024-01-12 21:19:48 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
tnoel
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#959
No description provided.
Delete Branch "wip/41903-iban-schwifty"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?