-
Notifications
You must be signed in to change notification settings - Fork 1
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
Mailgun release - DO NOT MERGE #150
Conversation
packages/mailgun/src/models/v1/emails/request/override-properties/index.ts
Outdated
Show resolved
Hide resolved
packages/mailgun/src/models/v1/emails/request/override-properties/override-properties.ts
Show resolved
Hide resolved
import { OverrideProperties } from '../override-properties'; | ||
import FormData = require('form-data'); | ||
|
||
export type MessageContentWhereHtmlContentCanBeInlineOnly = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SendEmailRequestWithoutTemplate
SendEmailRequestBasic
SendEmailRequestSimple
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See the comment on the other type name ⬇️
packages/mailgun/src/models/v1/emails/request/send-email-request/send-email-request.ts
Show resolved
Hide resolved
...ages/mailgun/src/models/v1/emails/request/send-email-request-base/send-email-request-base.ts
Outdated
Show resolved
Hide resolved
packages/mailgun/src/models/v1/emails/response/get-email-response/get-email-response.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved and according to our discussion we will need to think again about naming before releasing/GA
packages/mailgun/src/models/v1/emails/request/override-properties/override-properties.ts
Show resolved
Hide resolved
packages/mailgun/src/models/v1/emails/request/send-email-request/send-email-request.ts
Show resolved
Hide resolved
Are you sure for destination branch ? |
This is the target implementation of what should be generated from the OAS and the configuration.
There is a change in the API design, where the path parameters are now part of the request parameters and not in the request data object.
The models are also declined in a SDK version and an API version with associated transformer methods.