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

Fix bug which ignored AMADEUS_HOST property provided by environment #18

Merged
merged 1 commit into from
Oct 28, 2018

Conversation

nirmalvp
Copy link
Contributor

API documentation suggests that the user can pass a custom domain to
which the client will issue API calls, by setting an AMADEUS_HOST
property in the system environment variable. This functionality was
however broken due to a typo while reading the environment variable.

Fix bug by correcting the typo, thus picking the correct env variable.

API documentation suggests that the user can pass a custom domain to
which the client will issue api calls, by setting a AMADEUS_HOST
property in the system environment variable. This functionality was
however broken due to a typo while reading the environment variable.

Fix bug by correcting the typo, thus picking the correct env variable
@anthonyroux anthonyroux self-assigned this Oct 28, 2018
@anthonyroux anthonyroux self-requested a review October 28, 2018 12:10
@anthonyroux
Copy link
Contributor

Thank you for your contribution!

The version 1.1.2 should be published tomorrow.

@anthonyroux anthonyroux merged commit 78e7685 into amadeus4dev:master Oct 28, 2018
@anthonyroux
Copy link
Contributor

I deployed the new version, it always takes a few minutes/hours to be available on Maven Central.

Thanks again for your contribution! 👍

@nirmalvp
Copy link
Contributor Author

My pleasure.

On an unrelated note, I noticed that you do not have a Golang client yet for your APIs. I have started developing one from scratch in my local workspace.

I was wondering if you would be interested to accept contributions.

@anthonyroux
Copy link
Contributor

anthonyroux commented Oct 28, 2018

We are more than happy to accept contributions and even to work together to build it!

Actually my colleague @alnacle is working on a Golang version of our SDK, would you be interested to have a short call to talk about it and see how we can work together?

You can contact me at: Anthony.roux[at]amadeus.com

@nirmalvp
Copy link
Contributor Author

Awesome. Thank you. I have sent you a mail on your email to follow-up.

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

Successfully merging this pull request may close these issues.

None yet

2 participants