IOS - Mise à jour coordonnées bancaires ne fonctionne pas - Githubissues

CforGood / cforgood-mobile

App Mobile React Native

1 stars 0 forks source link

IOS - Mise à jour coordonnées bancaires ne fonctionne pas #72

Closed FredPetris closed 6 years ago

FredPetris commented 7 years ago

Pas de mise à jour des coordonnées bancaires à partir du profil

Kottidev commented 7 years ago

c'est quoi le problème dans ce cas ?

Kottidev commented 6 years ago

@FredPetris

FredPetris commented 6 years ago

Le problème c'est que je ne reçois aucune requête de mise à jour des données bancaires. Le point API est prêt mais la doc n'était pas à jour : ajout champ stripeToken dans WS PATCH USERS. (idem #75)

Kottidev commented 6 years ago

Ok j'ajouterais le champ stripeToken

FredPetris commented 6 years ago

De plus, il faut vraiment que le 5 € en mensuel et 50€ (et non 51) soient les valeurs par défaut, car si on ne modifie pas le montant, il n'est pas pris en compte et même quand on clique sur participer, il ne se passe rien, l'écran de saisie des coordonnées bancaires ne s'affiche pas.

Kottidev commented 6 years ago

à tester

Kottidev commented 6 years ago

A vérifier

FredPetris commented 6 years ago

KO pour les valeurs par défaut et pas besoin de d'enregistrer les données, si clique sur mettre à jour mes infos, il faut que le montant sélectionné soit automatiquement pris en compte.

Kottidev commented 6 years ago

il faut cliquer sur valider pour faire le Mise à jour de Carte bancaire

Kottidev commented 6 years ago

FredPetris commented 6 years ago

L'écran de saisie n'est pas conforme ! Il faut garder soit les champs, soit la carte mais pas les 2, c'est trop !
Je suis d'accord pour ce bouton "Valider", le soucis se situe sur l'écran précédent quand on modifie le montant, il faut d'abord "Enregistrer" la modification du montant avant de cliquer sur "Mettre à jour CB" pour que le montant soit pris en compte. Or si on clique sur "Mettre à jour CB", le montant sélectionner avec le slider doit automatiquement être pris en compte.

Kottidev commented 6 years ago

@AllanFloury , j'enleve la carte ?

FredPetris commented 6 years ago

Il faut être au plus près des mockups.

Kottidev commented 6 years ago

@AllanFloury , j'enleve la carte , ?

FredPetris commented 6 years ago

Enlève la carte. On utilise les champs.

Kottidev commented 6 years ago

je peux pas l'enlever maintenant , il y a des trucs lieé au champs du formulaire.

FredPetris commented 6 years ago

CF #74

Kottidev commented 6 years ago

je peux pas enlever la carte.

Kottidev commented 6 years ago

duplicated with #75 @FredPetris

can we close it ?