Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@fpseverino
Copy link
Member

Resolves #59, closes #49

Copy link
Member

@ptoffy ptoffy left a comment

Choose a reason for hiding this comment

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

I'd personally be happier if we credited the original PR and merged that instead

@fpseverino
Copy link
Member Author

I'd personally be happier if we credited the original PR and merged that instead

There are conflicts to solve in the original PR, and I'm not a maintainer of this repo

@fpseverino
Copy link
Member Author

The API breakage is failing because it searches for the main branch, but in this repo the default branch is called master. Again, not being a maintainer of this repo I can't rename the branch

@fpseverino fpseverino changed the title Add encoding for replyTo field in MailgunTemplateMessage Fix compilation issues Oct 9, 2025
@fpseverino
Copy link
Member Author

I'd personally be happier if we credited the original PR and merged that instead

In the end I became a maintainer and merged the original PR 😅

@fpseverino fpseverino merged commit b9c974f into main Oct 9, 2025
24 of 26 checks passed
@fpseverino fpseverino deleted the reply-to branch October 9, 2025 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

replyTo still not encoded

2 participants