Fediversity/Fediversity
12
1
Fork 3
Code Issues 22 Pull requests 2 Actions Projects 1 Activity

Set up mailing for Forgejo #37

Manually merged
Niols merged 1 commit from mail-forgejo into main 2024-11-27 18:12:22 +01:00
Conversation 5 Commits 1 Files changed 1 +12 -8
Niols commented 2024-11-27 14:24:06 +01:00
Owner
Copy link

Fixes #27.

Fixes https://git.fediversity.eu/Fediversity/Fediversity/issues/27.
Niols added 1 commit 2024-11-27 14:24:08 +01:00
Set up mailing for Forgejo
Some checks failed
/ check-pre-commit (pull_request) Failing after 0s
Details
448385c8e2
Niols commented 2024-11-27 14:27:20 +01:00
Author
Owner
Copy link

CI failure, as far as I understand, has nothing to do with us, but with code.forgejo.org being down.

CI failure, as far as I understand, has nothing to do with us, but with code.forgejo.org being down.
Niols force-pushed mail-forgejo from 448385c8e2 to 506de1cf24 2024-11-27 14:43:27 +01:00 Compare
fricklerhandwerk approved these changes 2024-11-27 14:46:21 +01:00
fricklerhandwerk left a comment
Owner
Copy link

LGTM but I don't have a reasonable way of testing this

LGTM but I don't have a reasonable way of testing this
Niols commented 2024-11-27 15:06:58 +01:00
Author
Owner
Copy link

You can go to Site administration, then Configuration > Summary, scroll down to Mailer configuration, enter your email and check that you get the message. However, clearly, this does not give us notifications on issue replies and all, so maybe we should wait before merging this.

You can go to Site administration, then Configuration > Summary, scroll down to Mailer configuration, enter your email and check that you get the message. However, clearly, this does not give us notifications on issue replies and all, so maybe we should wait before merging this.
Niols changed title from Set up mailing for Forgejo to WIP: Set up mailing for Forgejo 2024-11-27 15:07:02 +01:00
Niols force-pushed mail-forgejo from 506de1cf24 to 89d25fa7a5 2024-11-27 17:34:17 +01:00 Compare
Niols commented 2024-11-27 17:39:40 +01:00
Author
Owner
Copy link

I think I have now managed. If we can confirm, then we can merge this. I'll remove the WIP: prefix.

I think I have now managed. If we can confirm, then we can merge this. I'll remove the `WIP:` prefix.
Niols changed title from WIP: Set up mailing for Forgejo to Set up mailing for Forgejo 2024-11-27 17:39:43 +01:00
hans commented 2024-11-27 18:10:37 +01:00
Owner
Copy link

Ok, mail eh?

Ok, mail eh?
Niols commented 2024-11-27 18:11:51 +01:00
Author
Owner
Copy link

Thanks @hans! Email received indeed:

image

image

Will merge.

Thanks @hans! Email received indeed: ![image](/attachments/59941eba-b14c-4d8a-8683-a61432aae1e8) ![image](/attachments/e36c5b9f-026b-43df-a91c-712c3a2740e9) Will merge.
image.png
11 KiB
image.png
24 KiB
Niols referenced this pull request from a commit 2024-11-27 18:12:22 +01:00
Set up mailing for Forgejo (#37)
Niols manually merged commit 0e7eef5ea2 into main 2024-11-27 18:12:22 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
fricklerhandwerk
Labels
Clear labels   
blocked

   
project-management

Project management related tasks

  
question
No labels
blocked
project-management
question
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
3 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: Fediversity/Fediversity#37
No description provided.
Delete branch "mail-forgejo"

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?