docs: Update README features and documentation links#88
Merged
fsbraun merged 9 commits intodjango-cms:mainfrom Dec 12, 2025
Merged
docs: Update README features and documentation links#88fsbraun merged 9 commits intodjango-cms:mainfrom
fsbraun merged 9 commits intodjango-cms:mainfrom
Conversation
Contributor
Reviewer's GuideRefactors the README to present djangocms-rest as the primary package, updates feature/requirements/API information to match current capabilities, and restructures content to rely on external documentation instead of verbose in-file guides and examples. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #88 +/- ##
=======================================
Coverage 91.98% 91.98%
=======================================
Files 19 19
Lines 886 886
Branches 100 100
=======================================
Hits 815 815
Misses 44 44
Partials 27 27 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Contributor
There was a problem hiding this comment.
Hey there - I've reviewed your changes - here's some feedback:
- The README now suggests that
rest_frameworkdoes not need to be inINSTALLED_APPS, but the Usage section still references DRF’s browsable API—consider clarifying thatrest_frameworkmust be added toINSTALLED_APPSif users want the browsable API UI. - The README uses different localhost ports (
8080vs8000) in various sections; aligning these to a single default port will reduce confusion for new users following the setup flow. - The new API endpoints table now documents menu-related endpoints, so it might be worth double-checking that the Headless Mode considerations section doesn’t still imply menus are unavailable or only partially supported.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- The README now suggests that `rest_framework` does not need to be in `INSTALLED_APPS`, but the Usage section still references DRF’s browsable API—consider clarifying that `rest_framework` must be added to `INSTALLED_APPS` if users want the browsable API UI.
- The README uses different localhost ports (`8080` vs `8000`) in various sections; aligning these to a single default port will reduce confusion for new users following the setup flow.
- The new API endpoints table now documents menu-related endpoints, so it might be worth double-checking that the Headless Mode considerations section doesn’t still imply menus are unavailable or only partially supported.
## Individual Comments
### Comment 1
<location> `README.md:22` </location>
<code_context>
+- **Multi-Site Support** – Serve multiple websites from a single instance with isolated API responses
+- **Multi-language Content** – Use the robust i18n integration of Django CMS in your frontend
+- **Preview & Draft Access** – Fetch unpublished or draft content in your frontend for editing previews
+- **Permissions & Authentication** – Uses DRF- and Django-permissions for secure access control
+- **Menus & Breadcrumbs** – Exposes the built-in navigation handlers from Django CMS
+- **Caching & Performance** – Works with Django cache backends like Redis and Memcached
</code_context>
<issue_to_address>
**suggestion (typo):** Clarify phrasing around DRF and Django permissions and remove unnecessary hyphenation.
The phrase `Uses DRF- and Django-permissions` is awkward and the hyphen is misplaced. Consider wording it as `Uses DRF and Django permissions for secure access control` (or similar) to improve clarity and fix the hyphenation.
```suggestion
- **Permissions & Authentication** – Uses DRF and Django permissions for secure access control
```
</issue_to_address>
### Comment 2
<location> `README.md:113` </location>
<code_context>
-## Are there js packages for drop-in support of frontend editing in the javascript framework of my choice?
+## FAQ
+
+### Are there js packages for drop-in support of frontend editing in the javascript framework of my choice?
The good news first: django CMS headless mode is fully backend supported and works independently
</code_context>
<issue_to_address>
**suggestion (typo):** Capitalize "JS"/"JavaScript" in this heading for correct spelling.
In this heading, both instances are lowercase. To align with standard usage and the rest of the document, please change them to “JavaScript.”
```suggestion
### Are there JavaScript packages for drop-in support of frontend editing in the JavaScript framework of my choice?
```
</issue_to_address>
### Comment 3
<location> `README.md:183` </location>
<code_context>
Custom DRF serializers can be declared for custom plugins by setting its `serializer_class` property.
-## Does the TextPlugin (Rich Text Editor, RTE) provide a json representation of the rich text?
+### Does the TextPlugin (Rich Text Editor, RTE) provide a json representation of the rich text?
Yes, djangocms-text has both HTML blob and structured JSON support for rich text.
</code_context>
<issue_to_address>
**nitpick (typo):** Use "JSON" instead of "json" for consistency with standard capitalization.
Elsewhere in the README (e.g., "REST/JSON API") you use the capitalized form. For consistency, update "json representation" here to "JSON representation."
```suggestion
### Does the TextPlugin (Rich Text Editor, RTE) provide a JSON representation of the rich text?
```
</issue_to_address>Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
…/djangocms-rest into docs/align-with-release
fsbraun
approved these changes
Dec 12, 2025
Member
fsbraun
left a comment
There was a problem hiding this comment.
LGTM: Only one 8080 reference I noticed.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Refactored README to align with official documentation and support latest features.
Summary by Sourcery
Update the README to better describe djangocms-rest, its headless capabilities, and how to use it with updated endpoints and docs links.
Documentation: