-
Notifications
You must be signed in to change notification settings - Fork 5.6k
Onlyoffice docs link updates #18941
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
Onlyoffice docs link updates #18941
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. 2 Skipped Deployments
|
WalkthroughThis PR updates OnlyOffice Docspace component metadata: documentation URLs in descriptions now point to new API-backend/usage-api paths, and several component versions (actions, sources, package) are incremented. No functional logic or control flow changes. Changes
Sequence Diagram(s)(omitted — changes are metadata-only and do not modify control flow) Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes
Poem
Pre-merge checks and finishing touches❌ Failed checks (1 inconclusive)
✅ Passed checks (1 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (7)
components/onlyoffice_docspace/actions/create-room/create-room.mjs(1 hunks)components/onlyoffice_docspace/actions/invite-user/invite-user.mjs(1 hunks)components/onlyoffice_docspace/actions/upload-file/upload-file.mjs(1 hunks)components/onlyoffice_docspace/package.json(1 hunks)components/onlyoffice_docspace/sources/new-file/new-file.mjs(1 hunks)components/onlyoffice_docspace/sources/new-folder/new-folder.mjs(1 hunks)components/onlyoffice_docspace/sources/new-user-added/new-user-added.mjs(1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2025-01-23T03:55:15.166Z
Learnt from: GTFalcao
Repo: PipedreamHQ/pipedream PR: 15376
File: components/monday/sources/name-updated/name-updated.mjs:6-6
Timestamp: 2025-01-23T03:55:15.166Z
Learning: Source names in Monday.com components don't need to start with "New" if they emit events for updated items (e.g., "Name Updated", "Column Value Updated") rather than new items. This follows the component guidelines exception where the "New" prefix is only required when emits are limited to new items.
Applied to files:
components/onlyoffice_docspace/sources/new-file/new-file.mjs
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)
- GitHub Check: Lint Code Base
- GitHub Check: pnpm publish
- GitHub Check: Publish TypeScript components
- GitHub Check: Verify TypeScript components
🔇 Additional comments (7)
components/onlyoffice_docspace/package.json (1)
3-3: LGTM! Package version bumped appropriately.The version increment aligns with the metadata updates across component files.
components/onlyoffice_docspace/sources/new-file/new-file.mjs (1)
8-9: LGTM! Metadata updated appropriately.Documentation URL migrated to the new API backend path and version incremented.
components/onlyoffice_docspace/actions/upload-file/upload-file.mjs (1)
7-8: LGTM! Documentation and version updated correctly.The description now points to the new API backend documentation path, and the version has been incremented appropriately.
components/onlyoffice_docspace/actions/invite-user/invite-user.mjs (1)
6-7: LGTM! Metadata updated to reference new documentation.Documentation URL has been migrated to the API backend usage path, and version incremented.
components/onlyoffice_docspace/sources/new-folder/new-folder.mjs (1)
9-9: Version update looks good.components/onlyoffice_docspace/sources/new-user-added/new-user-added.mjs (1)
7-8: LGTM! Documentation URL and version updated appropriately.Metadata changes align with the OnlyOffice documentation migration to the new API backend structure.
components/onlyoffice_docspace/actions/create-room/create-room.mjs (1)
6-7: Documentation URL verified and accessible.The new OnlyOffice API backend documentation URL (https://api.onlyoffice.com/docspace/api-backend/usage-api/create-room/) returns HTTP 200, confirming it is accessible and points to the correct resource. Version has been appropriately incremented to 0.0.3.
Requested by OnlyOffice, docs were updated. Can merge after approving, no need for QA
Summary by CodeRabbit
Documentation
Chores