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

added support for multipass url generator #512

Merged
merged 7 commits into from
Aug 8, 2022

Conversation

knopa
Copy link
Contributor

@knopa knopa commented May 12, 2020

I am not sure how better to do the test. Technically best option is to find the user in Shopify customers by multipass identifier to assert it.

@nozzlegear
Copy link
Owner

Thanks for the pull request @knopa! I don't think I'm familiar with this feature, can you explain what this is for, and add a link to some documentation?

@knopa
Copy link
Contributor Author

knopa commented May 14, 2020

It's from list unimplemented functionality
https://help.shopify.com/api/reference/multipass

@knopa
Copy link
Contributor Author

knopa commented May 14, 2020

It's needed when you need to allow auto-create customers from your existing project not Shopify one. So they can use exist credentials to signin into shopify too

Copy link
Owner

@nozzlegear nozzlegear left a comment

Choose a reason for hiding this comment

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

Hey! I'm looking into finally merging this into master. I've left a review comment about changing the class to static, would you mind taking a look if you're still interested in getting this merged?

ShopifySharp/Services/Multipass/MultipassService.cs Outdated Show resolved Hide resolved
@knopa
Copy link
Contributor Author

knopa commented Aug 7, 2022

Hey, I have updated as your asked.

@knopa knopa requested a review from nozzlegear August 7, 2022 15:06
@nozzlegear nozzlegear merged commit 5a3519d into nozzlegear:master Aug 8, 2022
@nozzlegear nozzlegear added release pending The code has been merged into master and is pending a release on nuget. and removed release pending The code has been merged into master and is pending a release on nuget. labels Aug 12, 2022
@nozzlegear
Copy link
Owner

Released in 5.18.0 on nuget! Thanks for your contribution!

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

Successfully merging this pull request may close these issues.

2 participants