RSSNext / Follow

🧡 Follow your favorites in one inbox
https://follow.is
GNU General Public License v3.0
17.08k stars 711 forks source link

refactor: simplify useRegisterCommand types #1678

Closed lawvs closed 1 day ago

lawvs commented 1 day ago

Rename useRegisterCommandEffect to useRegisterCommand and simplify its implementation.

Remove the deprecated hook and update related imports accordingly. Introduce type safety improvements for command registration.

vercel[bot] commented 1 day ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
follow ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 20, 2024 6:29pm
1 Skipped Deployment | Name | Status | Preview | Comments | Updated (UTC) | | :--- | :----- | :------ | :------- | :------ | | **follow-external-ssr** | ⬜️ Ignored ([Inspect](https://vercel.com/rss3/follow-external-ssr/GPfX893GUrBowJHtr5MuvgizMRyX)) | [Visit Preview](https://follow-external-ssr-git-refactor-simplify-register-cmd-rss3.vercel.app) | | Nov 20, 2024 6:29pm |
follow-reviewer-bot[bot] commented 1 day ago

Suggested PR Title:

refactor(command): update command registration hooks

Change Summary: Refactor command registration with new useRegisterFollowCommand hook, replacing the deprecated useRegisterCommandEffect. Added TypeScript tests to validate command types and usage, enhancing code maintainability and clarity.

Code Review: No change requests necessary.