Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error handling on ServerCards #829

Merged
merged 5 commits into from
Nov 6, 2023
Merged

Error handling on ServerCards #829

merged 5 commits into from
Nov 6, 2023

Conversation

ssrahn
Copy link
Collaborator

@ssrahn ssrahn commented Nov 3, 2023

#710

  • Nutzerkennung/Passwort wird korrekt behandelt
  • Eintragen von duplizierten URLs nicht möglich
  • URLs die nicht erreichbar sind schmeißen cURL error
    -> zB: cURL error 6: Could not resolve host: staging-oc-admin.uni-osnabrueck.d (see https://curl.haxx.se/libcurl/c/libcurl-errors.html) for https://staging-oc-admin.uni-osnabrueck.d/services/services.json
    • Message wurde abgegriffen und angepasst
  • LTI
    • LTI kann während der Konfiguration ohne weiteres nicht behandelt werden
      -> Fehlerhafte Eingaben möglich
    • LTI wird nun nach der Konfiguration geprüft
      • Bei Fehlschlag schließt sich Dialog nicht und zeigt Meldung an
    • Bei Aufruf der Admin Seite werden alle Server periodisch überprüft (wie in der VideoTable)
      • Bei Fehler wird rotes Ausrufezeichen in der Card angezeigt
      • Bei klick auf die Card wird eine entsprechende Meldung angezeigt
  • Workflows
    • Einstellungen der default Workflows erst nach Abschließung des Server-Dialogs möglich
      • Dialog wird nach "Speichern" für neuen Server nicht mehr geschlossen
      • Stattdessen wird Hinweis angezeigt und zu Standardworkflows gescrollt
  • Fehlerhafte Videocards visualisieren
    • Rotes ausrufezeichen Icon an VideoCard
    • Bis jetzt lti calls, sollte man da noch mehr prüfen?

@ssrahn ssrahn added v:3 Everything related to the Stud.IP Opencast Plugin Version 3.x type:visual-clarity labels Nov 3, 2023
@ssrahn ssrahn requested a review from tgloeggl November 3, 2023 11:14
@ssrahn ssrahn self-assigned this Nov 3, 2023
@tgloeggl tgloeggl merged commit 9cadc92 into master Nov 6, 2023
5 checks passed
@tgloeggl tgloeggl deleted the fix710 branch November 6, 2023 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:visual-clarity v:3 Everything related to the Stud.IP Opencast Plugin Version 3.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants