Public Archive Improvements

[justinbeaty]

This PR does two things. I can split into separate PRs if requested.

1. Adds an example.com/archive/latest page that renders the most recent newsletter.
2. Adds a <content:encoded> element into the RSS feed. (Fixes Campaign body in RSS feed #1033)

For item 2, perhaps we should add a config option for this? Something like this (mockup):

Note: I did have to add complexity to the get-archived-campaigns query, which I think is an acceptable trade off and it makes the function's description more accurate:

listmonk/internal/core/campaigns.go

Lines 140 to 141 in 684c15a

	// GetArchivedCampaigns retrieves campaigns with a template body.
	func (c *Core) GetArchivedCampaigns(offset, limit int) (models.Campaigns, int, error) {

[knadh]

This is nice! Thanks @justinbeaty. Related: #1033

Yes, we should have that as a config option. Would you be able to add a commit for that or would you want me to? We've to:

• Add a DB migration for v2.5.0 for the new config field.
• Add it to the Vue UI and in settings.go
• Add it to constants (init.go) for it to be available to the handler to decide.

[justinbeaty]

We've to:

• Add a DB migration for v2.5.0 for the new config field.
• Add it to the Vue UI and in settings.go
• Add it to constants (init.go) for it to be available to the handler to decide.

Thanks for the list of TODO, I can add the commit.

[justinbeaty]

Added the config option with default value true. I guess that's probably okay, but could change it to false if you think that's better.

[justinbeaty]

I've pushed this code to production for a few days now and all seems well. I've been testing the public rss feed with inoreader.com.

[knadh]

Thanks @justinbeaty. Will review this and merge this in the coming few days.

[knadh]

Hi @justinbeaty. I've pushed a couple of commits with minor changes. Can you review?

[justinbeaty]

Fixed a typo in i18n/en.json ("Show full full e-mail content") and re-ran scripts/refresh-i18n.sh.

I will test this out today. For my own reference, I ran these postgres queries so I can re-run the update process:

update settings set value = '["v1.0.0", "v2.0.0", "v2.1.0", "v2.2.0", "v2.3.0", "v2.4.0"]' where key = 'migrations';
delete from settings where key = 'app.enable_rss_content';

[justinbeaty]

@knadh Besides the one comment above about default true/false values, all seems good. I tested locally and everything was working.

[knadh]

Thank you once again for the excellent PR @justinbeaty!

[Sjors]

Thanks for this! Are you planning on tagging a new release in the near future?

[justinbeaty]

@Sjors per the comment here it would seem there’s a release coming. Hope you enjoy it.