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

Added office ids to Master Pricer #44

Merged
merged 4 commits into from
Feb 25, 2017
Merged

Conversation

bimusiek
Copy link
Collaborator

@bimusiek bimusiek commented Feb 24, 2017

Support for multiple office ids in Master Pricer.

I could not really check it (except for looking at the logs) as we have no access to multiple office ids in master pricer (we have access to office ids though)

The error you get when it is not allowed in your master pricer is:
[1032] OID(S) NOT AUTHORISED: 1,2,3,4,5

When we will get access I can fully test.

@coveralls
Copy link

coveralls commented Feb 24, 2017

Coverage Status

Coverage increased (+0.0004%) to 99.809% when pulling ffdf3a7 on whatsahoy:master into 64cf33c on amabnl:master.

@DerMika DerMika merged commit a63b4c9 into amabnl:master Feb 25, 2017
@DerMika
Copy link
Collaborator

DerMika commented Feb 25, 2017

Thanks! You seem to be using every option of MasterPricer available :)

DerMika added a commit that referenced this pull request Feb 25, 2017
@DerMika DerMika added this to the 1.3.0 milestone Aug 15, 2017
atomy pushed a commit to mlamm/amadeus-ws-client that referenced this pull request Nov 26, 2018
…N-854-amadeus-amadeus-response-error-926-invalid to master

* commit 'e7fb047c404f7d7d37415d5360b7a350bd9fbaf6':
  VAN-854 | (fix) added error 996 to the "empty results" error list
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants