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

Stockpoint #229

Open
andersjerenvik opened this issue Jan 28, 2022 · 7 comments
Open

Stockpoint #229

andersjerenvik opened this issue Jan 28, 2022 · 7 comments

Comments

@andersjerenvik
Copy link

Hi,

Please add

StockPointCode
StockPointId

to OrderRow

Thanks;.)

Anders

@richardrandak
Copy link
Collaborator

#131
#133

@richardrandak
Copy link
Collaborator

Based on documentation examples, the order has

  • StockPointCode
  • StockPointId

OrderRow has

  • StockPointCode
  • StockPointId

Invoice has

  • StockPointCode

Invoice Row has

  • StockPointCode

Is that correct?

@andersjerenvik
Copy link
Author

andersjerenvik commented Feb 1, 2022

Yes, thats correct

I was wrong so I'm not going to use this fields now.
Fortnox doesn't have Stockplace on orderrows so this is not very useful.

Thanks anyway,

Anders

@richardrandak
Copy link
Collaborator

richardrandak commented Feb 1, 2022

I tried to add warehouse related properties, but the problem is with Order.DeliveryState.. if the warehouse module is not enabled, I get an error on when executing order update, even if the property is not modified. The error says the property can not be modified if the warehouse module is not enabled. I can't mark it as readonly (and therfore exclude it in deserialization) since it needs to be updated in some cases.

Går ej att redigera lagerstatus innan lagermodulen har aktiverats.

This would break the functionality.

@andersjerenvik
Copy link
Author

Ok, I See.

If you set it to Nullable, is it the same problem?

@richardrandak
Copy link
Collaborator

richardrandak commented Feb 1, 2022

In current use case yes, because the property is populated by the server even if you don't have the warehouse enabled (which is weird). :)
..so you get the order with the delivery state set to "delivery", but it is not accepted during the update.

@andersjerenvik
Copy link
Author

Ok, that's probably something to talk to Fortnox about then.

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