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

DeliveryState and WarehouseReady to Order. #235

Closed
TeroMalmberg88 opened this issue Mar 2, 2022 · 5 comments
Closed

DeliveryState and WarehouseReady to Order. #235

TeroMalmberg88 opened this issue Mar 2, 2022 · 5 comments

Comments

@TeroMalmberg88
Copy link

Fortnox has added these fields to the order model in the GET/PUT API. Could these be added to this project ASAP. These fields seems to indicate whether the order is ready for delivery or not. And this is very critical in deciding what orders to fulfill.

@richardrandak
Copy link
Collaborator

I tried to add it as part a result of #229 and got stopped by an issue with the DeliveryState, which would break the functionality for those who does not have warehouse enabled.
What I think I could do, is to add the properties but making DeliveryState read-only. Would that be enough for you, or do you need to set/update this property as well?

@TeroMalmberg88
Copy link
Author

We need to set/update it as well.

@richardrandak
Copy link
Collaborator

I could think of some workaround to enable updating the property separately as an alternative to the common Update method

@richardrandak
Copy link
Collaborator

richardrandak commented Mar 4, 2022

I have created a pre-release NuGet with additional Order properties as well as action to make the order "warehouse ready".
Please check it out - https://www.nuget.org/packages/Fortnox.NET.SDK/4.3.2-warehouse-rc

When creating FortnoxClient, set its property WarehouseEnabled to true, which will enable changing the DeliveryState of the order. In other case, the property won't be serialized.

Let me know if you have any issues.

@TeroMalmberg88
Copy link
Author

Quick test indicated that it works fine. Thank you.

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

No branches or pull requests

2 participants