search

ho-nl / BolCom_RetailerApi

Package that talks to the bol.com v8 API
10 stars 10 forks source link

GetStatusByProcessIdsHandler class no longer uses concurrent requests #55

Closed JeroenVanLeusden closed 1 year ago

JeroenVanLeusden commented 1 year ago

In https://github.com/ho-nl/BolCom_RetailerApi/pull/54 a change is introduced, particularly here that results in the GetStatusByProcessIdsHandler no longer being able to handle concurrent requests.

Obviously i don't have access to the Magento module itself but you might want to check if this change is intentional and if so, refactor the method to no longer use promises or revert to support concurrent requests again.

PascalBrouwers commented 1 year ago

Thank you @JeroenVanLeusden Will be fixed in 4.0.3: https://github.com/ho-nl/BolCom_RetailerApi/pull/56