Skip to content

feat(addons): generic per-addon settings store (AddonSetting + addon_settings + settings UI) - #22

Merged
Panelica merged 2 commits into
Panelica:mainfrom
hedon77:feat/addon-settings
Sep 1, 2026
Merged

feat(addons): generic per-addon settings store (AddonSetting + addon_settings + settings UI)#22
Panelica merged 2 commits into
Panelica:mainfrom
hedon77:feat/addon-settings

Conversation

@hedon77

@hedon77 hedon77 commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

What

Extracts the reusable addon-framework settings layer so addons can store configuration (API keys, endpoints, toggles) without depending on a specific addon.

  • \AddonSetting\ model (values encrypted at rest via \EncryptedValue\ cast).
  • \�ddon_settings\ migration (one row per addon/setting).
  • \AddonManager::settings()\ / \saveSettings().
  • \AddonController::saveSettings()\ + settings passed to the addon page.
  • Settings form on the addon page (text, textarea, password, checkbox, select).
  • \�ddons.modules.settings\ route.
  • Renames the admin section from \Addon Modules\ to \Extensions.

Why

This is the shared base for the KSeF (#20) and Company Lookup (#19) addons. It lets each addon live in its own PR without duplicating the settings plumbing.

Notes

@hedon77
hedon77 force-pushed the feat/addon-settings branch from 84bc2bd to ca05f7b Compare September 1, 2026 00:21
@hedon77

hedon77 commented Sep 1, 2026

Copy link
Copy Markdown
Contributor Author

Review — ready to merge

Generic addon settings store (AddonSetting + addon_settings migration + settings UI). Clean, framework-only.

Sugestia: zmerguj jako pierwszy — jest bazą dla #19 (company-lookup) i #20 (KSeF).

@Panelica
Panelica merged commit 3a06717 into Panelica:main Sep 1, 2026
1 check passed
Panelica pushed a commit that referenced this pull request Sep 1, 2026
Add a CHANGELOG entry for the merged billing/extensibility work (#14 tax
model, #22 addon settings framework, #13 Tpay gateway, #16 client-delete
security fix, #24 invoice number display) and list Tpay in the modules
table and gateway overview.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants