-
Notifications
You must be signed in to change notification settings - Fork 3
Add UiPath doc to Integrations section. #395
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
base: main
Are you sure you want to change the base?
Conversation
|
Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. WalkthroughAdds a new UiPath mobile-automation documentation page and a navigation entry for it in the integrations navigation. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10–15 minutes
Possibly related PRs
Suggested reviewers
Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: defaults Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
🧰 Additional context used🧠 Learnings (2)📚 Learning: 2025-11-26T08:47:40.925ZApplied to files:
📚 Learning: 2025-12-16T07:27:41.117ZApplied to files:
⏰ 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). (2)
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
🧹 Nitpick comments (4)
docs/modules/integrations/pages/uipath/mobile-test-uipath.adoc (4)
16-28: Consider using local section links instead of Confluence anchorsThe “Contents” list links out to specific anchors in the Confluence page, while the same sections exist locally on this page. That can create drift if one source is updated without the other.
Consider switching these to local xrefs/anchors so the ToC always points at the content in this page. For example:
-* https://kobiton.atlassian.net/...#Prerequisites[Prerequisites] + +* <<_prerequisites,Prerequisites>> +(and similar for the other sections).
56-58: Tighten wording and verify minimum UiPath package versionMinor copy edit and a quick check request for the dependency version:
- “Ensure the selected version to install is…” reads a bit awkwardly.
You can simplify to:
-In the search bar, enter *UiPath.MobileAutomation.Activities*. Select -the package and choose *Install*. Ensure the selected version to install -is 24.10.0 or later. +In the search bar, enter *UiPath.MobileAutomation.Activities*. Select +the package and choose *Install*. Ensure the selected version is +24.10.0 or later.Also, please confirm against UiPath’s latest docs that
24.10.0is still the correct minimum version to call out here.
74-76: Verify{plus}macro usage for the icon nameThe phrase “plus ({plus}) icon” relies on
{plus}being defined as an attribute. If that attribute isn’t configured globally, it may render literally instead of as+.You could avoid that dependency with:
-In the Mobile Device Manager window, select *Open Devices*, then the -*plus ({plus}) icon* to create a new device connection. +In the Mobile Device Manager window, select *Open Devices*, then the +*plus (+) icon* to create a new device connection.Same pattern applies later for the application “plus” icon.
219-221: Fix minor grammar in debug descriptionTiny grammar nit: singular subject with plural verb.
-If *Debug File* is selected, the Debug window appears while Mobile -Device Manager opens a new connection to the device and execute the -test. +If *Debug File* is selected, the Debug window appears while Mobile +Device Manager opens a new connection to the device and executes the +test.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (24)
docs/modules/integrations/images/uipath-img0000.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-img00000.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-img01.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-img02.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-img03.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-img04.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-img05.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-img06.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-img07.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-img08.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-img09.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-img10.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-img11.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-img12.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-img13.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-img14.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-img15.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-img16.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-img17.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-img18.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-img19.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-img20.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-img21.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-img22.pngis excluded by!**/*.png
📒 Files selected for processing (2)
docs/modules/integrations/nav.adoc(1 hunks)docs/modules/integrations/pages/uipath/mobile-test-uipath.adoc(1 hunks)
🧰 Additional context used
🧠 Learnings (2)
📓 Common learnings
Learnt from: tungmhoang
Repo: kobiton/docs PR: 392
File: docs/modules/release-notes/pages/all-releases/4_22.adoc:26-26
Timestamp: 2025-11-26T08:47:40.925Z
Learning: In Kobiton release notes (docs/modules/release-notes/pages/all-releases/*.adoc), it is acceptable to include placeholder text stating that user guides or documentation "will be available soon" when the feature is released but documentation is still in progress.
📚 Learning: 2025-11-26T08:47:40.925Z
Learnt from: tungmhoang
Repo: kobiton/docs PR: 392
File: docs/modules/release-notes/pages/all-releases/4_22.adoc:26-26
Timestamp: 2025-11-26T08:47:40.925Z
Learning: In Kobiton release notes (docs/modules/release-notes/pages/all-releases/*.adoc), it is acceptable to include placeholder text stating that user guides or documentation "will be available soon" when the feature is released but documentation is still in progress.
Applied to files:
docs/modules/integrations/pages/uipath/mobile-test-uipath.adoc
⏰ 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). (2)
- GitHub Check: publish-docs-test
- GitHub Check: Test Build
🔇 Additional comments (1)
docs/modules/integrations/nav.adoc (1)
49-50: UiPath nav entry looks consistent and correctThe new
UiPathentry follows the existing xref pattern and is placed logically alongside other integrations. No issues from a structure or naming standpoint.
| Kobiton currently supports mobile automation in UiPath Studio for: | ||
|
|
||
| * iOS devices (Portal 4.19.6S and above). + | ||
| * Android devices (Portal 4.21.1S and above). + | ||
|
|
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.
Change the orgiginal lines here to:
- iOS devices (Kobiton 4.19.6 and above).
- Android devices (Kobiton 4.21.1 and above).
tungmhoang
left a 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.
@tiffany-kobiton I have a few comments that need changes.
| * https://kobiton.atlassian.net/wiki/spaces/KOBITON/pages/4606885897/Mobile+Automation+with+UiPath+and+Kobiton#Connect-to-the-device-and-record-test-steps[Connect | ||
| to the device and record test steps] + | ||
| * https://kobiton.atlassian.net/wiki/spaces/KOBITON/pages/4606885897/Mobile+Automation+with+UiPath+and+Kobiton#Create-and-execute-test-with-recorded-steps[Create | ||
| and execute test with recorded steps] |
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.
Please remove the table of contents in the docs site, as it is already automatically generated based on the headings.
Also, don't refer to our confluence page in the public docs site, as end-users won't be able to accesss them.
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.
Do you mean Antora automatically generates a table of contents?
Those are the same links that the original document had.
| Kobiton currently supports mobile automation in UiPath Studio for: | ||
|
|
||
| * iOS devices (Portal 4.19.6S and above). + | ||
| * Android devices (Portal 4.21.1S and above). + | ||
|
|
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.
Change the orgiginal lines here to:
- iOS devices (Kobiton 4.19.6 and above).
- Android devices (Kobiton 4.21.1 and above).
tungmhoang
left a 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.
Some more change requests.
tungmhoang
left a 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.
More comments regarding image size
|
Requested changes made (conversations marked as resolved). Posted some questions on the Confluence doc. |
|
@tiffany-kobiton I have udpated the Confluence doc based on your comment. Please update this PR with the latest changes after you have reviewed the Confluence doc and had no more feedbacks. |
|
Also, @tiffany-kobiton looks like you have not pushed your local commit, so I cannot see the changes. |
|
Whoops, I will get that done asap.
…On Wed, Dec 17, 2025 at 1:53 AM ***@***.*** < ***@***.***> wrote:
*tungmhoang* left a comment (kobiton/docs#395)
<#395 (comment)>
Also, @tiffany-kobiton <https://github.com/tiffany-kobiton> looks like
you have not pushed your local commit, so I cannot see the changes.
—
Reply to this email directly, view it on GitHub
<#395 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/B26GPZF7GXPFTGW36BYI5ST4CD4YFAVCNFSM6AAAAACPDW4KWSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTMNRTHEZDGMJTGI>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
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: 2
📜 Review details
Configuration used: defaults
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (25)
docs/modules/integrations/images/uipath-add-a-device.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-add-application-app.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-add-application-web.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-add-new-activity.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-connection-details.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-create-connection.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-debug-activity-indicator.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-design-debug-ribbon-item.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-live-device-with-x.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-logging-enabled.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-main-activity-indicators.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-main-window.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-manage-packages.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-mobile-automation-activities-v24.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-mobile-automation-activities.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-mobile-automation-menu.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-mobile-device-connection-window.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-mobile-device-manager-applications-window.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-mobile-device-manager-live.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-mobile-device-manager-welcome-again.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-mobile-device-manager-welcome.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-mobile-device-recorded-sessions.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-mobile-device-search.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-mobile-test-mobile-automation.pngis excluded by!**/*.pngdocs/modules/integrations/images/uipath-mobile-test-properties.pngis excluded by!**/*.png
📒 Files selected for processing (1)
docs/modules/integrations/pages/uipath/mobile-test-uipath.adoc(1 hunks)
🧰 Additional context used
🧠 Learnings (2)
📚 Learning: 2025-11-26T08:47:40.925Z
Learnt from: tungmhoang
Repo: kobiton/docs PR: 392
File: docs/modules/release-notes/pages/all-releases/4_22.adoc:26-26
Timestamp: 2025-11-26T08:47:40.925Z
Learning: In Kobiton release notes (docs/modules/release-notes/pages/all-releases/*.adoc), it is acceptable to include placeholder text stating that user guides or documentation "will be available soon" when the feature is released but documentation is still in progress.
Applied to files:
docs/modules/integrations/pages/uipath/mobile-test-uipath.adoc
📚 Learning: 2025-12-16T07:27:41.117Z
Learnt from: tungmhoang
Repo: kobiton/docs PR: 395
File: docs/modules/integrations/pages/uipath/mobile-test-uipath.adoc:11-15
Timestamp: 2025-12-16T07:27:41.117Z
Learning: For all documentation files under docs/modules/integrations/pages/ (and its subdirectories) that reference platform version requirements for device support, use 'Kobiton' as the product name (not 'Portal') and express versions without the trailing 'S'. Example: 'Kobiton 4.19.6 and above' instead of 'Portal 4.19.6S and above'.
Applied to files:
docs/modules/integrations/pages/uipath/mobile-test-uipath.adoc
⏰ 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). (2)
- GitHub Check: Test Build
- GitHub Check: publish-docs-test
| Kobiton currently supports mobile automation in UiPath Studio for: | ||
|
|
||
| * iOS devices (Kobiton 4.19.6 and above). | ||
| * Android devices (Kobiton 4.21.1 and newer, including 4.21.1S). |
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.
Remove the trailing 'S' from the version number on line 14.
Per the documentation guidelines, version numbers should not include the trailing 'S' suffix. Change "4.21.1S" to "4.21.1". Based on learnings, versions should be expressed consistently without the 'S' suffix (e.g., 'Kobiton 4.21.1 and newer' instead of 'Kobiton 4.21.1S').
Apply this diff:
-* Android devices (Kobiton 4.21.1 and newer, including 4.21.1S).
+* Android devices (Kobiton 4.21.1 and newer).🤖 Prompt for AI Agents
In docs/modules/integrations/pages/uipath/mobile-test-uipath.adoc around line
14, remove the trailing 'S' from the version string so the line reads "Android
devices (Kobiton 4.21.1 and newer, including 4.21.1)" — update the text to
replace "4.21.1S" with "4.21.1" to follow the documentation guideline of
omitting the 'S' suffix.
Summary
Related PRs, issues, or features (optional)
Metadata
PR contributor checklist
My PR follows the Kobiton Docs contributor guidelines, meaning:
nav.adocfile.en-us) from my URLs.Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.