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

Bulk sending api #39

Merged
merged 16 commits into from
Jun 13, 2024
Merged

Bulk sending api #39

merged 16 commits into from
Jun 13, 2024

Conversation

narekhovhannisyan
Copy link
Collaborator

Motivation

Add support for Bluk Sending API to SDK.

Changes

  • Updated documentation.
  • Added usage sample.
  • Added unit tests.
  • Implemented integration.

Copy link
Contributor

@vittorius vittorius left a comment

Choose a reason for hiding this comment

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

Aren't we going to update the changelog as well?

src/__tests__/lib/api/BulkSending.test.ts Outdated Show resolved Hide resolved
src/__tests__/lib/api/BulkSending.test.ts Outdated Show resolved Hide resolved
import { MailtrapClient } from "mailtrap"

/**
* For this example to work, you need to set up a sending domain,
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* For this example to work, you need to set up a sending domain,
* For this example, you need to have ready-to-use sending domain

We introduced feature called Demo domain, so all new accounts have a pre-created domain to test sending, we might change wording here

  • or we can double check with Tudor

Copy link
Contributor

@vittorius vittorius left a comment

Choose a reason for hiding this comment

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

Again, let's add a new minor version to the Changelog and describe changes

CHANGELOG.md Outdated Show resolved Hide resolved
@narekhovhannisyan narekhovhannisyan merged commit 2bb26f7 into main Jun 13, 2024
1 check passed
@narekhovhannisyan narekhovhannisyan deleted the bulk-sending-api branch June 13, 2024 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants