Developer forum

Forum » Feature requests » Avalara - Customer Code

Avalara - Customer Code

E-mail notifications
Nuno Aguiar
Reply
Posted on 10/02/2017 14:10:20

Hi,

 

Currently in the Avalata Tax Provider, the CustomerCode is set bases on the AccessUserID

taxRequest.CustomerCode = order.CustomerAccessUserID.ToString();

 

We have a request from a customer (with ERP integration) where he would like to see the ExternalID for easier tax reconciliation purposes, as the ERP has no knowledge of what DW's ID is.

 

The request would be to have the CustomerCode a bit more flexible and configurable

  • Have a new parameter to take something like
    • ExternalID
    • UserID if ExternalID is empty
    • CustomerNumber if ExternalID is empty

 

Best Regards,

Nuno Aguiar


Replies

 
Imar Spaanjaars Dynamicweb Employee
Imar Spaanjaars
Posted on 10/02/2017 14:16:18
Reply

I wouldn't handle this automatically, but provide configurable UI for it (an option) in the connector's setting.

Or (or maybe in addition) make a subscriber OnBeforeCustomerId that lets us provide whatever value....

Imar

 
Nuno Aguiar
Posted on 15/03/2017 19:30:22
Reply

Hi,

 

Any news on this? 

Nuno

 
Nicolai Pedersen
Posted on 15/03/2017 23:39:31
Reply

Nope,but you can inherit the provider and add this extra setting there until it gets in standard. You are also welcome to file a pull request...

BR Nicolai

 
Imar Spaanjaars Dynamicweb Employee
Imar Spaanjaars
Posted on 16/03/2017 10:00:16
Reply

I guess the problem with inheriting is that you have to rewrite pretty much everything. Most calls to CustomerAccessUserID are in private methods which you can't simply override. Now, if there was a virtual GetCustomerId method that by default would return CustomerAccessUserID but that could be overridden to return something else, stuff would be a lot easier.

@Nuno: let's have a look at this; I like the idea of a pull request too.

Imar

 
Imar Spaanjaars Dynamicweb Employee
Imar Spaanjaars
Posted on 22/03/2017 11:38:49
Reply

Can we take you up on that Pull request offer? Nuno and I just implemented this for Dynamicweb 8 and 9. I'll send you the code in an email.

Thanks!

Imar

 
Nicolai Pedersen
Posted on 22/03/2017 12:31:53
Reply

Yes, of course!

I think you actually have commit access to head branches in TFS, so you can commit a shelveset if that is easiest for you. Otherwise just mail me the code.

BR Nicolai

 
Imar Spaanjaars Dynamicweb Employee
Imar Spaanjaars
Posted on 26/03/2017 15:00:30
Reply

I mailed you the code before I saw this post. Let me know if you still want a shelveset instead.

Thanks!

 
Nuno Aguiar
Posted on 24/05/2017 22:34:47
Reply

Hi,

 

Was this ever deployed into a DW version? The project is currently running in 8.9.2.1 and will be going live soon and we'll need it

 

Best Regards,

Nuno

 
Merethe Vrå Andersen Dynamicweb Employee
Merethe Vrå Andersen
Posted on 01/06/2017 15:02:29
Reply

Hi Nuno, 

The code is actually implemented in the latest 8.9.2 hotfix (8.9.2.4), but is untested because our Avalara test account is deactivated and we haven't got an answer from Avalara yet.

I hope that you can get it to work.

Kind regards,
Merethe

 
Nuno Aguiar
Posted on 01/06/2017 19:32:33
Reply

Hi Merethe,

 

I understand. We will have our customer test it once he upgrades.

 

Thank you,

Nuno

 
Imar Spaanjaars Dynamicweb Employee
Imar Spaanjaars
Posted on 09/10/2017 10:02:24
Reply

FY, this should nw be merged into the latest versions of Avalara....