search

ALLINAPPLI / snpnc

0 stars 0 forks source link

Mapping CF7 - ACF #50

Closed allinappliadmin closed 10 months ago

allinappliadmin commented 10 months ago

https://sandbox.snpnc.org/test-formulaire/ https://sandbox.snpnc.org/wp-admin/admin.php?page=wpcf7&post=26197&active-tab=0#/ https://sandbox.snpnc.org/wp-admin/post.php?post=2161&action=edit

allinappliadmin commented 10 months ago

je vais ajouter un champ supplémentaire dans la synchro mais c'est aussi une liste déroulante comme le grade/fonction donc si on peut le faire pour l'un on pourra le faire pour les deux. Donc les champs à synchroniser de CF7 vers ACF

telephone vers user_telephone
fonction vers fonction
typedecontrat vers type-de-contrat
compagnie vers la_compagnie
wyde22 commented 10 months ago

j'ai finalisé une première version de ce travail. Tout d'abord j'ai désactivé un bout de code du fork de frontend-registration-contact-form-7. C'est le bout de code qui synchronise l'enregistrement du nouvel adhérent dans le plugin func-snpnc. J'ai mis à jour le repository de ce fork. Du coup il faut mettre à jour ce fork dans le projet sandbox.snpnc.org.

j'ai également ajouté un message d'information dans l'onglet registration settings. Ce message stipule qu'à l'activation de l'option on active deux choses :

Modifier-le-formulaire-de-contact-‹-test-devweb-—-WordPress

maintenant toute la logique liée à la soumission du formulaire cf7 se trouve dans le plugin func-snpnc, précisément dans le fichier wordpress_hooks.php. Je pense qu'on peut encore faire mieux en ajoutant la logique de de cf7 présent dans le functions.php du thème. Mais on verra cela dans un second temps.

dans le traitement de la soumission du formulaire je fais appel à une autre fonction présente dans le fichier utils.php. Je fais mon mapping via ce code. La fonction native d'acf update_field fonctionne correctement.

je voulais mettre à jour le projet sur le serveur sandbox.snpnc.org, mais git n'est pas installé sur le serveur du coup je ne peux pas utiliser les commandes git pull et je ne peux pas faire de tests sur le serveur sandbox. Est ce possible d'installer la dépendance git sur le serveur stp @GuillaumeSorel ?

wyde22 commented 10 months ago

je compte 3.5h pour ce travail. Il y aura peut être un peu de temps supplémentaires pour les tests sur l'instance sandbox du projet. A échanger ensemble plutôt lundi après midi

allinappliadmin commented 10 months ago

j'ai cloné les 2 repo : le fork et snpnc j'ai regardé ton mapping dans utils.php et ça correspond bien j'ai fait une micro correction dans un commentaire, donc récupère la version de GitHub avant de bosser à nouveau. Pour l'instant la commande git ne fonctionne pas sur les users en chrooted, j'espère que ça va être le cas très rapidement. en attendant tu peux pousser sur github et me demander de mettre à jour, cela ne me dérange pas de le faire, c'est super rapide. j'ai laissé le repo de l'extension snpnc en public vu qu'il n'y a rien de sensible dedans.

allinappliadmin commented 10 months ago

je viens de tester et effectivement cela ne fonctionne pas encore. Le formulaire part bien mais la création du compte ne se fait pas. ça doit être lié à tes modifications en cours. le compte n'est pas créé et l'ajout aux listes de diffusion non plus, donc il y a quelque chose de l'ordre de la soumission à Wordpress des données qui bloque.

allinappliadmin commented 10 months ago

appelle-moi quand tu es dispo

allinappliadmin commented 10 months ago

et c'est noté pour ton temps

wyde22 commented 10 months ago

cela fonctionne maintenant sur le site sandbox. Il fallait faire un git pull des ressources du fork et aussi le plugin projet snpnc. Il fallait également activé le plugin projet afin que tout fonctionne correctement. La logique de création du rôle et du mapping se trouve dans ce plugin projet. J'ai fait un test avec un compte toto@titi.com

allinappliadmin commented 10 months ago

ah quel débile je fais ! j'avais forké les 2 projets mais pas activé...

wyde22 commented 10 months ago

pas de souci :)

Le lun. 20 nov. 2023 à 14:31, ALL IN APPLI admin @.***> a écrit :

ah quel débile je fais ! j'avais forké les 2 projets mais pas activé...

— Reply to this email directly, view it on GitHub https://github.com/ALLINAPPLI/snpnc/issues/50#issuecomment-1819069821, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADHVW7FSFFTJRW7JTTYWL2LYFNLSHAVCNFSM6AAAAAA7MI3RWSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJZGA3DSOBSGE . You are receiving this because you were assigned.Message ID: @.***>

-- Dewy Mercerais Tel : 06 63 33 84 91 Mail : @.*** Réseaux sociaux : linkedin https://www.linkedin.com/in/dewy-mercerais-aba83bb2/ Mon site web : https://dewy.fr/ Membre de Social Declik https://socialdeclik.com/, la communauté des freelances engagés 🌱

allinappliadmin commented 10 months ago

donc je confirme que ça fonctionne mieux quand c'est activé. je viens de faire plusieurs test et le champ 'grade' n'est pas récupéré je devrais être instructeur et ce n'est pas le cas, c'est la valeur par défaut 'CCP' qui est prise en compte. image

wyde22 commented 10 months ago

je te tel

Le lun. 20 nov. 2023 à 16:04, ALL IN APPLI admin @.***> a écrit :

donc je confirme que ça fonctionne mieux quand c'est activé. je viens de faire plusieurs test et le champ 'grade' n'est pas récupéré je devrais être instructeur et ce n'est pas le cas, c'est la valeur par défaut 'CCP' qui est prise en compte. [image: image] https://user-images.githubusercontent.com/62597730/284306028-e8777e7e-6196-497f-b3f2-85eb124aad53.png

— Reply to this email directly, view it on GitHub https://github.com/ALLINAPPLI/snpnc/issues/50#issuecomment-1819237765, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADHVW7FZKCE7YMQJDDE5X4TYFNWRDAVCNFSM6AAAAAA7MI3RWSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJZGIZTONZWGU . You are receiving this because you were assigned.Message ID: @.***>

-- Dewy Mercerais Tel : 06 63 33 84 91 Mail : @.*** Réseaux sociaux : linkedin https://www.linkedin.com/in/dewy-mercerais-aba83bb2/ Mon site web : https://dewy.fr/ Membre de Social Declik https://socialdeclik.com/, la communauté des freelances engagés 🌱

allinappliadmin commented 10 months ago

les deux champs liste déroulante sont configurés pareil dans ACF et la compagnie est bien prise en compte donc c'est peut-être du côté du CF7

[select compagnie first_as_label "-- Votre compagnie --" "Air Antilles" "Air Austral" "Air Calédonie" "Air Calin" "Air Caraïbes Atlantique" "Air Caraïbes Régional" "Air Corsica" "Air France" "Air France Tahiti" "Amelia" "ASL Airlines France" "Chalair" "Corsair" "easyJet" "Ewa Air" "Frenchbee" "Hop!" "La Compagnie" "Lux Aviation" "Norse" "Ryanair" "Transavia" "Travel Service" "TUIFly" "Volotea" "Vueling" "Autre" "Retraité"]

[select fonction first_as_label "-- Votre fonction --" "CCP" "Chef de cabine / Purser-Chief Purser" "Hôtesse-Steward / Cabin Crew" "Instructeur / Line Trainer"]