chore(auth): allow user refetch

[jiblett1000]

Types of changes

• Bug fix (a non-breaking change which fixes an issue)
• New feature (a non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Description

This is a small but important change. This separates the guard logic in the fetchUser method so that it maintains the current functionality while allowing the developer to refetch the user data when/if desired.

By putting if (!user) around the fetchUser call in the plugin we maintain that it will only be called once when there is no user state. Then, we we leave the if (token) within the fetchUser method to maintain security.

Use case:

While building a profile page, I needed to update the user avatar and info. The avatar in particular posed a bigger problem. I needed to refresh the user info after uploading it. However, after banging my head against the wall for a bit while trying to use the fetchUser method, I realized the !user.value guard in the method meant that it would never refetch the user data when a user was logged in. I moved that logic to the plugin and bingo, the avatar refreshed as intended.

That's about it. Thanks for reading!

Checklist:

• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes (if not applicable, please state why)

[benjamincanac]

@jiblett1000 Thanks for the PR, indeed it makes sense!