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.

Sign up for GitHub

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

Permettre de s'abonner plus d'une heure, un jour ou un mois avec Stri… #126

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Permettre de s'abonner plus d'une heure, un jour ou un mois avec Stri… #126

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
7bff389
Permettre de s'abonner plus d'une heure, un jour ou un mois avec Stri…
yohooololo Apr 29, 2024
49478a0
Fix, recuperation de la variable.
yohooololo Apr 29, 2024
b7137a1
RIen
yohooololo May 15, 2024
dff0ca7
Ajout d'une balise #CHANGER_MOYEN_PAIEMENT permetant de modifier la c…
yohooololo May 30, 2024
226e722
Ajout d'une balise #CHANGER_MOYEN_PAIEMENT permetant de modifier la c…
yohooololo May 30, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion presta/stripe/call/request.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -291,6 +291,8 @@ function presta_stripe_call_request_dist($id_transaction, $transaction_hash, $co
}
}

$interval_count = 1;
Copy link
Contributor

Choose a reason for hiding this comment

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

mais du coup là tu l'initialises à 1 mais tu ne l'utilises jamais ensuite non ? ça devrait être le fallback de si c'est pas fourni dans le tableau

Copy link
Author

Choose a reason for hiding this comment

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

Ce sont les plugins qui font l'interface entre les commandes et les abonnements qui sont censés les manipuler, non ?


$session_desc = [
'payment_method_types' => $payment_types,
'mode' => 'subscription',
Expand Down Expand Up @@ -321,7 +323,7 @@ function presta_stripe_call_request_dist($id_transaction, $transaction_hash, $co
'unit_amount' => $montant_echeance,
'recurring' => [
'interval' => $interval,
'interval_count' => 1,
'interval_count' => $echeance['interval_count'],
Copy link
Contributor

Choose a reason for hiding this comment

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

quelque chose comme $echeance['interval_count'] ?? $interval_count ?

autre question : est-ce que dans le tableau qui est générique et qui vient de "decrire_echeance", on doit attendre une clé qui s'appelle "interval_count" et qui est propre à stripe, ou bien on peut trouver un nom qui est plus générique et qui sémantiquement pourrait s'appliquer à tout ?

Copy link
Author

Choose a reason for hiding this comment

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

Alors, tu peux soulever le chantier de la cohérence des noms des variables pour "decrire_echeance", parce qu'entre les variables en français ("montant", "montant_init"), les variables en anglais ("count",, "count_init","date_start"), et les variables dans les 2 langues abregées, ("frequ", "nb"), j'en n'ai pas vraiment trouvé, perso, de cohérence. J'ai l'impression qu'elles ont été ajoutées en fonction de ce qu'amenaient les API de paiement. Alors j'ai fait pareil avec Stripe.

Ils font comment les autres prestataires pour proposer des abonnements trimestriels par exemple ?

Copy link
Author

Choose a reason for hiding this comment

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

quelque chose comme $echeance['interval_count'] ?? $interval_count ?

En fait, tu as raison. Je corrige ça...

//'trial_period_days' => 0, // default
],
//'billing_scheme' => 'per_unit', // implicite, non modifiable via price_data
Expand Down