New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix(activity): Update more activity providers to use new exceptions #48175

Open

nickvergessen wants to merge 1 commit into master from followup/48086/fix-more-activity-providers

+41 −46

Member

nickvergessen commented Sep 18, 2024

Follow up to refactor: move to new activity exception #48086

Checklist

Code is properly formatted
Sign-off message is added to all commits
Tests (unit, integration, api and/or acceptance) are included
Screenshots before/after for front-end changes
Documentation (manuals or wiki) has been updated or is not required
Backports requested where applicable (ex: critical bugfixes)

nickvergessen added the 3. to review label

nickvergessen added this to the Nextcloud 31 milestone

nickvergessen requested review from ChristophWurst, st3iny and miaulalala

September 18, 2024 14:18

nickvergessen self-assigned this


          fix(activity): Update more activity providers to use new exceptions

09fd00c

Signed-off-by: Joas Schilling <[email protected]>

nickvergessen force-pushed the followup/48086/fix-more-activity-providers branch from 2eb4efb to 09fd00c Compare

September 18, 2024 14:23

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

ChristophWurst Awaiting requested review from ChristophWurst ChristophWurst is a code owner

st3iny Awaiting requested review from st3iny st3iny is a code owner

miaulalala Awaiting requested review from miaulalala miaulalala is a code owner

At least 2 approving reviews are required to merge this pull request.

Labels