MokoConsulting/MokoSuiteBackup

fix: remote destinations tab broken (Add Destination + stuck loading) #208

Merged

jmiller merged 2 commits from fix/remote-destinations-modal into main

2026-07-04 22:00:46 +00:00

Author	SHA1	Message	Date
gitea-actions[bot]	9e6337d09f	chore(version): pre-release bump to 02.56.02-dev [skip ci] RC Revert / Rename rc/ back to dev/ (pull_request) Has been skipped Details Branch Cleanup / Delete merged branch (pull_request) Successful in 3s Details Universal: Build & Release / Promote to RC (pull_request) Has been skipped Details Universal: Build & Release / Build & Release Pipeline (pull_request) Successful in 27s Details Universal: Workflow Sync Trigger / Sync workflows to live repos (pull_request) Successful in 3m24s Details	2026-07-04 22:00:43 +00:00
jmiller	4ca9e23630	fix(profile): remote destinations tab broken — lazy-init Bootstrap modal Joomla: Extension CI / Release Readiness Check (pull_request) Failing after 7s Details Joomla: Extension CI / Lint & Validate (pull_request) Failing after 11s Details Universal: PR Check / Branch Policy (pull_request) Successful in 3s Details Universal: PR Check / Secret Scan (pull_request) Successful in 10s Details Generic: Repo Health / Site Health (pull_request) Has been skipped Details Universal: PR Check / Validate PR (pull_request) Failing after 14s Details Generic: Repo Health / Access control (pull_request) Successful in 3s Details Joomla: Metadata Validation / Validate Joomla Metadata (pull_request) Successful in 14s Details Universal: Pre-Release / Build Pre-Release (${{ inputs.stability \|\| github.ref_name }}) (push) Successful in 39s Details Universal: Build & Release / Promote to RC (pull_request) Has been skipped Details Universal: Build & Release / Build & Release Pipeline (pull_request) Has been skipped Details Generic: Project CI / Lint & Validate (pull_request) Successful in 45s Details Generic: Project CI / Tests (pull_request) Has been cancelled Details Joomla: Extension CI / Tests (PHP 8.2) (pull_request) Has been cancelled Details Joomla: Extension CI / Tests (PHP 8.3) (pull_request) Has been cancelled Details Joomla: Extension CI / PHPStan Analysis (pull_request) Has been cancelled Details Joomla: Extension CI / Build RC Pre-Release (pull_request) Has been cancelled Details Universal: PR Check / Build RC Package (pull_request) Has been cancelled Details Universal: PR Check / Report Issues (pull_request) Has been cancelled Details Generic: Repo Health / Scripts governance (pull_request) Has been cancelled Details Generic: Repo Health / Repository health (pull_request) Has been cancelled Details Generic: Repo Health / Report: Scripts Governance (pull_request) Has been cancelled Details Generic: Repo Health / Report: Repository Health (pull_request) Has been cancelled Details The remote destinations table stayed stuck at "Loading…" and the "Add Destination" button did nothing. Root cause: the profile-edit DOMContentLoaded handler instantiated the Bootstrap modal eagerly at the top (`bootstrap.Modal.getOrCreateInstance`). In Joomla 6, Bootstrap loads as a deferred ES module, so `bootstrap` is undefined at DOMContentLoaded — the reference threw a ReferenceError that aborted the entire handler, so loadRemotes() never ran and the Add button was never bound. (The purge and stepped-backup modals worked because they resolve the modal lazily inside click handlers.) - Resolve the modal lazily via getModal() at click-time (matches the working modals) - Explicitly load the bootstrap.modal web asset so window.bootstrap.Modal is registered Claude-Session: https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i	2026-07-04 17:00:15 -05:00