Skip to content

docs: refresh README for the unified canvas + recent features#22

Merged
vibhavkatre merged 1 commit into
mainfrom
docs/readme-refresh
Jul 20, 2026
Merged

docs: refresh README for the unified canvas + recent features#22
vibhavkatre merged 1 commit into
mainfrom
docs/readme-refresh

Conversation

@vibhavkatre

Copy link
Copy Markdown
Collaborator

Updates the README to match where the product is now:

  • One unified canvas section (Insert menu, in-frame editing, mixed document).
  • Sharing (view/comment/edit + public), Desk app launcher, and optional Frappe Drive integration added to features.
  • Frappe Cloud install section + an upgrade/rebuild note.
  • Frappe Draw logo in the header.
  • Notes the frontend now builds automatically on install (follows Fix: build the SPA on install so /draw isn't 'Page not found' #21).

Also updated the repo About description + topics via the GitHub API.

🤖 Generated with Claude Code

Rewrites the README around the current product: one unified canvas (Insert menu +
in-frame editing), Writer-style sharing, Desk app-launcher, optional Frappe Drive
integration, and a Frappe Cloud install note. Adds the Frappe Draw logo to the
header and notes that the frontend builds automatically on install.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@greptile-apps

greptile-apps Bot commented Jul 20, 2026

Copy link
Copy Markdown

Confidence Score: 4/5

Safe to merge after fixing the upgrade command — all other changes are accurate documentation.

The upgrade snippet instructs users to run bench get-app draw --branch main on an already-installed app, which will error out; only the one-line fix to bench update --app draw is needed before merging.

README.md — upgrade command block

Reviews (1): Last reviewed commit: "docs: refresh README for the unified can..." | Re-trigger Greptile

Comment thread README.md
Comment on lines +50 to +54
> Upgrading an existing install? Pull the latest and rebuild the frontend:
> ```bash
> bench get-app draw --branch main
> bench build --app draw && bench --site your-site.localhost clear-cache
> ```

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 bench get-app installs a new app and will fail if the app is already present. The correct upgrade path is bench update --app draw (or cd apps/draw && git pull).

Suggested change
> Upgrading an existing install? Pull the latest and rebuild the frontend:
> ```bash
> bench get-app draw --branch main
> bench build --app draw && bench --site your-site.localhost clear-cache
> ```
> Upgrading an existing install? Pull the latest and rebuild the frontend:
> ```bash
> bench update --app draw
> bench build --app draw && bench --site your-site.localhost clear-cache
> ```

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

@vibhavkatre
vibhavkatre merged commit 88a6221 into main Jul 20, 2026
3 checks passed
@vibhavkatre
vibhavkatre deleted the docs/readme-refresh branch July 20, 2026 09:44
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