Skip to content
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

Update README.md #2267

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

Update README.md #2267

wants to merge 1 commit into from

Conversation

anuragdotexe
Copy link

What’s New:
Gave the Strapi docs a proper refresh to make things clearer, easier to navigate, and more helpful for both new users and seasoned devs. Cleaned up some clutter, broke down key sections better, and added practical tips so contributors can jump in without a hitch.

Key Tweaks:

Simplified Intro:

Reworked the intro to quickly explain what Strapi is and why it rocks. Highlighted key benefits—control over data, flexible hosting, database support, and full customization. Streamlined Navigation:

Grouped docs by versions with short descriptions on which ones are maintained and which are legacy. Contributor Guide Love:

Sharpened up the steps for contributing—clear requirements (Node.js, NPM, Yarn) and an easy-to-follow guide for pull requests. Better Help Resources:

Added clear pathways for help (forum, Discord, GitHub) and linked to video tutorials and community spaces. Release & Versioning Explained:

Explained the independent versioning starting from Strapi 5 and how updates roll out weekly. Tidied Up Layout:

Made things readable with bullet points, headers, and snappier sentences to help devs find what they need fast. Why This Matters:

For Newbies: Quick understanding, smoother onboarding.
For Contributors: Easier to jump in and make meaningful contributions.
For Everyone: A cleaner, more enjoyable read that matches the Strapi vibe.

What does it do?

Describe the technical changes you did.

Why is it needed?

Describe the issue you are solving.

Related issue(s)/PR(s)

Let us know if this is related to any issue/pull request

What’s New:
Gave the Strapi docs a proper refresh to make things clearer, easier to navigate, and more helpful for both new users and seasoned devs. Cleaned up some clutter, broke down key sections better, and added practical tips so contributors can jump in without a hitch.

Key Tweaks:

Simplified Intro:

Reworked the intro to quickly explain what Strapi is and why it rocks.
Highlighted key benefits—control over data, flexible hosting, database support, and full customization.
Streamlined Navigation:

Grouped docs by versions with short descriptions on which ones are maintained and which are legacy.
Contributor Guide Love:

Sharpened up the steps for contributing—clear requirements (Node.js, NPM, Yarn) and an easy-to-follow guide for pull requests.
Better Help Resources:

Added clear pathways for help (forum, Discord, GitHub) and linked to video tutorials and community spaces.
Release & Versioning Explained:

Explained the independent versioning starting from Strapi 5 and how updates roll out weekly.
Tidied Up Layout:

Made things readable with bullet points, headers, and snappier sentences to help devs find what they need fast.
Why This Matters:

For Newbies: Quick understanding, smoother onboarding.
For Contributors: Easier to jump in and make meaningful contributions.
For Everyone: A cleaner, more enjoyable read that matches the Strapi vibe.
@strapi-cla
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copy link

vercel bot commented Oct 19, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
documentation ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 19, 2024 9:43am

@pwizla
Copy link
Collaborator

pwizla commented Oct 21, 2024

Hi @anuragdotexe,

Thank you very much for your suggestion. I appreciate the effort! However, in its current state, your suggested revision is IMHO less readable and does not deserve the purpose you declared 😅 Quoting you from the PR description:

Tidied Up Layout:
Made things readable with bullet points, headers, and snappier sentences to help devs find what they need fast.

Please see it for yourself here 😊 https://github.com/anuragdotexe/documentation/blob/patch-1/README.md

There are no headers and no bullet points. Is this what you intended? 🤔

Copy link
Collaborator

@pwizla pwizla left a comment

Choose a reason for hiding this comment

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

As mentioned in my comment, the current edits removes all formatting. Please add at least the following:

  • headers for sections,
  • bullet points,
  • clickable links,
  • usage of other formatting options (bold, italic) where necessary

Once done, please ping me and I will perform a deeper review.

@pwizla pwizla self-assigned this Oct 21, 2024
@pwizla pwizla added pr: updated content PRs updating existing documentation content source: repo PRs/issues not targeting a specific documentation but rather affecting the whole repo labels Oct 21, 2024
@pwizla
Copy link
Collaborator

pwizla commented Nov 12, 2024

Hi @anuragdotexe, have you had the chance to look at our requested changes and move forward with this PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: updated content PRs updating existing documentation content source: repo PRs/issues not targeting a specific documentation but rather affecting the whole repo
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants