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

MasterPricer Currency conversion #35

Closed
bimusiek opened this issue Dec 14, 2016 · 3 comments
Closed

MasterPricer Currency conversion #35

bimusiek opened this issue Dec 14, 2016 · 3 comments

Comments

@bimusiek
Copy link
Collaborator

bimusiek commented Dec 14, 2016

Hey,
I am looking into a way how to send currency conversion option in master pricer.
Is there a way to do this?

From docs:

5.38 Operation: 04.01 Fare Option - Currency Conversion
The currency conversion option may be combined with any other option.
It is mandatory that the ticketing price type be set to "CUC" to indicate a currency conversion request. It is mandatory for any query including the currency conversion option.
It is mandatory that a currency code be specified for the conversion. All price amounts for recommendations are converted in the requested Currency. There is no default currency for conversion, it must be specified.
The example below illustrates a low fare search including an optional Currency Conversion element specified with the following information.

Example:

<fareOptions>
    <pricingTickInfo>
        <pricingTicketing>
            <priceType>CUC</priceType>
        </pricingTicketing>
    </pricingTickInfo>
    <conversionRate>
        <conversionRateDetail>
            <currency>EUR</currency>
        </conversionRateDetail>
    </conversionRate>
</fareOptions>
@DerMika
Copy link
Collaborator

DerMika commented Dec 14, 2016

This is not supported yet. I'll make a fix to support currency conversion.

@DerMika DerMika added this to the 1.2.0 milestone Dec 14, 2016
@DerMika DerMika changed the title Currency exchange MasterPricer Currency conversion Dec 15, 2016
@DerMika
Copy link
Collaborator

DerMika commented Dec 16, 2016

Pushed a fix to support currency conversion. See the docs for an example: https://github.com/amabnl/amadeus-ws-client/blob/master/docs/samples/masterpricertravelboard.rst#currency-conversion

@DerMika DerMika closed this as completed Dec 16, 2016
@bimusiek
Copy link
Collaborator Author

@DerMika You are amazing! :) Thanks.

atomy pushed a commit to mlamm/amadeus-ws-client that referenced this issue Nov 26, 2018
…o master

* commit '8b21fc84950e8b9aaca25fa9c52cdcb4313937dc':
  VAN-749 Operating carrier text handling
  VAN-749 Operating carrier text handling
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