Repository navigation
fix(platform): preserve credential edits after remote deletion - #4409
yannickmonney wants to merge 1 commit into
Conversation
|
Exact-head independent review: PASS for The table-level owner keeps Edit mounted across credential refetches, derives deletion by id, retains the typed draft, shows the localized warning, and disables Save. The same Review note: the automated test file covers draft retention and reseeding but does not directly cover deletion warning/disabled Save or cross-session refetch; those remain manual coverage as recorded by |
33baa59 to
652b91f
Compare
d39bc6e to
bec2a4e
Compare
bec2a4e to
43a1929
Compare
Keep the Edit credential dialog mounted at table scope so a live refetch cannot discard its draft when the edited credential is deleted elsewhere. Show a localized warning and disable Save while retaining the draft.
Verified: scoped oxlint, oxfmt, TypeScript check. Focused Vitest could not resolve the repository @ alias in this environment.
Closes #4082
Current-main rebase
Replayed the previously accepted source
d39bc6e4e9a4ee6e98a89c67111c6a8c06c59d4fonto maind1373d84cd56972501403f62145ec52e6f65d44a, including the merged shared CI repair in #4625. The accepted feature payload and all current-main changes are preserved in one atomic commit. Configured commit and conflict checks pass; earlier behavioral proof remains recorded above. All seven native required checks and full merge-group validation remain required for this new source.Maintenance replay: preserves the accepted feature payload on main fd277c4, including merged #4649, #4650 and #4655. Retains the exact independently accepted one-line shared CLI inventory repair from #4654 (252f0df), pending native merge on main. The #4282 task-register union, where applicable, retains the accepted feature row and current-main rows. Existing behavioral evidence remains recorded above; no fresh full-feature/full-workspace or hosted-green claim. All seven native required contexts and full merge-group validation remain mandatory.