Skip to content

CELDEV-1304 - add PageAttachments recursive app script and vue implementation#539

Open
fpichler wants to merge 2 commits into
devfrom
CELDEV-1304-page-attachments
Open

CELDEV-1304 - add PageAttachments recursive app script and vue implementation#539
fpichler wants to merge 2 commits into
devfrom
CELDEV-1304-page-attachments

Conversation

@fpichler

@fpichler fpichler commented May 23, 2026

Copy link
Copy Markdown
Member

Summary

Adds the new Page Attachments admin app as a recursive appScript backed by the Vue admin frontend.

This introduces a /PageAttachments/:spaceName/:docName Vue route using VueFinder with an attachments API driver, adds the matching recursive Velocity appScript entrypoint, and redirects the legacy editattachments.vm template to the new app URL.

Details

  • Adds PageAttachments.vue with VueFinder configured for page attachment browsing, upload, delete, preview, download, and search.
  • Registers the new Vue route at /PageAttachments/:spaceName/:docName.
  • Adds cel/admin/PageAttachments++.vm as the recursive appScript shell for loading the embedded admin frontend.
  • Adds cel/admin/PageAttachments_setup++.vm to resolve the target document, check view rights, and either apply the simple layout or show the login template.
  • Updates editattachments.vm to redirect to /app/cel/admin/PageAttachments/<space>/<doc>.

Merge only after #538

@fpichler fpichler requested a review from msladek May 23, 2026 00:55
@fpichler fpichler force-pushed the CELDEV-1304-page-attachments branch from 5cfd28d to 9d2cec4 Compare May 31, 2026 01:43
@fpichler fpichler force-pushed the feature/add-recursive-appScript-setup-script branch from d2f4d1e to cd664c7 Compare June 7, 2026 23:28
Base automatically changed from feature/add-recursive-appScript-setup-script to dev June 7, 2026 23:30
@fpichler fpichler force-pushed the CELDEV-1304-page-attachments branch from 099b93a to 874463a Compare June 7, 2026 23:39
@msladek msladek assigned fpichler and unassigned msladek Jun 13, 2026
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.

2 participants