fix: Ensure Reactivity for 'to' Attribute in useLink Composable #20994
+186
−8
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR addresses the following issue:
If a component is initialized without the
to
attribute, adding the attribute later does not involve the client-side router anymore. Clicking on such a component causes the entire application to reload. This issue is caused by thenavigate
androute
properties returned by theuseLink
composable not being reactive. This PR adds reactivity to these properties.Additionally, this PR includes unit tests for the
useLink
composable to ensure its functionality and correctness.Markup:
In the current version, clicking on the 'Not Working' card causes the entire application to reload.
In the fixed version, it utilizes the client-side router.