Skip to content

fix #90 - feat: Write integrator documentation#216

Open
fantonangeli wants to merge 2 commits into
serverlessworkflow:mainfrom
fantonangeli:issue-90-feat-Write-integrator-documentation
Open

fix #90 - feat: Write integrator documentation#216
fantonangeli wants to merge 2 commits into
serverlessworkflow:mainfrom
fantonangeli:issue-90-feat-Write-integrator-documentation

Conversation

@fantonangeli

Copy link
Copy Markdown
Member

Closes #90

Description

Write integrator documentation covering how to install the npm package, embed the editor component, and provide workflow content to it. This should give consumers enough guidance to get started with setup and usage.

Motivation

Make it easier for integrators to adopt the editor by providing clear guidance on installation and implementation.

Proposed Implementation

No response

Definition of Done

  • Implementation: Fully implemented according to the Serverless Workflow spec.
  • Unit Tests: Comprehensive unit tests are included and passing.
  • Integration Tests: Verified within the monorepo and target environments (Web/VS Code).
  • Documentation: Updated README.md, ADRs, or official docs.
  • Performance: No significant regression in editor responsiveness.
  • Accessibility: UI changes comply with accessibility standards.

Signed-off-by: fantonangeli <fabrizio.antonangeli@gmail.com>
Copilot AI review requested due to automatic review settings July 3, 2026 10:11
@netlify

netlify Bot commented Jul 3, 2026

Copy link
Copy Markdown

Deploy Preview for swf-editor ready!

Name Link
🔨 Latest commit c268b10
🔍 Latest deploy log https://app.netlify.com/projects/swf-editor/deploys/6a478d35a8a65f00088227ac
😎 Deploy Preview https://deploy-preview-216--swf-editor.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds integrator-facing documentation so consumers can install and embed the @serverlessworkflow/diagram-editor React component, with the repo README linking to the package-level “Getting Started” guide.

Changes:

  • Add a top-level “Getting Started” section in the root README linking to the package docs.
  • Add package-level “Getting Started” docs including React peer requirements, installation commands, a usage example, and a props table.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
README.md Adds a “Getting Started” section that points integrators to the package documentation.
packages/serverless-workflow-diagram-editor/README.md Introduces integrator documentation (requirements, install, usage example, and props).

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

Comment thread packages/serverless-workflow-diagram-editor/README.md Outdated
Signed-off-by: fantonangeli <fabrizio.antonangeli@gmail.com>
@fantonangeli fantonangeli added the documentation Improvements or additions to documentation label Jul 3, 2026
| `locale` | `'en' \| 'fr'` | Yes | `'en'` | Language locale for the editor UI |
| `colorMode` | `'light' \| 'dark' \| 'system'` | No | `'system'` | Color theme for the editor |


Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Worth adding a section on adding a thin web component wrapper for non react applications?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@lornakelly I would avoid making this README too big and complex and keep it focused on the basic React integration.
I think we can provide a non-React host example, under a new /examples dir, wrapping the DiagramEditor in a Web Component.
I can create a separate issue for this. Wdyt?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: Write integrator documentation

3 participants