Nome file ICS non definito

[dax (Daniela)]

Panoramica

Quando si fa clic su “Aggiungi al calendario” nella finestra di anteprima dell’evento (quella che appare dopo aver fatto clic sulla data dell’evento):

il file .ics scaricato viene denominato undefined.ics e anche il titolo dell’evento all’interno del file del calendario è impostato su SUMMARY:undefined. Tuttavia, il download del calendario tramite l’opzione “Aggiungi al calendario” dal menu a 3 punti dell’evento funziona come previsto, utilizzando il titolo dell’evento sia per il nome del file che per il riepilogo del calendario.

Passaggi per riprodurre

1. Creare o aprire un argomento con un evento
2. Fare clic sulla data dell’evento mostrata nel post per espandere la finestra di anteprima
3. Nella finestra, fare clic su Aggiungi al calendario
4. Salvare il file .ics generato.
5. Facoltativamente, confrontare facendo clic sul menu a 3 punti dell’evento e utilizzando “Aggiungi al calendario” da lì

Risultati attesi

• Il file .ics scaricato dovrebbe essere denominato in base al titolo dell’evento
• Il contenuto del file del calendario dovrebbe avere un SUMMARY: corretto con il titolo dell’evento

Risultati osservati

• Il file scaricato è denominato undefined.ics
• Il titolo dell’evento nel file del calendario è SUMMARY:undefined
• (Quando scaricato dal menu a 3 punti, sia il nome del file che il riepilogo sono corretti.)

Contesto aggiuntivo

• Esempio di contenuto ICS non valido:

  BEGIN:VCALENDAR
  VERSION:2.0
  PRODID:-//Discourse//EN
  BEGIN:VEVENT
  UID:1762794000000_1762801200000
  DTSTAMP:20251105T173754Z
  DTSTART:20251110T170000Z
  DTEND:20251110T190000Z
  SUMMARY:undefined
  END:VEVENT
  END:VCALENDAR
  

Testato su Meta e diversi altri siti Discourse, stesso risultato.

[sam (Sam Saffron)]

Questo è un caso difficile, Dax, che è un effetto collaterale della nostra pipeline.

Generiamo il bbcode per le date qui:

plugins/discourse-calendar/assets/javascripts/discourse/components/discourse-post-event/dates.gjs

E lo elaboriamo qui:

plugins/discourse-calendar/assets/javascripts/discourse/components/discourse-post-event/dates.gjs

Quindi, nel contesto del blocco HTML elaborato, il pulsante “scarica ics” non è a conoscenza del post effettivo a cui appartiene (o dell’evento).

Abbiamo anche una pipeline diversa per la generazione di ics qui:

frontend/discourse/app/lib/download-calendar.js

Quindi, dobbiamo decidere da una prospettiva ingegneristica se:

1. Insegnare al “date cooking” come reindirizzare la generazione di ics a Discourse Calendar.

O

2. Fornire abbastanza contesto a Discourse Local Dates, in modo che possa generare autonomamente l’ics e mantenere il codice frammentato.

Non sono sicuro di quale sia la cosa giusta da fare qui, ma l’ho prioritarizzata in modo che il team possa analizzarla e risolverla.

[Ric (Ric Harvey)]

Ciao a entrambi per aggiungere un po’ di contesto, se fai clic sui tre puntini su un evento c’è un’opzione Aggiungi al calendario e questa funziona. Non so se questo può aiutarvi a indagare, ma sembra che sia stato risolto altrove nel codice.

[sam (Sam Saffron)]

C’è MOLTO qui:

FIX: "add to calendar" not providing correct ics

È venerdì (almeno da qualche parte ;p) quindi aspetterò lunedì per unire.

Questa modifica è incredibilmente estesa e dovrebbe darci un supporto ICS significativamente migliore.

• Unifica la pipeline per la generazione di ICS: utilizziamo un solo meccanismo sia per l’aggiunta al calendario che per il clic sulle date
• Corregge molti piccoli dettagli di sfumatura nel formato ics
  • Passiamo RRULE quindi se prendi un evento ricorrente
  • Corrette interruzioni di riga CRLF e aderenza generale al formato ICS
  • Supporto del fuso orario quindi quando prendi un ICS per un evento segnalerà il fuso orario corretto invece di essere un evento UTC - questo significa che la ricorrenza funzionerà.
• Espande il formato delle date locali per supportare un ics codificato opzionalmente

Una domanda aperta che ho è sì, rrule o no, rrule.

Se fai clic qui:

Intendiamo aggiungere l’evento ricorrente? O solo una singola istanza dell’evento?

Allo stesso modo, che dire di qui:

@lindsey Sono indeciso qui, posso vedere argomentato in entrambi i modi.

1. Ho fatto clic su un evento ricorrente e ho voluto aggiungere la ricorrenza al mio calendario

OPPURE

2. Ho fatto clic su un’ISTANZA di una ricorrenza e voglio solo aggiungerla.

Ho implementato (1) perché tendo a sentirmi più corretto, ma sono aperto a cambiarlo in 2 se preferisci.

[lindsey (Lindsey Fogle)]

@sam

1. Ho cliccato su un evento ricorrente e ho voluto aggiungere la ricorrenza al mio calendario

@sam

Ho implementato (1) perché tendo a ritenerlo più corretto, ma sono aperto a cambiarlo in 2 se preferisci.

Posso capire l’argomentazione in entrambe le direzioni, ma preferisco anch’io la 1. Penso che sia sia più corretta, sia più facile da “correggere” se non fosse ciò che l’utente voleva, perché la maggior parte dei software di calendario rende molto facile eliminare eventi extra con una singola azione (come Google Calendar):

Quindi il fastidio di:

• Non volevo rispondere a tutti gli eventi, quindi devo rimuovere quelli extra

È molto inferiore al fastidio di:

• Volevo rispondere a tutti gli eventi, quindi devo tornare qui ogni settimana e assicurarmi di continuare ad aggiungerli al mio calendario

[sam (Sam Saffron)]

Fantastico, ho mantenuto la RSVP a tutti.

È stato unito oggi