Ticket_ProcessEDoc Request

[FarahHourani]

Hello
Can u provide me with some tips or the steps to create new Ticket_ProcessEDoc request?

Thanks.

[DerMika]

Certainly!

I've written some explanation on how to implement a new message here and here. You can also check an implementation of a message I've implemented recently, for example this commit.

Hopefully these comments shed some light into the internal workings of the library. Let me know if you need any help!

[DerMika]

I will add that the hardest part of implementing a message is:

1. Generating all the classes needed for the XML request structure (not hard, just an annoyingly large amount of manual class creation)
2. Figuring out how to create the easiest possible RequestOptions class and how to translate those options into the XML message that must be sent.

For the second part, this is really a matter of studying all the samples for that message on the Amadeus Extranet and implement from there.

[FarahHourani]

Thank u so much for your answer and help.. Finally it's done now :D 💃

[DerMika]

I'm looking forward to the Pull Request! :)

[FarahHourani]

The problem is that i didn't implement the response handler i'm handling the error from my side for now i thought it's easier.. but i after implement it i will create a pull request

[DerMika]

Luckily the response handler isn't that much work compared to the rest. I wouldn't mind doing that myself (rather than risk not having this message implemented ;) ).

[FarahHourani]

I pushed my updates, if you want any fix tell me. #135

[DerMika]

Thanks a lot for your contribution!

Do you prefer that I ask you to add the missing items (Response handler, unittests, docs), or would you rather have me do it?

[FarahHourani]

Most welcome, I guess these things are gonna take a lot of time from me since i have a lot of other priorities right now so i guess u doing them is so much better

[FarahHourani]

Is it a hard part ?

[DerMika]

Not really, but I don't mind doing it.

[FarahHourani]

Cool, for me i have to figure out how to them, maybe in another message i'll be able to do them.

[DerMika]

I will merge your PR tonight and then make the necessary fixes.

Thanks again for the contribution!

[FarahHourani]

Okay, Thanks to u too 👍

[DerMika]

I've just merged #135, thanks again!