Add email integration working with secrets #9

2025-11-17T17:48:34Z

notplants commented

2025-11-17 17:48:34 +00:00

Continuing with all the same patterns as before, just added settings and secrets for email integration (tested & working)

Also found a nice interface in my IDE (pycharm) I'd never seen before for quickly and graphically squashing commits ... so less git messiness this time

Continuing with all the same patterns as before, just added settings and secrets for email integration (tested & working) Also found a nice interface in my IDE (pycharm) I'd never seen before for quickly and graphically squashing commits ... so less git messiness this time

notplants added 1 commit 2025-11-17 17:48:36 +00:00

Add email integration working with secrets b6f601ca24

notplants commented

2025-11-17 17:49:33 +00:00

This will complete this issue: #1

This will complete this issue: https://git.coopcloud.tech/coop-cloud/lasuite-docs/issues/1

notplants commented

2025-11-17 17:54:53 +00:00

@3wordchant I'm also not totally clear on the protocol for merging PRs here with coop cloud recipes.

Is there a defined maintainer for each recipe?

I'm always happy to receive a code review, even retrospectively on a merged PR, if you have the spoons. But should I just merge PRs optimistically if you are not actively maintaining/hosting this recipe rn?

@3wordchant I'm also not totally clear on the protocol for merging PRs here with coop cloud recipes. Is there a defined maintainer for each recipe? I'm always happy to receive a code review, even retrospectively on a merged PR, if you have the spoons. But should I just merge PRs optimistically if you are not actively maintaining/hosting this recipe rn?

3wordchant commented

2025-11-18 16:21:35 +00:00

Looks great overall!

Ideally maybe a release note about the secrets change… but practically I haven't heard of anyone except Autonomic using this recipe yet, so probably not required.

Is there a defined maintainer for each recipe?

Yes! There should be! Rollout is proceeding slowly.

There is no maintainer for this recipe yet

I'm also not totally clear on the protocol for merging PRs here with coop cloud recipes.

I think in cases where there is a maintainer, there's been discussion of making so only the maintainer(s) can merge PRs. The process is fairly undefined otherwise.

But should I just merge PRs optimistically if you are not actively maintaining/hosting this recipe rn?

Yes.

Looks great overall! Ideally maybe a release note about the secrets change… but practically I haven't heard of anyone except Autonomic using this recipe yet, so probably not required. > Is there a defined maintainer for each recipe? Yes! [There should be!](https://docs.coopcloud.tech/federation/resolutions/passed/025/) Rollout is proceeding slowly. There is no maintainer for this recipe yet ![image.png](/attachments/bac4554d-2dd5-4118-abff-00a878df76e2) > I'm also not totally clear on the protocol for merging PRs here with coop cloud recipes. I think in cases where there is a maintainer, there's been discussion of making so only the maintainer(s) can merge PRs. The process is fairly undefined otherwise. > But should I just merge PRs optimistically if you are not actively maintaining/hosting this recipe rn? Yes.

image.png

413 KiB

👍 1

notplants merged commit 16c68c085b into main

2025-11-18 17:17:53 +00:00

notplants referenced this issue from a commit

2025-11-18 17:17:55 +00:00

Merge pull request 'Add email integration working with secrets' (#9) from email-integration into main

notplants deleted branch email-integration

2025-11-18 17:17:57 +00:00

Sign in to join this conversation.