-
Notifications
You must be signed in to change notification settings - Fork 102
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
feat(shopware-6-client): add newsletter recipient endpoint #1942
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
💙 shopware-pwa-test-europe-west1-gcp-storefrontcloud-io successfully deployed at https://8634dea998dd7bcce8e2e437f10b240158f7cbdc.shopware-pwa-test.preview.europe-west1.gcp.storefrontcloud.io |
export async function isNewsletterSubscriber( | ||
contextInstance: ShopwareApiInstance = defaultInstance | ||
): Promise<{ | ||
status: string; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we expect here some specific values to add it as an enum or union of strings using types?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For now, this endpoint is not described in the documentation, which is why I added string, not enum or union. I think that we can change this type when documentation will be updated and we will be sure about all options
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good 👍🏻 only one question to answer and it's ready to go 💯
Changes
Add new newsletter recipient endpoint
Checklist