entrouvert
/
passerelle
15
0
Fork 0
Code Issues Pull Requests 15 Releases Activity

connecteur send ethic (#81143) #346

Merged
csechet merged 3 commits from wip/81143-connecteur-send-ethic into main 2023-09-19 15:01:36 +02:00
Conversation 2 Commits 3 Files Changed 15 +390 -227
csechet commented 2023-09-18 21:10:55 +02:00
Owner
Copy Link
No description provided.
csechet changed title from WIP: connecteur send ethic (#81143) to connecteur send ethic (#81143) 2023-09-18 21:11:04 +02:00
csechet changed title from connecteur send ethic (#81143) to WIP: connecteur send ethic (#81143) 2023-09-18 21:12:24 +02:00
csechet force-pushed wip/81143-connecteur-send-ethic from 06d130914a to 0ddf25a8c2 2023-09-19 10:38:50 +02:00 Compare
csechet commented 2023-09-19 12:27:31 +02:00
Author
Owner
Copy Link

La factorisation des connecteurs qui gèrent un crédit restant et les mails d'alerte associés c'est suite à :

https://dev.entrouvert.org/issues/69363#note-14

Plutôt contre, c'est aussi mon indicateur d'attendre 3 répétitions du code avant de commencer à se poser la question :D
Si un troisième connecteur similaire se pointe, je me désigne volontaire pour prendre le sujet !

Avec Send Ethic ça fait 3 : bim boum.

La factorisation des connecteurs qui gèrent un crédit restant et les mails d'alerte associés c'est suite à : https://dev.entrouvert.org/issues/69363#note-14 > Plutôt contre, c'est aussi mon indicateur d'attendre 3 répétitions du code avant de commencer à se poser la question :D > Si un troisième connecteur similaire se pointe, je me désigne volontaire pour prendre le sujet ! Avec Send Ethic ça fait 3 : bim boum.
csechet changed title from WIP: connecteur send ethic (#81143) to connecteur send ethic (#81143) 2023-09-19 12:33:55 +02:00
fpeters requested changes 2023-09-19 13:44:29 +02:00
passerelle/apps/ovh/models.py Outdated
@ -128,6 +111,17 @@ class OVHSMSGateway(SMSResource):
],
}
TEST_CREDIT_LEFT = {
fpeters commented 2023-09-19 13:39:18 +02:00
Owner
Copy Link

Je ne suis pas bien fan ce ces infos de test posées dans les connecteurs mais c'est déjà quelque chose qui est fait, donc bon, tant pis.

Je ne suis pas bien fan ce ces infos de test posées dans les connecteurs mais c'est déjà quelque chose qui est fait, donc bon, tant pis.
passerelle/apps/sendethic/models.py Outdated
@ -0,0 +31,4 @@
allow_premium_rate = None
class Meta:
verbose_name = 'Send Ethic'
fpeters commented 2023-09-19 13:39:53 +02:00
Owner
Copy Link

Sur https://www.sendethic.com/ ils s'écrivent Sendethic, j'utiliserais la même forme.

Sur https://www.sendethic.com/ ils s'écrivent Sendethic, j'utiliserais la même forme.
csechet marked this conversation as resolved
passerelle/apps/sendethic/models.py Outdated
@ -0,0 +43,4 @@
'test_vectors': [
{
'status_code': 400,
'response': {'Message': 'Grêve des PTTs.'},
fpeters commented 2023-09-19 13:40:45 +02:00
Owner
Copy Link

s/Grêve/Grève/ s/PTTs/PTT/ (en français on ne pluralise pas ainsi les acronymes).

s/Grêve/Grève/ s/PTTs/PTT/ (en français on ne pluralise pas ainsi les acronymes).
passerelle/apps/sendethic/models.py Outdated
@ -0,0 +106,4 @@
return response.ok, result
def send_msg(self, text, sender, destinations, **kwargs):
destinations = [f'+{dest[2:]}' for dest in destinations]
fpeters commented 2023-09-19 13:43:14 +02:00
Owner
Copy Link

Un commentaire pour dire qu'ainsi c'est le 00 initial qui est remplacé par un + ? (et une note pour rassurer comme quoi c'est toujours bien des numéros avec 00 qui seront reçus ?).

Un commentaire pour dire qu'ainsi c'est le 00 initial qui est remplacé par un + ? (et une note pour rassurer comme quoi c'est toujours bien des numéros avec 00 qui seront reçus ?).
passerelle/apps/sendethic/models.py Outdated
@ -0,0 +122,4 @@
if not success:
errors.append(message)
if any(errors):
fpeters commented 2023-09-19 13:43:44 +02:00
Owner
Copy Link

Juste if errors: suffit.

Juste ``` if errors:``` suffit.
passerelle/apps/sendethic/models.py Outdated
@ -0,0 +129,4 @@
success, result = self.request('get', endpoint='account/remainingcredits/sms/')
if not success:
self.logger.warning('Cannot retrieve credits for Send Ethic connector: %s', result)
fpeters commented 2023-09-19 13:44:16 +02:00
Owner
Copy Link

Pour ne pas les oublier, on a aussi des s/Send Ethic/Sendethic/ ici (et ci-dessous).

Pour ne pas les oublier, on a aussi des s/Send Ethic/Sendethic/ ici (et ci-dessous).
fpeters approved these changes 2023-09-19 14:09:41 +02:00
fpeters left a comment
Owner
Copy Link

Ça marche (en n'oubliant pas le fixup des commits d'ajustement dans le dernier).

Ça marche (en n'oubliant pas le fixup des commits d'ajustement dans le dernier).
csechet commented 2023-09-19 14:42:50 +02:00
Author
Owner
Copy Link

Merci Fred

Merci Fred
csechet force-pushed wip/81143-connecteur-send-ethic from c855e6ba3d to 199075ed80 2023-09-19 14:46:15 +02:00 Compare
csechet merged commit 199075ed80 into main 2023-09-19 15:01:36 +02:00
csechet deleted branch wip/81143-connecteur-send-ethic 2023-09-19 15:01:36 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
fpeters
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/passerelle#346
No description provided.
Delete Branch "wip/81143-connecteur-send-ethic"

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?