Skip to content

Add preheader and amp_html support - #51

Merged
dtlinley-dyspatch merged 8 commits into
masterfrom
add-preheader-support
Jul 18, 2022
Merged

dtlinley-dyspatch merged 8 commits into
masterfrom
add-preheader-support

Conversation

@dtlinley-dyspatch

Copy link
Copy Markdown
Contributor

Description

Adds support for getting and setting the optional preheader and amp_html fields on Template Version Details.

Also includes necessary updates to libraries to get the build working again, as well as minimal development instructions.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Version 5 is no longer supported; bumping up to version 8.
With newer versions of Java we need to use Mockito >=2.18.0
New optional fields are supported allowing the client to get and set the
preheader and amp_html fields on a TemplateVersionDetails object.

The amp_html field is the AMP-enabled version of an email; in cases
where AMP is not supported the email will fall back to either the HTML
or text version of the template.
@dtlinley-dyspatch
dtlinley-dyspatch requested a review from a team as a code owner July 7, 2022 19:13
@turnerdev

Copy link
Copy Markdown

The preheader field is also returned by the render operation, so should probably update RenderedTemplate.java too

The resend test was failing sometimes because the sendlog wouldn't exist
when the test tried to resend the email. Adding a 5 second wait is
hack-y but effective until we can remove our dependency on the live
production API.
@dtlinley-dyspatch
dtlinley-dyspatch merged commit bf4e6c9 into master Jul 18, 2022
@dtlinley-dyspatch
dtlinley-dyspatch deleted the add-preheader-support branch July 18, 2022 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants