Skip to content

Add Support for Travel Restrictions API#131

Merged
tsolakoua merged 3 commits intoamadeus4dev:masterfrom
siddydutta:travel-restrictions
Feb 14, 2022
Merged

Add Support for Travel Restrictions API#131
tsolakoua merged 3 commits intoamadeus4dev:masterfrom
siddydutta:travel-restrictions

Conversation

@siddydutta
Copy link
Copy Markdown
Contributor

@siddydutta siddydutta commented Feb 5, 2022

Fixes #119

Changes for this pull request

@siddydutta siddydutta changed the title Add support for travel restrictions API Add Support for Travel Restrictions API Feb 5, 2022
@gimperial gimperial requested a review from tsolakoua February 8, 2022 16:42
@tsolakoua
Copy link
Copy Markdown
Member

tsolakoua commented Feb 10, 2022

@siddydutta thanks a lot for your contribution! Could you please update your PR with the last changes of the master branch? I did a review and it all looks good :)

Comment thread README.rst Outdated
departureDate='2021-03-21')

# Travel Restrictions
amadeus.travel_restrictions.covid19_area_report.get(countryCode="US")
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The SDK maps every API path to a similar path. For example, /v1/duty-of-care/diseases/covid19-area-report
should be represented asamadeus.duty_of_care.diseases.covid19_area_report.
So the classes DutyOfCare and Diseases have to be created and delete the TravelRestrictions.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, understood, will make those changes.

Copy link
Copy Markdown
Member

@tsolakoua tsolakoua left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @siddydutta!!

@tsolakoua tsolakoua merged commit 564fa2b into amadeus4dev:master Feb 14, 2022
@siddydutta siddydutta deleted the travel-restrictions branch March 6, 2025 18:03
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.

Add support for Travel Restrictions API

2 participants