search

RRZE-Webteam / rrze-shorturl

WordPress-Plugin to enable a shorturl redirection service run on two servers (WP + PHP-only)
https://www.shorturl.rrze.fau.de
GNU General Public License v3.0
0 stars 1 forks source link

2Dos aus VC mit WW #89

Closed thenickless closed 4 weeks ago

thenickless commented 1 month ago
    • [x] edit URI in [shorturl-list] fehlt
    • [ ] Datum prüfen. Ist nach Update korrekt in DB und Ausgabe unter Berücksichtung die Rechte (max 1y vs 5y) in jedem Fall (shorten, edit via liste, endpoints)? Oder stimmt nur die Ausgabe in der Liste nicht? Update: SELECT-Rechte auf DB wurden mir jetzt für wm-Kennung (nicht IdM!) erteilt.
    • [x] In Ausgabe von [shorturl-list]: clickable links -> long2long, aber short+ für die Preview
    • [x] Input LongURL Prüfung stimmt nicht ganz. Bei "abc" soll "keine gültige Domain" ausgeben werden und nicht "Service ist unbekannt"
    • [x] Klare Unterscheidung Error/Notice : braucht LongURL nicht geshortet zu werden, weil Service: Notice. "Error: " löschen. Farben via CSS-Class (derzeit ist nur error umgesetzt) steuern für die Umrandung von Input bzw generell den Ablauf f Barrierefreiheit
    • [ ] In Ausgabe von [shorturl-list]: da jetzt die Zuordnung long:short nicht unique ist: Ausgabe aller shorts zu einer long und das am besten LIFO sortiert. (IdM weiterhin berücksichtigend)
thenickless commented 1 month ago

Subtask 1 "edit URI in [shorturl-list] fehlt":

Das fehlt nicht, sondern das geht beabsichtigt nicht. Genauso wenig wie genierte paths zu editieren. User sollen einfach den Link löschen und neu anlegen, wenn sie einen Custom URI falsch gesetzt haben. Die gleiche Vorgehensweise wie bei geniertem URI.

Eine andere Umsetzung ist konzeptionell nicht bedacht worden und würde an vielen Stellen in Code Änderungen erfordern. Aufwand ist hier abzuwägen. Wir warten am besten ab, wie oft das vorkommt und ob die User mit der derzeitigen Lösung glücklich sind.

Subtask ist damit closed. Sollte sich ein Bedarf ergeben, wird ein neues Issue angelegt.

thenickless commented 4 weeks ago

closed. subtask is transferred to milestone 2 (or a nice-to-have in 1) https://github.com/RRZE-Webteam/rrze-shorturl/issues/103