coop-cloud/lasuite-docs
2
0
Fork 1
Code Issues 5 Pull Requests Actions Packages Projects Releases Wiki Activity

Add email integration working with secrets #9

Merged
notplants merged 1 commits from email-integration into main 2025-11-18 17:17:53 +00:00
Conversation 3 Commits 1 Files Changed 3 +16 -2
notplants commented 2025-11-17 17:48:34 +00:00
Owner
Copy Link

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

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

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

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

image.png

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 deleted branch email-integration 2025-11-18 17:17:57 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something is not working
duplicate
This issue or pull request already exists
enhancement
New feature
help wanted
Need some help
invalid
Something is wrong
question
More information is needed
wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
3wordchant aadil abra-bot ammaratef45 Apfelwurm appletalk arjan basebuilder Brooke carla cas codegod100 coopcloud cyrnel decentral1se fauno flancian Frando iexos javielico jmakdah2 joe-irving kawaiipunk knoflook kolaente lambdabundesverband marlon mayel mirsal moritz nicksellen notplants p4u1 pau pharaohgraphy renovate-bot ripclap rix rscmbbng sef simon sixsmith stevensting tobias trav val virtualboys wolcen wykwit xynosis yksflip
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: coop-cloud/lasuite-docs#9
No description provided.
Delete Branch "email-integration"

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?