Problema breadcrump pagina foglia 'Le persone'

[Cate55]

Prerequisiti

• Ho già cercato tra le issue la presenza di un problema simile
• Ho letto le linee guida di contribuzione

Descrivi il problema

Buongiorno, vorrei evidenziare che l'ultimo fix al breadcrumb "Home/Scuola/Le persone/author" ha generato un errore 404 sulla pagina foglia.
La correzione proposta coinvolge una modifica al codice, precisamente alla riga 414 del file inc/breadcrumb.php:

-$this->items[] = "<a href='".home_url("la-scuola/persone")."'>".__("Le persone", "design_scuole_italia")."</a>";
+$this->items[] = "<a href='".home_url("la-scuola/le-persone")."'>".__("Le persone", "design_scuole_italia")."</a>";

Questa correzione sostituisce la parte finale dell'URL da "persone" a "le-persone". L'obiettivo è risolvere il problema associato al breadcrumb, prevenendo così l'errore 404.

Includi link o screenshot

Su quale sisitema operativo stai riscontrando il problema?

Windows

Su quale browser stai riscontrando il problema?

Chrome, Firefox, Microsoft Edge

Che versione del tema stai utilizzando?

2.7.0

[rcoopcsc]

Ciao @Cate55 fai tu la PR per questo fix?

[Cate55]

Ciao @Cate55 fai tu la PR per questo fix?

Volentieri, spero solo di riuscire e non combinare pasticci! Grazie

[Cate55]

Ciao @Cate55 fai tu la PR per questo fix?

Ciao @rcoopcsc ho creato la PR, credo sia in attesa di revisione, spero di non aver sbagliato! Grazie per tutto

[rcoopcsc]

@Cate55 non trovo la pull a cui ti riferisci se puoi inserisci il link in un messaggio grazie!

[Cate55]

@Cate55 non trovo la pull a cui ti riferisci se puoi inserisci il link in un messaggio grazie!

@rcoopcsc Non sono un collaborator e mi dava "Pull request creation failed. Validation failed: must be a collaborator"
Ho optato "Open a pull request that is ready for review" ma non ha sortito alcun effetto! Ho ricreato nuovamente e mi sono accorta del mio errore non avevo selezionato il branch
Cate55#1
Grazie!

[rcoopcsc]

@Cate55 ci siamo quasi. Solo controlla perchè hai fatto la PR alla tua stessa repository Cate55#1 mentre quello che vuoi in realtà è farla alla repository di designers italia. In altre parole, alla fine, devi vederla qui: https://github.com/italia/design-scuole-wordpress-theme/pulls/
Operativamente, se posso darti un consiglio procederei in questo modo: innanzitutto sincronizza il tuo fork, in modo che il tuo main non sia indietro rispetto alla versione corrente; poi crea un branch e fai lì la tua modifica. Salvalo, e poi quando è pronto fai la pull da quel branch alla repo di designers italia. Inserisci nel messaggio di commit cosa fa e cosa risolve, et voilà. Attendiamo fiduciosi :-)

[Cate55]

@Cate55 ci siamo quasi. Solo controlla perchè hai fatto la PR alla tua stessa repository Cate55#1 mentre quello che vuoi in realtà è farla alla repository di designers italia. In altre parole, alla fine, devi vederla qui: https://github.com/italia/design-scuole-wordpress-theme/pulls/ Operativamente, se posso darti un consiglio procederei in questo modo: innanzitutto sincronizza il tuo fork, in modo che il tuo main non sia indietro rispetto alla versione corrente; poi crea un branch e fai lì la tua modifica. Salvalo, e poi quando è pronto fai la pull da quel branch alla repo di designers italia. Inserisci nel messaggio di commit cosa fa e cosa risolve, et voilà. Attendiamo fiduciosi :-)

Perchè sono così imbranata!!Ho sincronizzato, ma non riesco a far apparire la PR! Studio ancora un po', grazie per la troppa pazienza :-)

[rcoopcsc]

Ottimo lavoro @Cate55 grazie!