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

Adds Tag support to plugin. #67

Closed
wants to merge 106 commits into from
Closed

Adds Tag support to plugin. #67

wants to merge 106 commits into from

Conversation

pgraham3
Copy link
Contributor

@pgraham3 pgraham3 commented Sep 9, 2020

Adds X-PM-Tag as a 'recognized header' so users can set a Tag when sending with the plugin.

Patrick Graham and others added 26 commits September 9, 2020 09:29
…e-for-test-emails

Decode HTML entities in site titles for test emails.
v1.13.5 - Handle apostrophes in From name.
Since both page-settings.php and wp-mail.php (class Postmark_Mail) need these settings a new file is created that will return an array of settings. The array of settings coming from the database is merged with the settings defined as constants, with constants taking priority.
In the WP Dashboard, show the user if settings have been overridden by constants, but don’t show them the actual constant values. This will somehwat obfuscate the API key from the user. (Think agencies that define constants for a client using the agencies sandboxed api key.)
Tested on WordPress v5.8.
@DorZuberi
Copy link
Contributor

Anything new about this PR?
Adding a Tag is very basic and required option...

@pgraham3
Copy link
Contributor Author

@DorZuberi I'll try to find some time to resurrect this PR this week.

@DorZuberi
Copy link
Contributor

@pgraham3 great, I would add a filter to the headers as well.

@pgraham3 pgraham3 mentioned this pull request Mar 15, 2023
@pgraham3
Copy link
Contributor Author

Moving changes to a fresh PR for better diff. #103

@pgraham3 pgraham3 closed this Mar 15, 2023
@pgraham3 pgraham3 deleted the add-tag-support branch March 15, 2023 21:44
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.

7 participants