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

Update to Angular 17 #228

Draft
wants to merge 38 commits into
base: master
Choose a base branch
from
Draft

Update to Angular 17 #228

wants to merge 38 commits into from

Conversation

PhilipAB
Copy link
Contributor

@PhilipAB PhilipAB commented Oct 25, 2024 •
edited
Loading

What this PR does / why we need it:
This PR updates Angular to v17 (hobbyfarm/hobbyfarm#455). Moreover, it introduces theming (dark/light/system) for the Admin UI.

  • Update to Angular 17
  • Theme settings (dark/light/system)
  • Rework markdown parsing because parser now might return an async response
  • Update ngx-dynamic-hooks to v3
  • Migrate to @if template syntax
  • Testing the update on a dev server
  • Fix glossary component: Tooltips at the bottom of the screen are currenlty cut off

PhilipAB added 30 commits October 22, 2024 12:39
Update ts version
91159d0
Update angular version
a715915
Update clarity version
fb0cc67
Refactor markdown component
6411f6a
Update cds core and add style sheets
0e36956
Add theme settings
9526918
Update @angular/cdk
c325771
Fix markdown rendering
9fca846
Set background color via variable to consider theme changes
e791f1b
Implement custom text tooltip component
4621ac3
Add prettier as dev dependency
7065709
Merge branch 'master' into ng-17
7a9fc32
Add lint-staged as dev dependency
ac60884
Migrate code to new @if/@for/@switch syntax
b484fb9
Update angular-split version
7b1cb2c
Migrate ngx-dynamic-hooks to v3
1dd068d
Replace deprecated xterm packages
a9b9f4c
Fix path to xterm style sheet
1ca5f41
remove unused Web Components polyfills
7c50fc7
update @auth0/angular-jwt
2031efd
update ng2-charts
2349d12
update packages
638b7fb
run nmp audit fix
a7a3484
Add authn service. Improve theme handler
1ae0130
Minor fix
f875dfa
Ensure that note component always receives valid type
64d5d4d
Enable strict null checks & function types
6fac9d2
Fix bug that steps do not load initially
0ff44df
Remove unused component
e87604f
Fix: Reference to scenario needed to correctly update steps
d1fa750
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@PhilipAB