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

add user settings module #280

Merged
merged 5 commits into from
May 6, 2021
Merged

add user settings module #280

merged 5 commits into from
May 6, 2021

Conversation

spahrson
Copy link
Contributor

Implementation of #274 and #272.

@spahrson spahrson requested a review from caebr April 27, 2021 12:30
@spahrson spahrson force-pushed the feature/272-user-settings branch 2 times, most recently from 46aea3a to 3899223 Compare April 27, 2021 19:23
@caebr
Copy link
Collaborator

caebr commented Apr 30, 2021

Zum GUI:

  • In der mobile Ansicht fehlt noch das Padding links und rechts; die Elemente kleben am Rand
  • Das Klicktarget sollte auf der Checkbox und dem Label sein
  • In der Deskopansicht schwebt dafür der Button ein bisschen im Nichts. Vielleicht würden Linien helfen? Ähnlich wie bei http://localhost:4200/#/open-absences/confirm im Edit-Screen

@caebr
Copy link
Collaborator

caebr commented Apr 30, 2021

Fragen:

  • Sind die Schüler auch berechtigt, denn neuen Bereich zu sehen?

@spahrson
Copy link
Contributor Author

spahrson commented May 5, 2021

Fragen:

  • Sind die Schüler auch berechtigt, denn neuen Bereich zu sehen?

Das ist eine sehr gute Frage. Ich weiss es nicht. Ich werde das morgen am Daily ansprechen.

@schefbi
Copy link
Contributor

schefbi commented May 6, 2021

@spahrson Ja, userSettings sind für alle Webbenutzenden.

@spahrson spahrson requested a review from caebr May 6, 2021 10:59
@spahrson spahrson merged commit 0a1e201 into master May 6, 2021
@mfehlmann mfehlmann deleted the feature/272-user-settings branch April 21, 2022 11:42
@mfehlmann mfehlmann restored the feature/272-user-settings branch April 21, 2022 11:42
@mfehlmann mfehlmann deleted the feature/272-user-settings branch April 21, 2022 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Neue Seite Einstellungen (settings) Komponente Benutzereinstellungen (/UserSettings/Cst)
3 participants