MokoConsulting/MokoGIT
1
0
Fork 0
Code Issues 146 Pull Requests Releases 96 Wiki Activity

feat(metadata): admin-configurable platform dropdown options (#777) #782

Merged
jmiller merged 1 commits from feature/metadata-platform-configurable into dev 2026-07-14 03:28:54 +00:00
Conversation 0 Commits 1 Files Changed 9
+147 -6
jmiller commented 2026-07-13 22:27:47 +00:00
Owner
Copy Link
Copy Source

Closes #777.

The repo metadata platform field was a hardcoded enum (and the list had drifted: it still had wordpress/mcp/a stray platform, missing npm). This makes it admin-configurable so taxonomy changes need no code change or redeploy.

Changes

  • modules/setting/metadata.go (new): [metadata] PLATFORM_OPTIONS setting, default joomla,dolibarr,go,npm,generic, loaded in loadCommonSettingsFrom.
  • Admin -> Metadata page (routers/web/admin/metadata.go + templates/admin/metadata.tmpl): edit the list (comma/newline input, normalized lowercase/dedupe), persisted to app.ini, applied in-memory immediately. New route + nav entry.
  • Repo Settings -> Metadata: platform dropdown now reads from the setting; a repo's current value stays selectable even if later removed from the list (no silent drop on save).
  • CHANGELOG updated.

Acceptance criteria

  • Options defined in a setting (admin UI + app.ini), not source
  • Dropdown populated from the setting
  • Add/remove/rename with no code change or redeploy
  • Existing repo values preserved (current value kept selectable)
  • Default seed joomla,dolibarr,go,npm,generic
  • REST API already accepts any configured value (no enum enforcement) — unchanged

Verification

  • go build of modules/setting, routers/web/admin, routers/web/repo/setting, routers/web: pass
  • gofmt clean

Note: package-type could get the same treatment later (issue notes this).

Closes #777. The repo metadata `platform` field was a hardcoded enum (and the list had drifted: it still had `wordpress`/`mcp`/a stray `platform`, missing `npm`). This makes it **admin-configurable** so taxonomy changes need no code change or redeploy. ### Changes - **`modules/setting/metadata.go`** (new): `[metadata] PLATFORM_OPTIONS` setting, default `joomla,dolibarr,go,npm,generic`, loaded in `loadCommonSettingsFrom`. - **Admin -> Metadata page** (`routers/web/admin/metadata.go` + `templates/admin/metadata.tmpl`): edit the list (comma/newline input, normalized lowercase/dedupe), persisted to `app.ini`, applied in-memory immediately. New route + nav entry. - **Repo Settings -> Metadata**: platform dropdown now reads from the setting; a repo's current value stays selectable even if later removed from the list (no silent drop on save). - CHANGELOG updated. ### Acceptance criteria - [x] Options defined in a setting (admin UI + `app.ini`), not source - [x] Dropdown populated from the setting - [x] Add/remove/rename with no code change or redeploy - [x] Existing repo values preserved (current value kept selectable) - [x] Default seed `joomla,dolibarr,go,npm,generic` - REST API already accepts any configured value (no enum enforcement) — unchanged ### Verification - `go build` of `modules/setting`, `routers/web/admin`, `routers/web/repo/setting`, `routers/web`: pass - `gofmt` clean Note: `package-type` could get the same treatment later (issue notes this).
jmiller added 1 commit 2026-07-13 22:27:48 +00:00
feat(metadata): admin-configurable platform dropdown options
Universal: Auto Version Bump / Version Bump (push) Successful in 11s
Details
PR RC Release / Build RC Release (pull_request) Successful in 2s
Details
Universal: PR Check / Branch Policy (pull_request) Successful in 2s
Details
Universal: PR Check / Require Docs Update (pull_request) Has been skipped
Details
Universal: PR Check / Wiki Update Reminder (pull_request) Has been skipped
Details
Generic: Project CI / Lint & Validate (pull_request) Successful in 29s
Details
Generic: Project CI / Tests (pull_request) Successful in 29s
Details
Universal: PR Check / Validate PR (pull_request) Successful in 7s
Details
Universal: PR Check / Secret Scan (pull_request) Successful in 44s
Details
Universal: PR Check / Build RC Package (pull_request) Has been cancelled
Details
Universal: PR Check / Report Issues (pull_request) Has been cancelled
Details
Branch Cleanup / Delete merged branch (pull_request) Has been cancelled
Details
RC Revert / Rename rc/ back to dev/ (pull_request) Has been cancelled
Details
506e4a49d6 
The repo metadata 'platform' field was a hardcoded enum; changing the
taxonomy meant a code change + redeploy. Make it admin-configurable:

- modules/setting/metadata.go: new [metadata] PLATFORM_OPTIONS setting
  (default joomla,dolibarr,go,npm,generic), loaded in loadCommonSettingsFrom
- routers/web/admin/metadata.go + templates/admin/metadata.tmpl: new
  Admin -> Metadata page to edit the list (persisted to app.ini), with a
  nav entry and route
- routers/web/repo/setting/metadata.go + template: platform dropdown now
  reads from the setting; a repo's current value stays selectable even if
  removed from the list (no silent drop)

Closes #777
jmiller merged commit 4b5cadfd79 into dev 2026-07-14 03:28:54 +00:00
jmiller deleted branch feature/metadata-platform-configurable 2026-07-14 03:28:55 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
breaking-change
Breaking API or behavior change
 ci-cd
CI/CD pipeline changes
 config
Configuration changes
 dependencies
Dependency updates
 deploy-failure
Deployment failed
 docker
Docker/container changes
 documentation
Documentation changes
 feature: wiki
Wiki system enhancements
 good first issue
Good for newcomers
 health-check
Repo health check result
 help wanted
Extra attention needed
 mokostandards
Related to MokoStandards framework
 push-failure
Git push operation failed
 security
Security vulnerability or hardening
 size/l
200-500 lines changed
 size/m
50-200 lines changed
 size/s
10-50 lines changed
 size/xl
500+ lines changed
 size/xs
< 10 lines changed
 standards-drift
Deviates from MokoStandards
 standards-update
MokoStandards compliance update
 sync-failure
Sync or mirror failed
 tech-debt
Technical debt and TODO/FIXME items
 upstream
upstream
Inherited from upstream Gitea
 work-in-progress
Draft or incomplete work
bug
Something is not working
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 wontfix
This won't be fixed
No labels
Priority -
Type -
Milestone
No items
No Milestone
Assignees
Clear assignees
jmiller (Jonathan Miller) moko-deploy (Moko Deploy Bot)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: MokoConsulting/MokoGIT#782
Delete Branch "feature/metadata-platform-configurable"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?