Skip to content

Answer:30#1455

Open
phhien203 wants to merge 2 commits intotomalaforge:mainfrom
phhien203:phhien203-interop-rxjs-signal
Open

Answer:30#1455
phhien203 wants to merge 2 commits intotomalaforge:mainfrom
phhien203:phhien203-interop-rxjs-signal

Conversation

@phhien203
Copy link

✅ Challenge Submission Checklist

Start your PR title with: Answer:${challenge_number}

⚠️ Important Notice

If you would like personal feedback or a detailed review, please support the project on GitHub:

You can also submit a PR without sponsorship to:

  • Be listed among the answered challenges, or
  • Receive a review from a community member. 🔥

@github-actions github-actions bot added 30 interop rxjs signal answer answer sponsor sponsor the project to be reviewed PR requests a review labels Mar 15, 2026
@phhien203
Copy link
Author

This code refactoring uses the NgRx Signal Store to move all the logic from the component to the store. I also split the store state into smaller slides and encapsulated it with custom store features.

@tomalaforge tomalaforge removed the to be reviewed PR requests a review label Mar 22, 2026
@tomalaforge
Copy link
Owner

nice implementation of the signal store. I've never used it, but your implementation is very smart and decopled.

@phhien203
Copy link
Author

This approach helps me build pages with data table, paging, sorting, and filtering really fast. Because we already had all of lego blocks and just pick and compose it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

30 interop rxjs signal answer answer sponsor sponsor the project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants