docs: release Open Self Service 1.5.0#606
Conversation
…ntrol and Zendesk integration
WalkthroughAdds a v1.5.0 release blog post; updates three integration links to point to their overview pages; and replaces anchor tags with Docusaurus Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 1 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 Fix all issues with AI agents
In `@apps/docs/blog/releases/o2s/1.5.0.md`:
- Line 97: Replace the misspelled word "benn" with "been" in the release note
sentence that currently reads "Zendesk integration docs has benn updated for
ticket creation, custom fields, and Survey.js-driven forms." so the line becomes
"Zendesk integration docs has been updated for ticket creation, custom fields,
and Survey.js-driven forms."
- Line 28: Fix the spelling error in the release note sentence that currently
reads "Global `RolesGuard` and `PermissionsGuard` are now abailable..." by
replacing "abailable" with "available" so it reads "Global `RolesGuard` and
`PermissionsGuard` are now available, so all protected endpoints can enforce
roles and permissions consistently."
- Line 64: The relative link for the Survey.js form block is incorrect: replace
the text link target "../../docs/integrations/forms/surveyjs/overview" in the
line containing "Using our [Survey.js form
block](../../docs/integrations/forms/surveyjs/overview)" so it goes up three
levels (i.e., "../../../docs/integrations/forms/surveyjs/overview") to match the
other doc links and correctly resolve the page.
🧹 Nitpick comments (1)
apps/docs/blog/releases/o2s/1.5.0.md (1)
23-23: Minor grammar improvement suggested.The phrase "an already existing Role-Based Access Control" could be simplified to "the existing Role-Based Access Control" for better readability.
✏️ Proposed grammar improvement
-We've introduced a comprehensive Permission-Based Access Control system that works together with an already existing Role-Based Access Control. You now get a full, organization-level access control: roles and permissions are kept at the organization/customer level, so the same user can have different access when switching between organizations. +We've introduced a comprehensive Permission-Based Access Control system that works together with the existing Role-Based Access Control. You now get a full, organization-level access control: roles and permissions are kept at the organization/customer level, so the same user can have different access when switching between organizations.
…n `DocLinkTiles` component
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
apps/docs/docs/integrations/overview.md (1)
8-8: Fix minor grammar typos in the overview text.Line 8 and Line 54 contain small grammatical mistakes. Please correct them for polish.
✏️ Proposed fix
-This sections provides a list of all integrations that are a part of the open-source project. +This section provides a list of all integrations that are a part of the open-source project. @@ -- [how to extend and integration](../guides/integrations/extending-integrations.md). +- [how to extend an integration](../guides/integrations/extending-integrations.md).Also applies to: 54-54
🤖 Fix all issues with AI agents
In `@apps/docs/src/components/DocLinkTiles/index.tsx`:
- Line 3: In DocLinkTiles (apps/docs/src/components/DocLinkTiles/index.tsx)
replace any usage of the `href` prop passed to the imported `Link` component
with the `to` prop so Docusaurus routing is used; locate the JSX where `Link`
(from '@docusaurus/Link') is rendered (e.g., inside the DocLinkTiles component
or any child components in this file) and rename the prop from `href={...}` to
`to={...}` (preserve the same value/variable) and remove or update any related
prop typings/props interfaces if they reference `href`.
… in O2S 1.5.0 release notes
What does this PR do?
Summary by CodeRabbit
New Features
Documentation
Chores
Bug Fixes
✏️ Tip: You can customize this high-level summary in your review settings.