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

O365: Office Integration finalisieren #6231

Open
jone opened this issue Feb 3, 2020 · 3 comments
Open

O365: Office Integration finalisieren #6231

jone opened this issue Feb 3, 2020 · 3 comments
Labels
Milestone

Comments

@jone
Copy link
Member

@jone jone commented Feb 3, 2020

Harvest: [0.322.3 / 40] Projekt - Integration Office Online in OneGov GEVER (OGIP 58) - Implementation WOPI-Schnittstelle

Ursprung: #6189

Ziel

Ziel ist es, die Office Online Integration mit WOPI zu finaliseren. Die Implementation steht im PR #5993 bereits zur Verfügung.

Was ist noch nötig an Tests, Dokumentation, etc.?

Todos

  • PR #5993 reviewen

    • Für ein Review ist es vermutlich nötig, den branch auszuchecken und zu testen
    • Zum testen gibt es zwei Möglichkeiten:
      • Lokal, mit einem reverse tunnel via rlab
      • Auf lab.onegovgever.ch ab branch deployen (müsste zuerst aktualisiert werden)
      • Deployment ist in der Doku beschrieben
    • GEVER-seitig müssen zum Installieren zwei Registry-Einträge gesetzt werden, IWOPISettings.discovery_url und IWOPISettings.enabled)
  • PR mergen und auf dev deployen und konfigurieren (PO macht das Review auf dev)

  • Für das Deployen auf dev muss evtl. noch die Domain freigeschaltet werden

@jone jone mentioned this issue Feb 3, 2020
2 of 2 tasks complete
@lukasgraf

This comment has been minimized.

Copy link
Member

@lukasgraf lukasgraf commented Feb 3, 2020

@jone
Für ein Refinement braucht es aus meiner Sicht noch folgendes:

  • Eine Beschreibung (High-Level), wie die Integration mit OO / WOPI funktioniert.
    Wenn eine solche Beschreibung schon existiert (im OGIP, oder in Doku von Microsoft), bitte entweder hier in diese Issue reinkopieren oder entsprechend verlinken.

  • Der PR #5993 muss finalisiert werden, damit klar ist welche Arbeiten schon gemacht sind, und welche noch ausstehen:

    • PR-Checkliste anwenden, und fehlende Punkte ergänzen oder begründen.

    • Technische Dokumentation scheint zu fehlen. Wir brauchen etwas in einem ähnlichen Stil wie die Doku für die OneOffix Integration.

    • Angabe dazu, was in diesem PR genau umgesetzt wird, und was nicht. U.a.:

      • Welche Aspekte der WOPI durch den PR implementiert werden, und welche nicht (ggf. aus dem OGIP kopieren, falls diese Angaben noch stimmen)
      • Ob die Implementierung für das alte Frontend, das neue, oder beides ist. Bzw. was bezüglich des User-Interfaces in diesem PR gemacht wurde, und ob noch etwas fehlt.
    • Wie die Fehlerbehandlung gemacht wurde, und welche Edge-Cases möglicherweise noch problematisch sein könnten.

    • Erläuterung dazu, wie der PR getestet werden kann, bevor er gemerged wird. Lokal wird wahrscheinlich schwierig, wenn der WOPI Host von aussen erreichbar sein muss. Wie kann der PR stattdessen getestet werden?

Danach sollten wir in der Lage sein, zu bestimmen was jetzt genau noch fehlt / gemacht werden muss.

@jone

This comment has been minimized.

Copy link
Member Author

@jone jone commented Feb 17, 2020

@lukasgraf der #5993 wurde aktualisiert. Könntest du nochmals eine Einschätzung machen?

@lukasgraf

This comment has been minimized.

Copy link
Member

@lukasgraf lukasgraf commented Feb 17, 2020

@jone der PR sieht aus meiner Sicht gut aus!
Ich habe diesen Issue, und die anderen beiden, noch mininal ergänzt. Die Vorschläge sind nicht viel detailierter als "Wir müssen uns noch anschauen wie wir dies und das lösen können", aber müsste aus meiner Sicht für ein Refinement reichen.

ready-for-refinement (weiss nicht, wie genau Du das mit den Labels handhaben willst)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.