Skip to content

Feat/refactor signals state#196

Open
maxine-mwanda wants to merge 2 commits intoitalanta:mainfrom
maxine-mwanda:feat/refactor-signals-state
Open

Feat/refactor signals state#196
maxine-mwanda wants to merge 2 commits intoitalanta:mainfrom
maxine-mwanda:feat/refactor-signals-state

Conversation

@maxine-mwanda
Copy link

What was done

  • Refactored SelectBudgetPageComponent and BudgetTableComponent to use Angular Signals instead of RxJS Observables.
  • Removed all subscriptions and SubSink usage.
  • Added effect() to reactively update the table data.
  • Added computed() to derive overview budgets.

Benefits of Signals over Observables in this UI

  • Cleaner, more declarative code.
  • Reduced boilerplate and less chance of memory leaks from forgotten subscriptions.

Difference in coding style

  • Signals simplify state flow in component hierarchies, especially for parent → child data passing.

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.

1 participant