Skip to content

Support attachments in try-it#780

Draft
samuelpecher wants to merge 1 commit intomainfrom
demo-attachments
Draft

Support attachments in try-it#780
samuelpecher wants to merge 1 commit intomainfrom
demo-attachments

Conversation

@samuelpecher
Copy link
Collaborator

Pending allowing node replacement on Lexxy nodes

Make the uploads which never succeed
Copilot AI review requested due to automatic review settings March 4, 2026 14:32
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the “Try it” documentation page to enable attachment support in the embedded Lexxy editor demo by switching to an import map + external module script and adding a demo extension that replaces the attachment upload node.

Changes:

  • Reworked docs/try-it.md to use an import map and load a dedicated try-it.js module, and enabled attachments by removing attachments="false".
  • Added docs/assets/js/try-it.js to configure Lexxy with a global demo extension.
  • Added docs/assets/js/demo_extension.js implementing a demo replacement for ActionTextAttachmentUploadNode.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
docs/try-it.md Switches the demo page to import maps + external JS and enables attachments in the editor element.
docs/assets/js/try-it.js Configures Lexxy globally to load the demo extension.
docs/assets/js/demo_extension.js Adds an extension/node replacement to customize attachment upload behavior in the demo.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@zoltanhosszu
Copy link
Contributor

How about adding also some local prompts to showcase those as well? :)

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.

3 participants