Lo script ‘Aggiungi utente al gruppo….’ ha un input di campo diverso per i due trigger

Guardando lo script “Aggiungi utente a gruppo tramite campo personalizzato utente”, sembra esserci un piccolo problema. Se viene selezionato il trigger “Primo accesso utente”, l’input “Nome campo personalizzato utente” dovrebbe essere impostato sul nome del campo utente, ad esempio “Lingua nativa”. Ma se viene selezionato il trigger “Ricorrente”, l’input “Nome campo personalizzato utente” deve essere impostato in base all’id del campo utente. Ad esempio “user_field_5”.

Dettagli su come trovare il nome per quando viene utilizzato il trigger Ricorrente sono qui: Finding Userfield Custom Name for 'Add user to group through User Custom Field' - #2 by leonardo.

Idealmente, il nome visualizzato (ad esempio “Lingua nativa”) verrebbe utilizzato come valore corretto per entrambi i trigger disponibili. Il codice pertinente è qui: https://github.com/discourse/discourse-automation/blob/main/lib/discourse_automation/scripts/add_user_to_group_through_custom_field.rb. Il modo in cui il trigger “Primo accesso utente” trova il corretto campo personalizzato utente dal nome del campo (invece dell’id) è qui:

https://github.com/discourse/discourse-automation/blob/main/lib/discourse_automation/scripts/add_user_to_group_through_custom_field.rb#L54-L67

Forse c’è un modo retrocompatibile in cui quella logica potrebbe essere applicata al trigger Ricorrente.

3 Mi Piace

Tieni presente che questo è menzionato nel walkthrough:

Ho menzionato la rielaborazione dello script per coerenza, anche se non è ancora stata considerata prioritaria. Ci sono anche le impostazioni di amministrazione public user custom fields e staff user custom fields che sarebbe bello allineare ad esso.

(E c’è anche poll groupable user fields, che utilizza il ‘friendly name’)

4 Mi Piace

Ecco una PR per correggere il problema:

2 Mi Piace

Questo è stato ora risolto con un nuovo menu a discesa per selezionare i tuoi User Custom Fields esistenti, che funziona per entrambi i trigger: :partying_face:

2 Mi Piace