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

Migrate to built-in control flow #705 #742

Closed
wants to merge 2 commits into from
Closed

Conversation

hupf
Copy link
Collaborator

@hupf hupf commented Oct 16, 2024

Siehe #705

  • Automatische Control Flow Schema Migration (ng generate @angular/core:control-flow) ausführen
  • @for ... trackBy konkret spezifizieren wo es möglich ist (mit unique ID)
  • ngFor und ngIf ersetzen wo nicht automatisch möglich
  • Custom *bkdLet="..." mit @let ersetzen → erfodert Update auf Angular 18.1
  • Testen ob alles noch funktioniert → am besten gerade mit dem Stand von Dependencies updaten - 0.5PT #724, damit man nur 1x testen muss.

@hupf hupf self-assigned this Oct 16, 2024
@hupf hupf force-pushed the feature/705-control-flow branch from 46cc50a to ee6d0a3 Compare October 16, 2024 08:20
@hupf hupf force-pushed the feature/705-control-flow branch from ee6d0a3 to f884c85 Compare October 16, 2024 11:30
@hupf hupf mentioned this pull request Oct 17, 2024
@hupf hupf changed the title Draft: Migrate to built-in control flow #705 Migrate to built-in control flow #705 Oct 17, 2024
@hupf hupf requested a review from caebr October 17, 2024 14:39
@hupf hupf force-pushed the feature/705-control-flow branch from 9e54db6 to 519cc85 Compare October 17, 2024 14:41
@caebr
Copy link
Collaborator

caebr commented Oct 23, 2024

Mir ist beim Testen noch Folgendes aufgefallen:

  • Offene Absenzen: Wenn gesucht wird und es kein Resultat gibt, verschindet das Suchfeld
  • Absenzen melden: Wenn sich der Datepicker öffnet, springt der Content hoch und verschwindet teilweise hinter dem Header

@hupf
Copy link
Collaborator Author

hupf commented Nov 25, 2024

Mir ist beim Testen noch Folgendes aufgefallen:

* Offene Absenzen: Wenn gesucht wird und es kein Resultat gibt, verschindet das Suchfeld

* Absenzen melden:  Wenn sich der Datepicker öffnet,  springt der Content hoch und verschwindet teilweise hinter dem Header

Ich habe die beiden Sachen gefixt. Das zweite war wohl schon länger so und hat einfach die Einbettung beim Standalone betroffen – das konnte ich anpassen.

@hupf
Copy link
Collaborator Author

hupf commented Nov 25, 2024

Ich schliesse diesen PR, damit ich die Changes gleich mit #743 mergen kann.

@hupf hupf closed this Nov 25, 2024
@hupf hupf deleted the feature/705-control-flow branch December 18, 2024 09:30
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.

2 participants