Compare commits
3 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| 6227d59eae | |||
| e238bde036 | |||
| 43c0604ac6 |
@@ -54,8 +54,11 @@ attachments:
|
||||
---
|
||||
|
||||
Campaign body goes here, in Markdown. It's sent to listmonk with
|
||||
content_type "markdown" — listmonk renders the HTML *and* derives the
|
||||
plaintext alternative itself.
|
||||
content_type "markdown" for the HTML part; listmonk never derives a
|
||||
plaintext alternative on its own, so this same source is also sent
|
||||
verbatim as `altbody` — recipients on plaintext-only clients see the
|
||||
raw Markdown (and personalization still resolves, since the `{{ }}`
|
||||
expressions are still there), rather than getting an HTML-only email.
|
||||
```
|
||||
|
||||
- **`subject`** — required.
|
||||
|
||||
@@ -217,6 +217,10 @@ func attachmentFilename(slug, relPath string, content []byte) string {
|
||||
func campaignUnchanged(existing *listmonk.Campaign, input listmonk.CampaignInput) bool {
|
||||
return existing.Subject == input.Subject &&
|
||||
existing.Body == input.Body &&
|
||||
// altbody always mirrors body (see CampaignInput.payload) -- comparing
|
||||
// it here is what catches a campaign synced before that started, whose
|
||||
// stored altbody is still null even though its body hasn't changed.
|
||||
existing.AltBody == input.Body &&
|
||||
existing.FromEmail == input.FromEmail &&
|
||||
existing.TemplateID == input.TemplateID &&
|
||||
existing.Type == input.Type &&
|
||||
|
||||
@@ -31,6 +31,7 @@ type fakeCampaign struct {
|
||||
Status string
|
||||
Subject string
|
||||
Body string
|
||||
AltBody string
|
||||
FromEmail string
|
||||
TemplateID int
|
||||
Type string
|
||||
@@ -138,7 +139,7 @@ func (c fakeCampaign) toJSON() map[string]any {
|
||||
}
|
||||
return map[string]any{
|
||||
"id": c.ID, "name": c.Name, "status": c.Status, "subject": c.Subject,
|
||||
"body": c.Body, "from_email": c.FromEmail, "template_id": c.TemplateID,
|
||||
"body": c.Body, "altbody": c.AltBody, "from_email": c.FromEmail, "template_id": c.TemplateID,
|
||||
"type": c.Type, "tags": c.Tags, "lists": lists, "media": media,
|
||||
}
|
||||
}
|
||||
@@ -155,7 +156,7 @@ func (f *fakeListmonk) createCampaign(w http.ResponseWriter, r *http.Request) {
|
||||
body := decodeBody(r)
|
||||
c := fakeCampaign{
|
||||
ID: f.id(), Name: str(body["name"]), Status: "draft",
|
||||
Subject: str(body["subject"]), Body: str(body["body"]),
|
||||
Subject: str(body["subject"]), Body: str(body["body"]), AltBody: str(body["altbody"]),
|
||||
FromEmail: str(body["from_email"]), TemplateID: toInt(body["template_id"]),
|
||||
Type: str(body["type"]), Tags: toStrings(body["tags"]),
|
||||
ListIDs: toInts(body["lists"]), MediaIDs: toInts(body["media"]),
|
||||
@@ -171,6 +172,7 @@ func (f *fakeListmonk) updateCampaign(w http.ResponseWriter, r *http.Request) {
|
||||
if f.campaigns[i].ID == id {
|
||||
f.campaigns[i].Subject = str(body["subject"])
|
||||
f.campaigns[i].Body = str(body["body"])
|
||||
f.campaigns[i].AltBody = str(body["altbody"])
|
||||
f.campaigns[i].FromEmail = str(body["from_email"])
|
||||
f.campaigns[i].TemplateID = toInt(body["template_id"])
|
||||
f.campaigns[i].Type = str(body["type"])
|
||||
|
||||
@@ -289,6 +289,7 @@ type Campaign struct {
|
||||
Status string
|
||||
Subject string
|
||||
Body string
|
||||
AltBody string
|
||||
FromEmail string
|
||||
TemplateID int
|
||||
Type string
|
||||
@@ -335,7 +336,23 @@ func (in CampaignInput) payload() map[string]any {
|
||||
"lists": in.ListIDs,
|
||||
"content_type": "markdown",
|
||||
"body": in.Body,
|
||||
// listmonk never derives a plaintext alternative from body/HTML on its
|
||||
// own (models/campaigns.go only compiles AltBodyTpl, and
|
||||
// internal/manager/message.go only emits a text/plain part, when
|
||||
// altbody is explicitly non-null) — confirmed against the deployed
|
||||
// v6.2.0 source. Reusing the same raw Markdown+template source as the
|
||||
// altbody gets it personalized identically to body (AltBodyTpl compiles
|
||||
// whenever the text contains {{ }} expressions), at the cost of
|
||||
// Markdown syntax like *emphasis* showing up literally in plaintext
|
||||
// clients — an acceptable, well-established tradeoff for Markdown.
|
||||
"altbody": in.Body,
|
||||
"type": in.Type,
|
||||
// listmonk's create/update handlers default an omitted messenger to
|
||||
// "email" before validating, but the test-send handler validates the
|
||||
// raw request body as-is — an omitted messenger there fails with
|
||||
// "Unknown messenger .". eec-campaigns only ever sends email, so set
|
||||
// it explicitly everywhere rather than relying on that asymmetry.
|
||||
"messenger": "email",
|
||||
}
|
||||
if in.FromEmail != "" {
|
||||
p["from_email"] = in.FromEmail
|
||||
@@ -365,6 +382,7 @@ func parseCampaign(data []byte) (*Campaign, error) {
|
||||
Status string `json:"status"`
|
||||
Subject string `json:"subject"`
|
||||
Body string `json:"body"`
|
||||
AltBody string `json:"altbody"`
|
||||
FromEmail string `json:"from_email"`
|
||||
TemplateID int `json:"template_id"`
|
||||
Type string `json:"type"`
|
||||
@@ -393,6 +411,7 @@ func parseCampaign(data []byte) (*Campaign, error) {
|
||||
Status: parsed.Status,
|
||||
Subject: parsed.Subject,
|
||||
Body: parsed.Body,
|
||||
AltBody: parsed.AltBody,
|
||||
FromEmail: parsed.FromEmail,
|
||||
TemplateID: parsed.TemplateID,
|
||||
Type: parsed.Type,
|
||||
|
||||
Reference in New Issue
Block a user