This repository was archived by the owner on May 29, 2026. It is now read-only.
fix: cascade error messages, settings validation, deep links, creation policy (#263 #265 #266 #267)#278
Merged
Conversation
…eation policy - #263 deleteProject cascade error messages now explicitly tell the user some data may remain and to retry, rather than silently halting. - #265 allow_project_creation is now read by ProjectList: 'admins' policy hides the New project button and creation dialog from non-admin users. Added a UI control in Settings.vue so admins can configure the policy. - #266 SettingsService.setAdminSettings validates default_columns as a non-empty JSON array of non-empty strings before persisting; malformed values are rejected with a logger warning instead of being stored verbatim. New validateDefaultColumns() method is public and testable. - #267 DeepLinkRegistrationListener URL templates switched from hash-mode (#/tasks/{uuid}) to history-mode paths (/apps/planix/projects/{uuid}) matching the router configuration. All five hardcoded 'planix' strings replaced with Application::APP_ID.
This was referenced May 27, 2026
[LOW] Documentation link points to conduction.nl marketing site instead of planix.conduction.nl
#271
Closed
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
ProjectList.vuenow reads the setting from the settings store; when set to'admins', the New project button, creation dialog mount, and empty-state CTA are all hidden from non-admin users. Added a UI control inSettings.vueso admins can configure the policy.SettingsService.setAdminSettingsnow callsvalidateDefaultColumns()before persisting; malformed or invalid shapes are rejected with a logger warning.DeepLinkRegistrationListenerURL templates switched from hash-mode (#/path) to history-mode (/apps/planix/path) matching the Vue Router configuration. All five hardcoded'planix'literals replaced withApplication::APP_ID(closes [LOW] DeepLinkRegistrationListener hardcodes 'planix' instead of using Application::APP_ID #273 overlap).Test plan