Skip to content
This repository has been archived by the owner on Oct 4, 2024. It is now read-only.

Farbeinstellungen fertig #143

Merged
merged 9 commits into from
May 12, 2024
Merged

Farbeinstellungen fertig #143

merged 9 commits into from
May 12, 2024

Conversation

Lica200
Copy link
Collaborator

@Lica200 Lica200 commented May 12, 2024

No description provided.

Copy link

codecov bot commented May 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 65.19%. Comparing base (56b69d2) to head (85a1855).
Report is 11 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #143      +/-   ##
============================================
- Coverage     68.35%   65.19%   -3.16%     
  Complexity      181      181              
============================================
  Files            49       51       +2     
  Lines           534      566      +32     
  Branches         15       16       +1     
============================================
+ Hits            365      369       +4     
- Misses          160      188      +28     
  Partials          9        9              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Drumber Drumber self-requested a review May 12, 2024 13:33
- Moved logic for updating theme preference to user.service
- Created types for user settings and color modes
- Updated list style
Copy link
Collaborator

@Drumber Drumber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool. jetzt haben wir endlich verschiedene Themes 🎉

Ich habe noch paar Änderungen bezüglich dem Updaten der User Settings vorgenommen und das Styling etwas angepasst.

✅ Code-QA ist hiermit abgeschlossen.

@Drumber Drumber merged commit 6dae2be into main May 12, 2024
3 of 4 checks passed
@Drumber Drumber deleted the johanna branch May 14, 2024 06:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants