Skip to content
This repository was archived by the owner on May 29, 2026. It is now read-only.

feat(preferences): generic per-user preferences endpoint#266

Merged
rubenvdlinde merged 1 commit into
developmentfrom
feat/preferences-endpoint
May 25, 2026
Merged

feat(preferences): generic per-user preferences endpoint#266
rubenvdlinde merged 1 commit into
developmentfrom
feat/preferences-endpoint

Conversation

@rubenvdlinde

Copy link
Copy Markdown
Contributor

Adds GET/PUT /api/preferences/{key} (IConfig user values, pref_*, key-sanitised) backing the shared CnSupportDialog auto-mount's per-user seen-flag (lib already on beta.101 here, so the dialog auto-mounts via CnAppRoot — this upgrades its persistence from per-browser to per-user). Mirrors pipelinq#553 / nat#68. PHPCS 0 errors.

GET/PUT /api/preferences/{key} (IConfig user values, pref_* namespace,
key-sanitised). Backs the shared nextcloud-vue CnSupportDialog
auto-mount's per-user "seen" flag. NoAdminRequired + NoCSRFRequired;
unauthenticated → 401 and the client falls back to localStorage.
@rubenvdlinde rubenvdlinde merged commit 66e2843 into development May 25, 2026
7 of 16 checks passed
@rubenvdlinde rubenvdlinde deleted the feat/preferences-endpoint branch May 25, 2026 21:21
@github-actions

Copy link
Copy Markdown
Contributor

Quality Report — ConductionNL/decidesk @ cb457a8

Check PHP Vue Security License Tests
lint
phpcs
phpmd
psalm
phpstan
phpmetrics
eslint
stylelint
composer
npm ✅ 383/383
PHPUnit ⏭️
Newman ⏭️
Playwright ⏭️

Quality workflow — 2026-05-25 21:22 UTC

Download the full PDF report from the workflow artifacts.

rubenvdlinde added a commit that referenced this pull request May 25, 2026
Reverse-specs the generic per-user preferences endpoint added in #266,
annotating getPreference/setPreference with @SPEC references. Closes the
2-method spec-coverage gap to zero. Comment-only; no behaviour change.
@rubenvdlinde rubenvdlinde restored the feat/preferences-endpoint branch May 28, 2026 21:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant