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

Implementace odeslání zprávy do simpleApi #3

Merged
merged 1 commit into from
Aug 27, 2018

Conversation

Sharsie
Copy link

@Sharsie Sharsie commented Mar 28, 2018

Ahoj, trochu jsem se pral s odesíláním zpráv s attachmentama, tak bych chtěl nasdílet tuto úpravu.

Nějak se mi tam nezdá to generování některých API class, vůbec to nesedí na to, co se reálně musí poslat na backend czeboxu.

tFilesArray, a dmFile mi přijde špatně vygenerované, zkoušel jsem i aktualizovat WSDL (mám v jiné feature větvi), ale i tam je to chybně.

Pro dmFile jsem tam udělal extending class, přejmýšlím, zda neudělat i pro tFilesArray.

A pak mě hrozně mate, proč do tMessageCreateInput nemůžu poslat dmEnvelope, s ní ten request neprojde. Místo toho funguje tMessageEnvelopeSub.

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Pull request does not pass validation.

@Sharsie
Copy link
Author

Sharsie commented Mar 28, 2018

Testy kdyžtak doplním, co tady řve CI s manifestem netuším :)

@dfridrich dfridrich merged commit ef7feea into dfridrich:master Aug 27, 2018
@Sharsie Sharsie deleted the feature/MessageWithAttachments branch August 27, 2018 08:53
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.

2 participants