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

DocIssuance_IssueTicket Consolidator method #23

Closed
fayax opened this issue Oct 6, 2016 · 3 comments
Closed

DocIssuance_IssueTicket Consolidator method #23

fayax opened this issue Oct 6, 2016 · 3 comments

Comments

@fayax
Copy link

fayax commented Oct 6, 2016

Is it possible to use consolidator method with DocIssuance_IssueTicket?
Amadeus suggest our DocIssuance request must be in this format as we are issuing ticket directly on airline stock.

<DocIssuance_IssueTicket>
        <optionGroup>
                <switches>
                       <statusDetails>
                       <indicator>ET</indicator>
                       </statusDetails>
                </switches>
        </optionGroup>
        <otherCompoundOptions>
                <attributeDetails>
                       <attributeType>ETC</attributeType>
                            <attributeDescription>1A</attributeDescription>
                       </attributeDetails>
        </otherCompoundOptions>
</DocIssuance_IssueTicket>
@DerMika
Copy link
Collaborator

DerMika commented Oct 6, 2016

Not yet. I'll put support for that option on my to-do list.

@DerMika DerMika added this to the 1.1.0 milestone Oct 7, 2016
@DerMika
Copy link
Collaborator

DerMika commented Oct 7, 2016

Pushed a fix to support all the otherCompoundOption options in DocIssuance_IssueTicket.

Here is a sample of how to make the request you mentioned above: https://github.com/amabnl/amadeus-ws-client/blob/master/docs/samples.rst#docissuance (third example, scroll down a bit).

Let me know if that fixes your problem.

@DerMika
Copy link
Collaborator

DerMika commented Oct 12, 2016

Hey @fayax, I'm going to close this issue because I'd like to prepare for the release of v 1.1.

If you have any problems with this feature, add a comment here or open a new issue.

@DerMika DerMika closed this as completed Oct 12, 2016
atomy pushed a commit to mlamm/amadeus-ws-client that referenced this issue Nov 26, 2018
…N-600-service.amadeus-return-empty-result to master

* commit '698bfc0b8d32259934329972b4038c36b178aa13':
  VAN-600 service.amadeus / return empty result for some ama errors
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants