Skip to content

[codex] share modal shell across dialogs - #787

Open
saurabhhhcodes wants to merge 1 commit into
mohitkumhar:mainfrom
saurabhhhcodes:refactor/issue-419-shared-modal-shell
Open

[codex] share modal shell across dialogs#787
saurabhhhcodes wants to merge 1 commit into
mohitkumhar:mainfrom
saurabhhhcodes:refactor/issue-419-shared-modal-shell

Conversation

@saurabhhhcodes

Copy link
Copy Markdown
Contributor

Summary

  • introduce a shared app_modal_shell helper
  • route notes modal, profile import modal, and admin delete modal through the shared shell
  • keep the existing profile-specific shell for the complex profile dialogs

Closes #419

Validation

  • python3 -m pytest tests/test_modal_macro.py -q
  • python3 - <<'PY' Jinja parse check for templates/_macros.html, templates/macros/modals.html, templates/profile.html, and templates/admin/dashboard.html
  • node -c static/js/admin.js
  • git diff --check

@vercel

vercel Bot commented Jul 4, 2026

Copy link
Copy Markdown

Someone is attempting to deploy a commit to the mohitkumhar's projects Team on Vercel.

A member of the Team first needs to authorize it.

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.

1 participant