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.

Sign up for GitHub

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

Jump to bottom

[VNDA] - Feat: add user #893

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

[VNDA] - Feat: add user #893

wants to merge 5 commits into 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:

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Oct 1, 2024

Tagging Options

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

  • 👍 for Patch 0.59.21 update
  • 🎉 for Minor 0.60.0 update
  • 🚀 for Major 1.0.0 update

@yuriassuncx yuriassuncx changed the title [WIP] VNDA - Feat: add user [VNDA] - Feat: add user Oct 2, 2024
gabrielMatosBoubee
gabrielMatosBoubee requested changes Oct 3, 2024
View reviewed changes
vnda/utils/user.ts Outdated Show resolved Hide resolved
@IncognitaDev IncognitaDev linked an issue Nov 12, 2024 that may be closed by this pull request
Missing useUser hook in VNDA integration #384
Open
@matheusgr
Copy link
Contributor

@yuriassuncx Can you rebase, so we can run checks again?

@yuriassuncx
Copy link
Contributor Author

@yuriassuncx Can you rebase, so we can run checks again?

done

@yuriassuncx yuriassuncx requested a review from matheusgr December 2, 2024 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@IncognitaDev IncognitaDev IncognitaDev left review comments

@gabrielMatosBoubee gabrielMatosBoubee Awaiting requested review from gabrielMatosBoubee

@matheusgr matheusgr Awaiting requested review from matheusgr

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Missing useUser hook in VNDA integration
4 participants
@yuriassuncx @matheusgr @IncognitaDev @gabrielMatosBoubee