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.

Sign up for GitHub

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

Jump to bottom

TODOs for vue3 dashboard #102

Closed
18 tasks done
C0ntroller opened this issue Jun 27, 2022 · 1 comment · Fixed by #103
Closed
18 tasks done

TODOs for vue3 dashboard #102

C0ntroller opened this issue Jun 27, 2022 · 1 comment · Fixed by #103
Assignees
@C0ntroller @Noxdor
Labels
discussion Left open for discussion enhancement New feature or request testing wanted Part of this pull request requires further testing

Comments

@C0ntroller
Copy link
Member

C0ntroller commented Jun 27, 2022 •
edited
Loading

Vor dem merge:

  • Logins für nicht-ZIH @Noxdor
  • Funktionalität für Auswahl Fakultät @C0ntroller
  • Funktionalität für Auswahl Raketen @C0ntroller
  • Nur freigeschaltete Raketen anzeigen @C0ntroller @Noxdor
  • Funktionalität für extra E-Mail Benachrichtigungen (broken) @C0ntroller
  • Alle Einstellungen vom onboarding übernehmen @Noxdor
  • Debug Logs entfernen @Noxdor
  • Leerzeichen im onboarding fixen @Noxdor
  • Typo in "Opalkurse im Dashboard" ("Öffne" groß schreiben) @Noxdor
  • "Abkürzungen für Suchmaschinen": OPAL klein schreiben auf linker Seite des Pfeils @Noxdor
  • src/css löschen @Noxdor (gibt es aus Versehen usage irgendwo?)
  • "saved_click_counter" -> "savedClickCounter" @C0ntroller
  • Nochmal alle chrome-API Befehle anschauen und an restlichen Codestyle angleichen @C0ntroller
  • extra Benachrichtigung für E-Mail braucht Rechte für notifications
  • Rocket Icon Migration @C0ntroller
  • Dieses Issue. Wir müssen mindestens den Befehl inder Pipeline anpassen.
  • Aus der Pipeline rausnehmen, dass wir den Build mit 3 unterschiedlichen Node-Versionen testen. Ist eher unnötig. Oder @OliEfr ?
  • Versionsnummer auf 7 erhöhen
@C0ntroller C0ntroller added enhancement New feature or request discussion Left open for discussion testing wanted Part of this pull request requires further testing labels Jun 27, 2022
C0ntroller added a commit that referenced this issue Aug 2, 2022
@C0ntroller
Fix rocket icon setting
ef65cb2 
We need a migration for this -> #102
C0ntroller added a commit that referenced this issue Aug 3, 2022
@C0ntroller
Fixing all the rocket stuff (#102)
31cb8b7
@C0ntroller C0ntroller mentioned this issue Aug 3, 2022
Merged
@OliEfr
Copy link
Member

OliEfr commented Aug 3, 2022

Aus der Pipeline rausnehmen, dass wir den Build mit 3 unterschiedlichen Node-Versionen testen. Ist eher unnötig. Oder @OliEfr ?

Ja, nur auf relevanter node-version testen ist ausreichend

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@C0ntroller C0ntroller

@Noxdor Noxdor

Labels
discussion Left open for discussion enhancement New feature or request testing wanted Part of this pull request requires further testing
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Dev vue merge TUfast-TUD/TUfast_TUD
3 participants
@OliEfr @C0ntroller @Noxdor