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

Angular 17 #1037

Open
wants to merge 33 commits into
base: master
Choose a base branch
from
Open

Angular 17 #1037

wants to merge 33 commits into from

Conversation

Tzahi12345
Copy link
Owner

@Tzahi12345 Tzahi12345 commented Dec 2, 2023 •
edited
Loading

Huge frontend update to Angular 17 from Angular 15.

This primarily will update all templates to comply with the new Angular control flow

  • Replaces all ngIf/ngFor with @if and @for syntax
  • Updates theming to work with new syntax

This refactor is worth it imo because it should provide significant build and runtime performance upgrades.

It also updates the TS version, but importantly requires Node.js v18

Tzahi12345 and others added 30 commits December 1, 2023 16:32
@Tzahi12345
Force ngx-avatars to use angular 16
d02d100
@Tzahi12345
Force update ngx-avatars
d10401c
@Tzahi12345
Temporarily removed ngx-avatars
caadf4f
@Tzahi12345
Updated to Angular 16
c3c8f50
@Tzahi12345
Updated Angular Material to v16
69d8751
@Tzahi12345
Updated to Angular 17
517c9e1
@Tzahi12345
Updated Angular Material to v17
f57e0ab
@Tzahi12345
Updated styles.scss to match new Angular syntax
7cf5d86 
Added back ngx-avatars

Made required dependency updates
@Tzahi12345
Added missing saveAs imports
6152df3
@Tzahi12345
Updated templates to new Angular control flow
25e4c11
@Tzahi12345
Updates version info/requirements in README
2c155b7
@Tzahi12345
Updates node version in CI
4a97fa4
@Tzahi12345
Updated docker frontend build node version to 18
9556f9c
@Tzahi12345
Frontend dev environment now uses non-prod mode by default
84464db
@Tzahi12345
Merge branch 'master' of https://github.com/Tzahi12345/YoutubeDL-Mate…
3912655 
…rial into angular-17
@Tzahi12345
Reverted using production as defaultConfiguration in angular.json
9b38c56
@Tzahi12345
Merge branch 'master' of https://github.com/Tzahi12345/YoutubeDL-Mate…
2cf2154 
…rial into angular-17
@Tzahi12345
mocha is now a backend dev dependency
dcb5369
@Tzahi12345
Install pm2 without npm
f96ffab
@Tzahi12345
Remove curl install from pm2
eb8cd3f
@Tzahi12345
Cleaned up pm2 installcommand
1bf348f
@Tzahi12345
Updated npm version in Dockerfile
026f24a
@Tzahi12345
Removed version from docker npm install
b4cf1e3
@Tzahi12345
Updated docker Ubuntu to 24.04
fb92975
@Tzahi12345
Updated UID/GID to 1001
7ac6a50
@Tzahi12345
Removed libicu70
4258b82
@Tzahi12345
Removed libicu70 from fetch-twitch-downloader
72399b0
@Tzahi12345
Downgraded ubuntu to 23.04
c56987d
@Tzahi12345
Trying removal of python
25bf7a6
@Tzahi12345
Added back python and added flag to pip to force install pycryptodomex
e2e3dd2
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
@Tzahi12345