Skip to content
This repository was archived by the owner on Sep 21, 2023. It is now read-only.

Multi label support#52

Merged
Petschko merged 31 commits into
masterfrom
multi-labels
Sep 23, 2018
Merged

Multi label support#52
Petschko merged 31 commits into
masterfrom
multi-labels

Conversation

@Petschko
Copy link
Copy Markdown
Owner

@Petschko Petschko commented Sep 7, 2018

I finished the multi-label support (also for other functions, which allows multiple requests in one call)

Sadly I cannot test everything, so I would love to get some feedback about those:

  • * Thermo-Test user works
  • * createShipment still works
  • * createShipment works with multiple elements
  • * createShipment shows multiple labels in Response
  • * createShipment works with "ExportDocument"
  • * createShipment shows the "ExportDocument" in Response
  • createShipment works with "COD (Cash on delivery)"
  • createShipment shows the "COD (Cash on delivery)" label in response
  • * doManifest still works
  • * doManifest works with multiple elements
  • * doManifest shows result(s) in Response
  • * deleteShipment still works
  • * deleteShipment works with multiple elements
  • * deleteShipment shows the result(s) in Response
  • * getShipmentLabel still works
  • * getShipmentLabel works with multiple elements
  • * getShipmentLabel shows the result(s) in Response
  • * getExportDoc still works
  • * getExportDoc works with multiple elements
  • * getExportDoc shows the result(s) in Response
  • * validateShipment still works
  • * validateShipment works with multiple elements
  • * validateShipment shows the result(s) in Response

Just use the "multi-label" Branch for testing my changes.

I made some bigger changes, so I want to ensure everything works, before merging. I will merge, when every * element is tested

Marked old fields, getter & setter as deprecated
@Petschko
Copy link
Copy Markdown
Owner Author

I would be super thankfully if someone can test this with the Export-Document, this is the last thing we need to test =)

@Petschko Petschko merged commit 451e5ff into master Sep 23, 2018
This was referenced Sep 23, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant