feat: add ics file drag&drop - Githubissues

JM-Lemmi / ical-relay

Relay ical urls and edit them on the fly with different modules.

GNU Affero General Public License v3.0

9 stars 2 forks source link

feat: add ics file drag&drop #232

Closed nilsfast closed 9 months ago

nilsfast commented 9 months ago

Fixes #68

To do:

[ ] Add hint in front end that import via DnD is possible

JM-Lemmi commented 9 months ago

Man könnte auch für den "plus" Button unten rechts auch ein "Datei upload" hinzufügen. Und dort einen "wusstest du: es geht auch über drag&drop" hinweis anzeigen, während das File selection Fenster offen ist.

nilsfast commented 9 months ago

@JM-Lemmi vielleicht noch die alte API.js gecached?

JM-Lemmi commented 9 months ago

Nochmal ganz neu gebaut, Firefox geleert und ohne Cache neugeladen, jetzt ist der Fehler tatsächlich ein anderer.

nilsfast commented 9 months ago

Mal wieder das alt-bekannte Problem mit den gespeicherten Profiles, die in die URL eingesetzt werden. Es zeigt jetzt einen Fehler im Frontend an.

JM-Lemmi commented 9 months ago

Warum muss denn ein Profil hinterlegt sein?

nilsfast commented 9 months ago

Muss eigentlich nicht. Einfacher wäre wahrscheinlich das zu nehmen, was gerade angezeigt wird.

JM-Lemmi commented 9 months ago

Je länger ich überlege, desto eher ist der File Size check an dieser Stelle sowieso quatsch. Die Begrenzung sollte technische limits haben, nämlich die postgres größe. Alles andere wird von der WAF/dem Proxy erledigt.

nilsfast commented 9 months ago

Die Fehlermeldung im Frontend ist auch nicht sehr aussagekräftig. Hier vielleicht den Status Code und Body aus der Response auch anzeigen? Zumindest als zweite Zeile oder so

So gut ist leider die API Class nicht, ich mache das jetzt über Standard fetch

JM-Lemmi commented 9 months ago

scheint jetzt zu funktionieren, allerdings zeigt das Frontend den neuen Termin nicht an und die ICS bekommt mit jedem Aufruf den Termin ein weiteres mal dazu. Ich denke das ist ein Problem an einer anderen Stelle.