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
✨ Maijet: retrieve subject from messages stream #30924
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Before Merging a Connector Pull RequestWow! What a great pull request you have here! 🎉 To merge this PR, ensure the following has been done/considered for each connector added or updated:
If the checklist is complete, but the CI check is failing,
|
Hi ! Regards, |
Thanks for the contribution @mickaelandrieu. Right now the team is dedicated to review Hackathon contribution during October. I added this to the backlog to revisit in the future. |
Hi, as it's a connector contribution... why not add it as part of the hackathon ? 😁 |
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.
Sorry the delay @mickaelandrieu I made a quick review.
airbyte-integrations/connectors/source-mailjet-mail/source_mailjet_mail/manifest.yaml
Show resolved
Hide resolved
ping @marcosmarxm :) |
Thank you @marcosmarxm 🤩 |
@@ -34,5 +34,6 @@ Mailjet APIs are under rate limits for the number of API calls allowed per API k | |||
|
|||
| Version | Date | Pull Request | Subject | | |||
| :------ | :--------- | :-------------------------------------------------------- | :----------------------------------------- | | |||
| 0.1.1 | 2022-04-19 | [#24689](https://github.com/airbytehq/airbyte/pull/24689) | Add listrecipient stream | | |||
| 0.1.2 | 2022-12-18 | [#30924](https://github.com/airbytehq/airbyte/pull/30924) | Adds Subject field to `message` stream | |
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.
- 2022
+ 2023
I'll fix it later
Co-authored-by: Marcos Marx <marcosmarxm@users.noreply.github.com> Co-authored-by: marcosmarxm <marcosmarxm@gmail.com>
Co-authored-by: Marcos Marx <marcosmarxm@users.noreply.github.com> Co-authored-by: marcosmarxm <marcosmarxm@gmail.com>
Co-authored-by: Marcos Marx <marcosmarxm@users.noreply.github.com> Co-authored-by: marcosmarxm <marcosmarxm@gmail.com>
Co-authored-by: Marcos Marx <marcosmarxm@users.noreply.github.com> Co-authored-by: marcosmarxm <marcosmarxm@gmail.com>
Hi,
the Mailjet connector is slow and uncomplete atm.
Regarding the completion of the fields, the Mailjet REST API disallow by default the sending of the message subject, this is why we should allow it => https://dev.mailjet.com/email/reference/messages#v3_get_message
Regards,