Skip to content

fix(activitydata): use parameter validation regex from server#9398

Merged
mgallien merged 2 commits intomasterfrom
bugfix/9327/activity-placeholder
Feb 2, 2026
Merged

fix(activitydata): use parameter validation regex from server#9398
mgallien merged 2 commits intomasterfrom
bugfix/9327/activity-placeholder

Conversation

@nilsding
Copy link
Member

Since 31.0.0 the rich parameters are subject to validation (starting with a letter; only letters, digits, dashes, underscores, and periods may follow). Reuse the server's validation regexp for these parameters in the activity data.

Resolves #9327.

@nilsding nilsding added this to the 33.0.0 milestone Jan 30, 2026
@nilsding nilsding self-assigned this Jan 30, 2026
@nilsding nilsding added the bug label Jan 30, 2026
@nilsding nilsding requested a review from mgallien as a code owner January 30, 2026 10:06
@nilsding
Copy link
Member Author

/backport to stable-4.0

@mgallien mgallien enabled auto-merge February 2, 2026 08:51
Signed-off-by: Jyrki Gadinger <nilsding@nilsding.org>
Since 31.0.0 the rich parameters are subject to validation (starting
with a letter; only letters, digits, dashes, underscores, and periods
may follow).  Reuse the server's validation regexp for these parameters
in the activity data.

Resolves #9327.

Signed-off-by: Jyrki Gadinger <nilsding@nilsding.org>
@nilsding nilsding force-pushed the bugfix/9327/activity-placeholder branch from 72b5f4b to 737547c Compare February 2, 2026 10:11
@github-actions
Copy link

github-actions bot commented Feb 2, 2026

Artifact containing the AppImage: nextcloud-appimage-pr-9398.zip

Digest: sha256:db9f9f9d58c4fa3f5f2089b9879b1e004d6ea7ec31b830fa5d26ba2103d02456

To test this change/fix you can download the above artifact file, unzip it, and run it.

Please make sure to quit your existing Nextcloud app and backup your data.

@mgallien mgallien merged commit 6771ee2 into master Feb 2, 2026
21 of 22 checks passed
@mgallien mgallien deleted the bugfix/9327/activity-placeholder branch February 2, 2026 10:58
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 2, 2026

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.

[Bug]: Lack of placeholder filling for Table app activities

2 participants