Continuando da Configuring Twitter login and rich embeds for Discourse - #114 by Hifihedgehog
Ho ricevuto un’email a riguardo ed ero molto confuso poiché non ricordavo di aver fatto un post del genere.
Sono curioso di sapere come sia stato scambiato “taylor” qui nella citazione di risposta? È un errore di modifica manuale (non sono sicuro di come fare un blocco di citazione con una fonte manualmente, o se ciò sia anche possibile). O c’è un bug da qualche parte nella logica di risposta? @tshenry
Ho appena provato a citare alcuni post e penso che stia succedendo qualcosa di strano con il nome utente e il nome completo. Probabilmente questo recente commit:
Ciao @JammyDodger,
puoi darmi maggiori informazioni su
Con le impostazioni del sito prioritize username in ux attivate e display name on posts disattivate, dovresti vedere il nome utente nella citazione, e viceversa per vedere il nome completo.
Mi dispiace, avrei potuto essere molto più utile lì. 
Ho appena provato a citare diversi post in questo argomento e ci sono stati alcuni risultati strani nel mio composer:
Post di @Hifihedgehog:
[quote=", post:114, topic:13395"]
hai ragione! Ci sono voluti un paio di minuti e sono stato immediatamente approvato. Le correzioni semplici sono sempre gradite! :smile:
[/quote]
Post di @tshenry:
[quote="Taylor, post:113, topic:13395"]
Puoi ottenere l'accesso all'API v1.1 gratuitamente con il piano Elevated. Devi semplicemente fare domanda dalla tua dashboard sviluppatore. L'ho trovato molto veloce e facile. I login di Twitter di Discourse hanno funzionato senza intoppi dopo di che :tada:
[/quote]
(quello è lo stile per cui @taylor ha ricevuto una notifica)
Ah, grazie per averlo segnalato… sì, questo non è desiderabile / previsto. Ci darò un’occhiata!
Quindi il motivo per cui @taylor è stato taggato in questa citazione (e ha ricevuto un’email) è perché il suo nome utente - Taylor è lo stesso del nome completo di @tshenry - Taylor 
che (il nome completo) ha la priorità corretta nelle citazioni a causa delle modifiche aggiunte in DEV: Prioritize full name when setting active (#15820) · discourse/discourse@5a93ce4 · GitHub.
Comunque, grazie per il suggerimento, lo sistemerò!
Per quanto ne so, lascia anche le citazioni senza nomi completi vuote:
[quote=", post:114, topic:13395"]hai ragione! Ci sono voluti un paio di minuti e sono stato approvato all’istante. Le correzioni semplici sono sempre gradite! 
[/quote]
È a posto così adesso @isaac?
Sì, abbiamo annullato la modifica e sto lavorando per correggere la logica prima di unire di nuovo.
Nell’aggiornare la logica delle citazioni per consentire l’attivazione/disattivazione di nome utente o nome completo tramite un’impostazione del sito, ho riscontrato un problema (parte del problema è stata notata sopra), permettetemi di creare uno scenario ipotetico…
Ripastare tutte le citazioni è anche fuori discussione. Se questa impostazione del sito fosse stata attivata/disattivata più volte e noi ripastassimo più volte, una citazione che viene convertita da un nome completo → nome utente non è univoca e potrebbe trovare più utenti.
La via che vedo per noi è aggiungere un attributo di dati aggiuntivo alla citazione di data-full-name="true" (o qualcosa di simile) per verificare se dovremmo sostituire il nome utente con il nome completo nell’UX, invece di manipolare data-username.
es.
data-username="always.users.username" data-post="1" data-topic="1" data-full-name="true"
e risparmiarci il fastidio di preoccuparci se stiamo interrogando o inviando notifiche sul nome utente o sul nome completo. Pensieri? Vale la pena aggiornare PrettyText e il nostro costruttore di citazioni per questa modifica?
Preferisco non aggiungere il supporto per il passaggio. Questa è una decisione che prendi all’inizio della configurazione della tua community, supportare un passaggio estremamente fluido è estremamente costoso e comporta una serie di compromessi.
Se ti cito con il nome utente prioritario, renderizziamo questo markdown
[quote="isaacjanzen, post:12, topic:217633"]
Thoughts?
[/quote]
Se ti cito con il nome prioritario, renderizziamo questo markdown
[quote="Isaac Janzen, post:12, topic:217633"]
Thoughts?
[/quote]
Bisogna fare attenzione con la virgola nei nomi completi, ma supporto la sua rimozione per questo caso d’uso o immagino che tu possa introdurre qualche trucco di escape.
Stai dicendo che consiglieresti di abbandonare il tentativo di aggiungere la possibilità di visualizzare le citazioni con nomi completi? Inevitabilmente ci imbatteremo in questi casi di passaggio di consegne… ad esempio, qualsiasi community esistente con una citazione
No, non completamente… meta ha disabilitato prioritizza il nome utente nell'esperienza utente. La citazione che ho appena fatto dovrebbe essere: (a cui manca un avatar che deve essere corretto)
Il markup qui dovrebbe essere:
[quote="Isaac Janzen, post:14, topic:217633"]
Stai dicendo che consiglieresti di abbandonare il tentativo di aggiungere la possibilità di visualizzare le citazioni con nomi completi?
[/quote]
L’unica modifica necessaria è far sì che il widget di citazione sia consapevole dell’impostazione in modo che riempia il nome completo invece del nome utente. Non preoccuparti della migrazione e della cronologia.
Non sono sicuro che siamo sulla stessa lunghezza d’onda qui… quando abbiamo unito DEV: Prioritize full name when setting active (#15820) · discourse/discourse@5a93ce4 · GitHub abbiamo ottenuto
# quote.js
# opts.displayName = true se le impostazioni del sito danno priorità al nome completo
const name = opts.displayName
? opts.name || post.name
: opts.username || post.username;
# costruisci il markdown della citazione
const params = [
name,
`post:${opts.post || post.post_number}`,
`topic:${opts.topic || post.topic_id}`,
];
per ottenere l’output di
ma il problema era che la ricerca dell’utente basata sul nome completo stava rompendo le citazioni esistenti che erano state create con lo username.
Quindi sembra che sia impossibile ignorare
Sono super confuso perché meta ha:
Allora perché il nome manca nella citazione sopra? La modifica è stata annullata?
Concordo, dobbiamo fare di più qui:
Dobbiamo recuperare il nome utente dalla combinazione “post / topic” e inserirlo nel markdown cotto (dovremmo farlo incondizionatamente), altrimenti non possiamo visualizzare gli avatar per il nome completo.
Ci consente anche di correggere questo caso limite, in modo che non venga visualizzato il mio avatar su questa citazione errata. (o addirittura evidenziare una citazione errata)
Ah, sì
Sono d’accordo e posso aggiungerlo.
Sono totalmente aperto a correggere alcuni casi limite mentre ci sono:
Penso che la cosa migliore da fare per ora sia semplicemente “sovrascrivere”. Se il post ha un nome / nome utente, usalo al posto di quello fornito nella citazione. Aggiorna con il nome corrente / nome utente corrente.
Va bene, ho un PR aperto che chiarisce le cose.
Ora passiamo il nome utente quando
siteSettings.display_name_on_posts && !siteSettings.prioritize_username_in_ux && post.name
così come il nome completo per garantire di non ottenere discrepanze durante l’interrogazione per nome utente / avatar.
ad es.
[quote="Isaac Janzen, post:3, topic:7, full:true, username:isaac.janzen"]
bing bong
[/quote]
Cosa ne pensi @sam?
Suppongo di poter accettare una modifica al markdown in modo da aggirare la necessità di addentrarci in un’enorme avventura di sicurezza. Tuttavia, la modifica deve essere iper-chirurgica.
Puoi confermare che la nuova sintassi prolissa si verifica solo ed esclusivamente se sia siteSettings.display_name_on_posts che !siteSettings.prioritize_username_in_ux sono attivi?
[quote="Isaac Janzen, post:3, topic:7, username:isaac.janzen"]
....
[/quote]
Ciò significa che se non abiliti sia display_name_on_posts SIA disabiliti prioritize_username_in_ux
Allora il vecchio formato di citazione dovrebbe rimanere:
[quote="isaac.janzen, post:3, topic:7"]
....
[/quote]
Sono disposto ad accettarlo come passo intermedio per ora, ma il nostro obiettivo a lungo termine sarebbe quello di rimuovere questo rumore dal blocco di citazione (rafforzare la sicurezza, limitare gli abusi) e supportare il formato di citazione abbreviato che abbiamo sempre avuto, affrontando il problema della sicurezza come siamo costretti a fare ora per le onebox inline, ad esempio: Quoting broken when name matches username - #20 by isaac ← https://meta.discourse.org/t/quoting-broken-when-name-matches-username/217633/20?u=sam
Menziono @tobiaseigen qui per visibilità poiché questo si collega a discussioni precedenti.
In conclusione:
[quote="TOTALE OPZIONALE SARÀ PESCATO DAL POST SE LA SICUREZZA SI ALLINEA E LE INFORMAZIONI SUL POST SONO PRIORITARIE RISPETTO A QUESTO TESTO, post:3, topic:7"]. - raccomandiamo di aspettare un po’ su questo.Unito e distribuito DEV: Prioritize full name when display_name_on_posts active by janzenisaac · Pull Request #16078 · discourse/discourse · GitHub
Posso confermare.