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

fix(deps): update dependency resend to v3 #11040

Merged
merged 2 commits into from
Jul 21, 2024
Merged

fix(deps): update dependency resend to v3 #11040

merged 2 commits into from
Jul 21, 2024

Conversation

Josh-Walker-GM
Copy link
Collaborator

This PR upgrade the resend mail handler to use the latest v3 resend node SDK.

There are two major bumps here:

Both change types and the v3 change also make some changes to the contacts api but our mail handler doesn't wrap this.

This is a major change given that it's possible that users would be broken by the underlying change.

@Josh-Walker-GM Josh-Walker-GM added release:chore This PR is a chore (means nothing for users) changesets-ok Override the changesets check labels Jul 21, 2024
@Josh-Walker-GM Josh-Walker-GM added this to the next-release-major milestone Jul 21, 2024
@Josh-Walker-GM Josh-Walker-GM self-assigned this Jul 21, 2024
Comment on lines +11 to +17
// Note: Resend SDK no longer exports the type Tag but it's simple enough to copy
// out here. It just makes us a little more susceptible to getting out of sync if
// the Resend SDK changes.
tags?: {
name: string
value: string
}[]
Copy link
Collaborator Author

@Josh-Walker-GM Josh-Walker-GM Jul 21, 2024

Choose a reason for hiding this comment

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

I'm not thrilled with this, I'd rather continue to use an exported type from the package, but such is life.

@Josh-Walker-GM Josh-Walker-GM marked this pull request as ready for review July 21, 2024 20:35
@Josh-Walker-GM Josh-Walker-GM merged commit 52e961c into main Jul 21, 2024
46 of 52 checks passed
@Josh-Walker-GM Josh-Walker-GM deleted the jgmw/resend-v3 branch July 21, 2024 20:36
dac09 added a commit to dac09/redwood that referenced this pull request Jul 22, 2024
…-dual-pkg

* 'main' of github.com:redwoodjs/redwood:
  fix(deps): update dependency ts-morph to v23 (redwoodjs#11044)
  chore(deps): update dependency typescript to v5.5.3 (redwoodjs#11035)
  Update saving-data.md (redwoodjs#11043)
  chore(deps): update chore (redwoodjs#11042)
  fix(deps): update dependency ts-morph to v22 (redwoodjs#11041)
  fix(deps): update dependency resend to v3 (redwoodjs#11040)
  chore(deps): update dependency lerna to v8.1.7 (redwoodjs#11039)
  fix(deps): update dependency http-proxy-middleware to v3 (redwoodjs#11038)
  chore(deps): update dependency ora to v8 (redwoodjs#11037)
  fix(deps): update dependency uuid to v10 (redwoodjs#11036)
  chore(deps): update dependency jsdom to v24.1.1 (redwoodjs#11033)
  chore(test): Document vitest issue in cli-helper test (redwoodjs#11032)
@Josh-Walker-GM Josh-Walker-GM added release:breaking This PR is a breaking change and removed release:chore This PR is a chore (means nothing for users) labels Jul 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changesets-ok Override the changesets check release:breaking This PR is a breaking change
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

1 participant