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

[EVENT REQUEST] for mapping shopify B2B customers #25604

Closed
FrankAuwers opened this issue Dec 15, 2023 · 2 comments
Closed

[EVENT REQUEST] for mapping shopify B2B customers #25604

FrankAuwers opened this issue Dec 15, 2023 · 2 comments
Labels
event-request Request for adding an event

Comments

@FrankAuwers
Copy link
Contributor

FrankAuwers commented Dec 15, 2023

Describe the request

Make the following event as Integration Event.

 [InternalEvent(false)]
/// <summary> 
/// Description for OnBeforeMapCustomer.
/// </summary>
/// <param name="ShopifyOrderHeader">Parameter of type Record "Shopify Order Header".</param>
/// <param name="Handled">Parameter of type Boolean.</param>
internal procedure OnBeforeMapCustomer(var ShopifyOrderHeader: Record "Shpfy Order Header"; var Handled: Boolean)
begin
end;

Additional context

We want to use this for mapping B2B customers. But if it is not a B2B customer it must follow the standard for B2C Customers.

Internal work item: AB#494821

@FrankAuwers FrankAuwers changed the title [EVENT REQUEST] for mapping customers [EVENT REQUEST] for mapping shopify B2B customers Dec 15, 2023
@AleksandricMarko AleksandricMarko added the event-request Request for adding an event label Dec 15, 2023
@pri-kise
Copy link
Contributor

I thought there were some discussions on adding a B2B feature for the Shopify Connector by microsoft.
https://www.yammer.com/dynamicsnavdev/threads/2445981539434496

Additionally the IsHandled pattern should be avoided for the Shopify connector.
We should think how extensibility can be achived by Interfaces .

@AndreiPanko

@AndreiPanko
Copy link
Contributor

Yes, we discussed it here: #24967

The current way of mapping is not extensible enough and should reconsider it.

The B2B work is already in progress and you can see some of it (customer) in insider builds. Not yet completed.

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

No branches or pull requests

4 participants