best-brands / bol-com-retailer-api

A Bol.com API client with full object mapping
10 stars 6 forks source link

Billingdetails missing in orders #3

Closed WeTurkstra closed 3 years ago

WeTurkstra commented 3 years ago

Hey,

I seem to missing billings details in all the orders i fetch. Do you experience the same? Is this another BOL.com mistake?

Thanks.

harm-smits commented 3 years ago

I have no idea, can you check how old your order is?

WeTurkstra commented 3 years ago

All the orders are from today. I used postman to call the api directly and it seems like the billingDetails are completly missing.

Its a bit like the other issue we had a few weeks ago.

harm-smits commented 3 years ago

They changed it again indeed -_-

WeTurkstra commented 3 years ago

Great... hahaha. I love it when stuff randomly change. Our customers love it too😂.

Well i will make a workaround for it.

harm-smits commented 3 years ago

Great, I am currently not in the office so please let me know what exactly you change so I can merge it accordingly. Especially interested in the change in raw response.

This is becoming quite the nuisance :/

WeTurkstra commented 3 years ago

Yes ideed. Waht is will do is use the shipping Details for the billing details. I dont think its wise to do this in this repository.

There is no real way to fetch the billing details right now as far as i can see.

harm-smits commented 3 years ago

https://twitter.com/bol_com/status/1400398349761748993?s=19

harm-smits commented 3 years ago

Well, it's not a bug technically, since in the spec they define this as 'optional'. But it's needed for VAT invoices regardless.

WeTurkstra commented 3 years ago

Yea. I saw your tweet earlier and retweeted it. So it looks like a technical issue on their side?

harm-smits commented 3 years ago

Yep, no clue when it will be resolved though :(

WeTurkstra commented 3 years ago

I looks like the orders are coming through right now. It might be fixed!

harm-smits commented 3 years ago

Yep, same here. I don't know why they don't test new deployments tbh. It scares me and I don't like it.

harm-smits commented 3 years ago

Seeing as this has been resolved by bol.com, I am going to close this issue.