Skip to content

refactor(movies): update to ng17, rxa 17, eslint#280

Open
BioPhoton wants to merge 32 commits into
mainfrom
update-to-ng17
Open

refactor(movies): update to ng17, rxa 17, eslint#280
BioPhoton wants to merge 32 commits into
mainfrom
update-to-ng17

Conversation

@BioPhoton
Copy link
Copy Markdown
Member

@BioPhoton BioPhoton commented Nov 12, 2023

This PR includes:

  • Nx update v17
  • Angular update to v17
  • RxAngular update to v16.3.0
  • package.json overwrites
  • Refactoring of the state logic to functional style

Possible in a follow up PR:

  • update to new builder
  • bundle budget updates
  • webpage test measures

Comment thread projects/movies/src/app/pages/movie-list-page/movie-list-page.adapter.ts Outdated
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Nov 12, 2023

Visit the preview URL for this PR (updated for commit c998cac):

https://angular-movies-a12d3--update-to-ng17-w2uueccr.web.app

(expires Sun, 07 Apr 2024 12:51:22 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: f5e1819659ee450e0b7241d53391169f76c6f1d4

Comment thread package.json Outdated
Comment thread package.json Outdated
Comment thread package.json Outdated
Comment thread package.json Outdated
@eneajaho eneajaho mentioned this pull request Nov 12, 2023
Comment thread projects/movies/src/app/ui/pattern/star-rating/star-rating.component.ts Outdated
@JeanMeche
Copy link
Copy Markdown
Collaborator

Can we look to get the v17 migration merged ? :)

@eneajaho eneajaho changed the title refactor(movies): update to ng17, rxa 16.3.0, eslint refactor(movies): update to ng17, rxa 17, eslint Mar 8, 2024
eneajaho added 2 commits March 8, 2024 13:47
refactor(movies): migrate to use application builder SSR features
@eneajaho
Copy link
Copy Markdown
Collaborator

eneajaho commented Mar 8, 2024

Icons are not working on Firebase. But they work locally. Probably because of prerendering happening at build time.

@hoebbelsB
Copy link
Copy Markdown
Collaborator

any updates on this? :D

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.

6 participants