coop-cloud/ghost
coop-cloud
/
ghost
0
0
Fork 1
Code Issues 1 Pull Requests Projects Releases Wiki Activity

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

Merged
xynosis merged 5 commits from xynosis/ghost:main into main 2022-11-16 13:10:50 +00:00
Conversation 2 Commits 5 Files Changed 5 +28 -6
xynosis commented 2022-07-12 13:17:38 +00:00
Member
Copy Link

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.

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
decentral1se approved these changes 2022-07-18 08:16:47 +00:00
decentral1se left a comment
Owner
Copy Link

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
Owner
Copy Link

@xynosis shall we merge this as-is?

@xynosis shall we merge this as-is?
xynosis merged commit f5fee003b3 into main 2022-11-16 13:10:50 +00:00
xynosis commented 2022-11-16 13:12:39 +00:00
Author
Member
Copy Link

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.
Reviewers
No reviewers
decentral1se
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: coop-cloud/ghost#2
No description provided.
Delete Branch "xynosis/ghost:main"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?