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

Add additional PostNL order data to order API #63

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sirajea
Copy link

@sirajea sirajea commented Jan 8, 2020

Submitting issues trough Github

Please follow the guide below

  • You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
  • Put an x into all the boxes [ ] relevant to your issue (like this: [x])
  • Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: https://tig.nl/postnl-magento-extensies/

Issues with outdated version will be rejected.

  • I've verified and I assure that I'm running the latest version of the TIG PostNL Magento extension.

What is the purpose of your issue?

  • Bug report (encountered problems with the TIG PostNL Magento extension)
  • Site support request (request for adding support for a new site)
  • Feature request (request for a new functionality)
  • Question
  • Other

Description of your issue, suggested solution and other information

I'd like to add the following additional PostNL data to be available through the Magento1 Order API:

  • Order type
  • Order delivery date
  • Order expected delivery time start
  • Order expected delivery time end

Please see pull request. Thanks!

Support TIG

On Github we will respond in English even when the question was asked in Dutch.

@tig-dennisvanderhammen
Copy link
Contributor

Hi,

Thanks for your submission!
I'll discuss this with our team. I personally don't see any issues adding these fields. It's likely that you can expect these fields in one of the upcoming releases!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

3 participants