fix: `toRefs()` expects a reactive object but received a plain one (warning)

[rozsazoltan]

Problem

const state = inject(STATE_INJECT_KEY, {} as Ref<OnboardingState>)
const { step, isFirstStep, isLastStep, options, next, previous, exit: stateExit, finish } = toRefs(state.value)

[!WARNING] toRefs() expects a reactive object but received a plain one.

Solution

const state = inject(STATE_INJECT_KEY, {} as Ref<OnboardingState>)
const { step, isFirstStep, isLastStep, options, next, previous, exit: stateExit, finish } = state.value

The use of toRefs is unnecessary when we do not intend to transform a reactive object. The type of the state variable used in VOnboardingStep.vue is already of type ref(). I consider the per-variable transformation performed on it to be unnecessary; the variables can be declared without using toRefs() as I did in the modifications of this PR. This eliminates the warning message. The type of the state will always be ref(), so using toRefs() is not justified.

[!NOTE]

Converts a reactive object to a plain object where each property of the resulting object is a ref pointing to the corresponding property of the original object. Each individual ref is created using toRef().

Vue Docs - toRefs

[netlify[bot]]

✅ Deploy Preview for eloquent-lamport-f2ffa1 canceled.

Name	Link
🔨 Latest commit	f60652f12eeaec6e873fa6bb9ccb1831477d9d76
🔍 Latest deploy log	https://app.netlify.com/sites/eloquent-lamport-f2ffa1/deploys/64f309ad558810000828483f

[netlify[bot]]

✅ Deploy Preview for beamish-dango-bc9c2d canceled.

Name	Link
🔨 Latest commit	f60652f12eeaec6e873fa6bb9ccb1831477d9d76
🔍 Latest deploy log	https://app.netlify.com/sites/beamish-dango-bc9c2d/deploys/64f309ada84071000884c897

[fatihsolhan]

Thank you @rozsazoltan, I appreciate your support on v-onboarding 💪

[github-actions[bot]]

:tada: This PR is included in version 2.5.2 :tada:

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot :package::rocket: