Update Ghost to 5.2.4, add release notes, add mail config into env file #2

2022-07-12T13:17:38Z

xynosis commented

2022-07-12 13:17:38 +00:00

Previous recipe had no versioning, so I've tentatively marked this as 0.1.0+5.2.4 as I've not tested for long enough yet to be comfy with calling this 1.0.0. Everything appears stable so far, though, so could maybe be bumped to 1.0.0 if someone else can also use this recipe and test on their server.

xynosis added 5 commits 2022-07-12 13:17:39 +00:00

Add mail settings to env file b316127970

Update Ghost to 5.2.4 e5e5fe40f2

Add git tag acd5017bc4

Add release notes aaec04db86

Make readme more specific e317f23198

decentral1se approved these changes 2022-07-18 08:16:47 +00:00

decentral1se left a comment

Solid, nice work @xynosis!

One thing to note: we typically separate the SMTP config out into a compose.smtp.yml so that folks doing a new deploy doing a new deployment don't automatically have to fight a mail server. If they want mail, they can configure it as an addition. An example would be: https://git.coopcloud.tech/coop-cloud/matrix-synapse/src/branch/main/compose.smtp.yml.

This change looks fine tho, not asking you to re-jig it unless you want to. Merge away as you see fit.

Solid, nice work @xynosis! One thing to note: we typically separate the SMTP config out into a `compose.smtp.yml` so that folks doing a new deploy doing a new deployment don't automatically have to fight a mail server. If they want mail, they can configure it as an addition. An example would be: https://git.coopcloud.tech/coop-cloud/matrix-synapse/src/branch/main/compose.smtp.yml. This change looks fine tho, not asking you to re-jig it unless you want to. Merge away as you see fit.

decentral1se commented

2022-11-14 22:46:44 +00:00

@xynosis shall we merge this as-is?

xynosis merged commit f5fee003b3 into main

2022-11-16 13:10:50 +00:00

xynosis referenced this issue from a commit

2022-11-16 13:10:50 +00:00

Merge pull request 'Update Ghost to 5.2.4, add release notes, add mail config into env file' (#2) from xynosis/ghost:main into main

xynosis commented

2022-11-16 13:12:39 +00:00

Thanks for the prompt @decentral1se - had completely forgotten these hadn't been merged! I've merged as is for now, but whenever I get round to updating for a newer version of Ghost I'll separate out into compose.smtp.yml.

Thanks for the prompt @decentral1se - had completely forgotten these hadn't been merged! I've merged as is for now, but whenever I get round to updating for a newer version of Ghost I'll separate out into `compose.smtp.yml`.

❤️ 1

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: coop-cloud/ghost#2

Allow edits from maintainers