fix: promote dev features to main #279

2026-06-29T16:34:52Z

jmiller commented

2026-06-29 16:34:52 +00:00

Summary

Add 5 new admin views (Conditions, Snippets, Replacements, Templates, Modules) with models and templates
Add 6 submenu entries to manifest so new views are navigable
Fix menu module icon overrides for actual element names
Fix heartbeat button to show proper JSON errors instead of die()
Fix togglePublished CSRF to return JSON instead of die()
Fix N+1 query in Conditions view (subselect counts)
Add .catch() error handling on all toggle-published AJAX calls
Add updates.xml to repo root for Joomla update checker
Remove dead update server migration code from install script

Supersedes #276 (closed due to merge conflicts from template sync).

https://claude.ai/code/session_01Jo2JpjCwfHAh2HHRSjczKq

## Summary - Add 5 new admin views (Conditions, Snippets, Replacements, Templates, Modules) with models and templates - Add 6 submenu entries to manifest so new views are navigable - Fix menu module icon overrides for actual element names - Fix heartbeat button to show proper JSON errors instead of die() - Fix togglePublished CSRF to return JSON instead of die() - Fix N+1 query in Conditions view (subselect counts) - Add .catch() error handling on all toggle-published AJAX calls - Add updates.xml to repo root for Joomla update checker - Remove dead update server migration code from install script Supersedes #276 (closed due to merge conflicts from template sync). https://claude.ai/code/session_01Jo2JpjCwfHAh2HHRSjczKq

jmiller added 6 commits 2026-06-29 16:34:52 +00:00

feat: add Conditions, Snippets, Replacements, Templates, Modules views 6a717342db

Five new admin views with models, templates, and list UI:
- Conditions: condition sets with group/rule counts and inline publish
- Snippets: reusable text blocks with {snippet alias} syntax
- Replacements: search/replace rules with regex and area badges
- Templates: content templates with category and description
- Modules: advanced module manager with position and client badges
Also adds togglePublished endpoint to DisplayController.

Claude-Session: https://claude.ai/code/session_01Jo2JpjCwfHAh2HHRSjczKq

fix: heartbeat button shows proper errors instead of failing silently d0db1f55dc

- CSRF check returns JSON instead of die() with raw text
- JS parses non-JSON responses gracefully and shows server error
- Visual feedback (check/cross icon) on success/failure
- 3-second icon revert after result

Claude-Session: https://claude.ai/code/session_01Jo2JpjCwfHAh2HHRSjczKq

feat: add missing submenu entries and fix menu module icon overrides 3ecdbacbeb

- Add 6 submenu items to manifest (Conditions, Snippets, Templates,
  Replacements, Automation, Modules) so views are navigable
- Fix icon overrides for actual element names (com_mokosuite_crm,
  com_mokosuite_erp, com_mokoog, com_mokoshop)
- Add icon mappings for 11 additional MokoSuite components
- Fix item-level CSS classes to match Joomla admin sidebar

Claude-Session: https://claude.ai/code/session_01Jo2JpjCwfHAh2HHRSjczKq

fix: address PR review findings - CSRF response, N+1 query, error handling 6c668dae20

- togglePublished: return JSON on CSRF failure instead of die()
- Conditions view: fold group/rule counts into main query as subselects
  instead of N+1 per-item queries
- All 5 toggle-published templates: add .catch() for AJAX error feedback

Claude-Session: https://claude.ai/code/session_01Jo2JpjCwfHAh2HHRSjczKq

fix: add updates.xml and remove dead update server migration code

Universal: Pre-Release / Build Pre-Release (${{ inputs.stability || github.ref_name }}) (push) Successful in 15s

Details

ce1541208c

Add updates.xml to repo root for Joomla update checker.
Remove unused migrateUpdateServerUrls(), fixUpdateRecords(), and
cleanupStaleUpdateSites() methods from install script.

Claude-Session: https://claude.ai/code/session_01Jo2JpjCwfHAh2HHRSjczKq

chore(version): pre-release bump to 02.52.13-dev [skip ci]

Universal: Build & Release / Promote to RC (pull_request) Has been skipped

Details

Universal: Build & Release / Build & Release Pipeline (pull_request) Successful in 19s

Details

RC Revert / Rename rc/ back to dev/ (pull_request) Has been skipped

Details

Branch Cleanup / Delete merged branch (pull_request) Successful in 2s

Details

Universal: Workflow Sync Trigger / Sync workflows to live repos (pull_request) Failing after 3m0s

Details

c452077269

jmiller merged commit 15510cfa1a into main

2026-06-29 16:35:23 +00:00

jmiller deleted branch fix/promote-dev-features

2026-06-29 16:35:24 +00:00

Sign in to join this conversation.

Priority -

Type -

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: MokoConsulting/MokoSuiteClient#279