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

Add email as input to setCurrentOrganization mutation #130

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

Conversation

Nadeera-Gayashan
Copy link
Collaborator

@Nadeera-Gayashan Nadeera-Gayashan commented Mar 12, 2024

What problem is this solving?

Add email as input to setCurrentOrganization mutation. The use case is a custom flow that allows a user to join a "Special School Program" for musical instrument rentals. The school is represented by a B2B organization. There is a modal component where the user chooses a school, then is prompted to log in, and then is immediately added to the relevant B2B organization. Finally, that organization is set as their current organization. All of this is done before the page has been reloaded, so their email does not yet exist in their sessionData.

Copy link

vtex-io-ci-cd bot commented Mar 12, 2024

Hi! I'm VTEX IO CI/CD Bot and I'll be helping you to publish your app! 🤖

Please select which version do you want to release:

  • Patch (backwards-compatible bug fixes)

  • Minor (backwards-compatible functionality)

  • Major (incompatible API changes)

And then you just need to merge your PR when you are ready! There is no need to create a release commit/tag.

  • No thanks, I would rather do it manually 😞

Copy link

Messages
📖 ❤️ Thanks!
📖

🎉 PR additions = 7, PR deletions = 5

Generated by 🚫 dangerJS against e037d0d

Copy link

const {
email: { value: email },
} = sessionData.namespaces.profile
const email = sessionData?.namespaces?.profile?.value ?? userEmail
Copy link
Contributor

@enzomerca enzomerca Mar 12, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you add to the PR description what is the use case that does not have the email on the sessionData?

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.

2 participants