pleroma/pleroma
5
5
Fork 8
Code Issues 775 Pull requests 73 Projects Releases 1 Packages Wiki Activity

DigestEmailsWorker: Change Oban queue to "background" #7865

Merged
feld merged 2 commits from phnt/oban-digest-queue into develop 2026-04-01 19:08:40 +00:00
Conversation 3 Commits 2 Files changed 3 +11 -1
phnt commented 2026-03-26 22:38:16 +00:00
Owner
Copy link

The mailer queue has been long gone and that left Oban jobs always
stuck in the "available" state that would never execute.

Checklist

  • Adding a changelog: In the changelog.d directory, create a file named <code>.<type>.
The mailer queue has been long gone and that left Oban jobs always stuck in the "available" state that would never execute. ### Checklist - [ ] Adding a changelog: In the `changelog.d` directory, create a file named `<code>.<type>`. <!-- `<code>` can be anything, but we recommend using a more or less unique identifier to avoid collisions, such as the branch name. `<type>` can be `add`, `change`, `remove`, `fix`, `security` or `skip`. `skip` is only used if there is no user-visible change in the PR (for example, only editing comments in the code). Otherwise, choose a type that corresponds to your change. In the file, write the changelog entry. For example, if a PR adds group functionality, we can create a file named `group.add` and write `Add group functionality` in it. If one changelog entry is not enough, you may add more. But that might mean you can split it into two PRs. Only use more than one changelog entry if you really need to (for example, when one change in the code fix two different bugs, or when refactoring). -->
DigestEmailsWorker: Change Oban queue to "background"
Some checks are pending
ci/woodpecker/pr/woodpecker Pipeline is pending
Details
83293dc554
 
The mailer queue has been long gone and that left Oban jobs always
stuck in the "available" state that would never execute.
phnt force-pushed phnt/oban-digest-queue from 83293dc554
Some checks are pending
ci/woodpecker/pr/woodpecker Pipeline is pending
Details
to 799199f6b5
Some checks failed
ci/woodpecker/pr/woodpecker Pipeline failed
Details
2026-03-26 22:38:27 +00:00 Compare
feld commented 2026-04-01 18:47:12 +00:00
Owner
Copy link

I have my head in my hands right now and I'm weeping for missing this

edit: this also tells me nobody cares about digest emails...

I have my head in my hands right now and I'm weeping for missing this edit: this also tells me nobody cares about digest emails...
Fix the daily email digest job which was not executing
Some checks failed
ci/woodpecker/pr/woodpecker Pipeline failed
Details
01ced6bea2
feld changed title from WIP: DigestEmailsWorker: Change Oban queue to "background" to DigestEmailsWorker: Change Oban queue to "background" 2026-04-01 19:08:29 +00:00
feld merged commit a3404e91bc into develop 2026-04-01 19:08:40 +00:00
feld deleted branch phnt/oban-digest-queue 2026-04-01 19:08:41 +00:00
phnt commented 2026-04-01 19:35:03 +00:00
Author
Owner
Copy link

@feld wrote in #7865 (comment):

edit: this also tells me nobody cares about digest emails...

Yeah, there's also a missing Gettext translation or something I'll look into later, which is why this PR was WIP. Besides the fact that I misconfigured my email setup and haven't received a digest for a test account, which should hopefully arrive today...

@feld wrote in https://git.pleroma.social/pleroma/pleroma/pulls/7865#issuecomment-115206: > edit: this also tells me nobody cares about digest emails... Yeah, there's also a missing Gettext translation or something I'll look into later, which is why this PR was WIP. Besides the fact that I misconfigured my email setup and haven't received a digest for a test account, which should hopefully arrive today...
feld commented 2026-04-01 21:24:51 +00:00
Owner
Copy link

apologies for hijacking the PR then, just assumed it was WIP from no migration or changelog :)

we can chase the Gettext issue now that the job will actually execute...

apologies for hijacking the PR then, just assumed it was WIP from no migration or changelog :) we can chase the Gettext issue now that the job will actually execute...
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
AP C2S

   
AdminAPI

   
Breaking API

Affects clients

  
Breaks existing config

   
Check if still valid

   
Data Protection

   
Do not merge

   
Document in the changelog

   
Doing

   
Feature Request / Enhancement

   
MastoAPI

   
NL2

Improved Mobile Experience

  
NL4

Improved instance customization and QoL options

  
OStatus

   
OTP Releases

   
Refactor

   
Security

   
To Do

   
TwitterAPI

   
bug

   
complicated

   
confirmed

   
deps

   
discussion

   
documentation

   
easy

   
feature: to be merged in stable

   
fix: to be merged in stable

   
incident

Denotes a disruption to IT services and the associated issues require immediate attention

  
needs to be backported to maint

   
needs-changes

   
needs-info

   
needs-review

   
performance

   
regression

   
unclear
No labels
AP C2S
AdminAPI
Breaking API
Breaks existing config
Check if still valid
Data Protection
Do not merge
Document in the changelog
Doing
Feature Request / Enhancement
MastoAPI
NL2
NL4
OStatus
OTP Releases
Refactor
Security
To Do
TwitterAPI
bug
complicated
confirmed
deps
discussion
documentation
easy
feature: to be merged in stable
fix: to be merged in stable
incident
needs to be backported to maint
needs-changes
needs-info
needs-review
performance
regression
unclear
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
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
pleroma/pleroma!7865
No description provided.