Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[VNDA] - Feat: add user #993

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

yuriassuncx
Copy link
Contributor

What is this Contribution About?

This Pull Request introduces the implementation of the user hook for integrating VNDA - E-commerce Platform - with Deco.cx. This hook captures the user's accessToken and makes a request to the VNDA API to retrieve the associated user ID.

Changes Made

  1. Utility Function:

    • A utility function was created to extract the accessToken from the user's cookie and make a call to the VNDA API to fetch the user data.
  2. Updated useUser Hook:

    • The useUser hook has been updated to incorporate the new user loading logic.
    • The context structure was adjusted to store user data and loading state.
  3. Loader Implementation:

    • A loader was implemented to make the request to the VNDA API, using the accessToken to retrieve user information.

Issue Link

Please link to the relevant issue that this pull request addresses:

Copy link
Contributor

Tagging Options

Should a new tag be published when this PR is merged?

  • 👍 for Patch 0.64.22 update
  • 🎉 for Minor 0.65.0 update
  • 🚀 for Major 1.0.0 update

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant