Skip to content

📝(doc) Slight improvement of README.md #848

New issue

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

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

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

xibe
Copy link

@xibe xibe commented Apr 7, 2025

Purpose

I tried to make the README file a bit better and easier to read for newcomers, by rephrasing a few sentences, moving things around, and in general trying to make it clearer.

As a next step, we would certainly benefit from re-using the copy on the https://docs.numerique.gouv.fr/ site -- both because it is great, and because it would bring a consistent message throughout.

So please consider this a first pass :)

Proposed changes

  • Rewrote the headline.
  • Updated and reorganize the Write/Collaborate/Self-host sections, trying especially to remove negative wording. Also provided link to Markdown syntax, for those who don't know it.
  • Made typographic fixes here and there.

@xibe
Copy link
Author

xibe commented Apr 7, 2025

Looking around, this PR fixes #786 but not #805 nor #812.
Feel free to reuse!

@AntoLC AntoLC requested a review from virgile-dev April 8, 2025 19:27
@virgile-dev
Copy link
Collaborator

virgile-dev commented Apr 14, 2025

@xibe thanks for taking a stab at this. It looks great. Thanks
I see one minor issue with your commit message it start with a cap and it's missing a description, does not have to be long just describe in a few words what you did.
📝(doc) **S**light improvement of README.md

Also for security reasons you now need to sign your commits (doc on how to proceed)

* 📑 Export your content in multiple formats (`.odt`, `.docx`, `.pdf`) with customizable templates.
* 📚 _Coming soon! Built-in wiki functionality to turn your team's collaborative work into organized knowledge._

> ⚠️ **Note: The docx/pdf exporters rely on [packages Blocknote project](https://github.com/suitenumerique/docs/blob/main/src/frontend/apps/impress/package.json#L22C7-L23C53), which are under the AGPL-3.0 licence. Because the Docs project uses the MIT license, we recommend that you do not run Docs as a commercial product -- unless you are willing to sponsor [BlockNote](https://github.com/TypeCellOS/BlockNote). We are working on complying with the terms of the Blocknote license.**
Copy link
Collaborator

@virgile-dev virgile-dev Apr 14, 2025

Choose a reason for hiding this comment

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

Hey, I've reworked the phrasing for this. As we are working on this : #845


During Docs installation, you will be prompted if you want to install them or not. Please ensure to fulfill your obligation regarding BlockNoteJS licensing (see https://github.com/TypeCellOS/BlockNote/blob/main/packages/xl-pdf-exporter/LICENSE and https://www.blocknotejs.org/about#partner-with-us).

Also note that Docs works perfectly fine without these export features.```


## Getting started 🔧

### Test it
### Test Docs right now!
Copy link
Collaborator

@virgile-dev virgile-dev Apr 14, 2025

Choose a reason for hiding this comment

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

This was recently merged : #812 . Can you update accordingly?

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