Skip to content

Conversation

@miaulalala
Copy link
Collaborator

@miaulalala miaulalala commented Nov 10, 2025

For activities with very long content, track them and add them to the setup check.

Needs nextcloud/server#56340

Fixes #2178

For activities with very long content, track them and add them to the setup check
Related server PR:

Signed-off-by: Anna Larch <[email protected]>
@cypress
Copy link

cypress bot commented Nov 10, 2025

Activity    Run #3013

Run Properties:  status check passed Passed #3013  •  git commit 11d4672c88: feat: track overly long activities
Project Activity
Branch Review feat/2178/overly-long-activities
Run status status check passed Passed #3013
Run duration 02m 51s
Commit git commit 11d4672c88: feat: track overly long activities
Committer Anna
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 10
View all changes introduced in this branch ↗︎

return SetupResult::success($this->l10n->t('No overly long activities detected.'));
}

return SetupResult::warning($this->l10n->t('There are ' . $activities . ' that generated more than 2000 characters for their content. Please check the logs for more details.'));
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
return SetupResult::warning($this->l10n->t('There are ' . $activities . ' that generated more than 2000 characters for their content. Please check the logs for more details.'));
return SetupResult::warning($this->l10n->t('There are ' . $activities . ' activities that generated more than 2000 characters for their content. Please check the logs for more details.'));

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Handle overly long activity subject and message params

2 participants