Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

feat: display sandbox in the docs for quick visualization #21

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

feat: display sandbox in the docs for quick visualization #21

wants to merge 9 commits into from

Conversation

grabbou
Copy link
Contributor

@grabbou grabbou commented Jan 24, 2025 •
edited
Loading

  • Creates ui package instead of builder Vite app
  • Deploys ui inside documentation as Sandbox
  • Configures Starlight to use Tailwind
  • Removes redundant styles
  • Updates styling on the Sandbox so margin: 0 is not required on a custom card

TBD:

  • Add Monaco support with proper syntax highlighting
  • Do not require flows-ai to be imported from CDN (string/replace with bundled version?)

Considerations:

  • In the future, we should allow drag&drop of a file where flow is defined

CleanShot 2025-01-25 at 17 52 38@2x

@cloudflare-workers-and-pages Cloudflare Workers and Pages
Copy link

cloudflare-workers-and-pages bot commented Jan 25, 2025 •
edited
Loading

Deploying flows-ai with  Cloudflare Pages  Cloudflare Pages

Latest commit: a49ee31
Status: ✅  Deploy successful!
Preview URL: https://391e9e78.flows-ai.pages.dev
Branch Preview URL: https://feat-docs-sandbox.flows-ai.pages.dev

View logs

@grabbou grabbou requested a review from Q1w1N January 25, 2025 11:08
@grabbou grabbou marked this pull request as ready for review January 25, 2025 16:53
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@pkarw pkarw pkarw approved these changes

@Q1w1N Q1w1N Awaiting requested review from Q1w1N

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@grabbou @pkarw