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.

Sign up for GitHub

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

Jump to bottom

ATC Refunds #56

Closed
bimusiek opened this issue Mar 30, 2017 · 9 comments
Closed

ATC Refunds #56

bimusiek opened this issue Mar 30, 2017 · 9 comments
Labels
new message
Milestone
1.4.0

Comments

@bimusiek
Copy link
Collaborator

Hey,
I just checked Ticket_Checkeligiblity and it works, slowly going to implement rest of the endpoints in our system. Thanks a lot for your work!

I only realized now, there are 2 endpoints missing to be able to support ATC Refunds:

  • DocRefund_InitRefund
  • DocRefund_UpdateRefund

There is also last endpoint that process the refund, but we are not IATA licensed so our consolidator will be taking care of it. (thus, we dont need it)

Thanks. 👍
@DerMika
Copy link
Collaborator

DerMika commented Mar 30, 2017

I'm confused. Do you mean that you still need the 2 DocRefund messages mentioned?

@bimusiek
Copy link
Collaborator Author

Yes

DerMika added a commit that referenced this issue Apr 12, 2017
@DerMika
Implemented message DocRefund_InitRefund (issue #56)
9dac0ed
@DerMika
Copy link
Collaborator

DerMika commented Apr 12, 2017

One down, one to go. Samples for DocRefund_InitRefund here

@DerMika
Copy link
Collaborator

DerMika commented Apr 12, 2017

Oh god, the next one seems like a lot of work. Tell me when it becomes urgent, because I don't have the time right now!

@bimusiek
Copy link
Collaborator Author

@DerMika I will focus on ATC and LCCs and then I will work on DocRefund, so no hurry. We will probably need it in a month or something.

@DerMika DerMika added this to the 1.4.0 milestone May 2, 2017
@DerMika DerMika mentioned this issue May 7, 2017
Merged
@DerMika
Copy link
Collaborator

DerMika commented May 7, 2017

DocRefund_UpdateRefund is now also implemented. See sample

@DerMika DerMika closed this as completed May 7, 2017
@DerMika
Copy link
Collaborator

DerMika commented May 15, 2017

I've added DocRefund_ProcessRefund too. I know you said you didn't need it, but since I was working on the refunds anyway...

@DerMika
Copy link
Collaborator

DerMika commented May 15, 2017

FYI: Version 1.4.0 has just been released with the 3 new DocRefund messages included.

@bimusiek
Copy link
Collaborator Author

Thanks! Still waiting for ATC activation 😞 Amadeus people have some issues with enabling it since weeks.

atomy pushed a commit to mlamm/amadeus-ws-client that referenced this issue Nov 26, 2018
@f-woelfing-invia
Merge pull request amabnl#56 in FLIGHT/service.amadeus from VAN-1001 …
a1c7cd8 
…to master

* commit 'b779350bf07c634d776b5859ce1b50e0914460f8':
  VAN-1001 - pinning redis/mysql versions in docker-containers
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
new message
Projects
None yet
Milestone
1.4.0
Development

No branches or pull requests
2 participants
@bimusiek @DerMika