Skip to content

docs: drop appiumpro.com references - #1098

Merged
KazuCocoa merged 1 commit into
appium:mainfrom
alaahong:chore/drop-appiumpro
Jul 26, 2026
Merged

docs: drop appiumpro.com references#1098
KazuCocoa merged 1 commit into
appium:mainfrom
alaahong:chore/drop-appiumpro

Conversation

@alaahong

@alaahong alaahong commented Jul 26, 2026

Copy link
Copy Markdown
Member

Summary

Removes appiumpro.com references from this repository, since the domain is no longer maintained by the Appium team (see appium/appium#22499).

Follows the same approach as appium/appium#22500 — links are removed rather than replaced, unless removal would break surrounding prose, in which case the URL is inlined as plain text.

Closes appium/appium#22499

Checklist

  • I have read the Contributing Guide
  • I have signed the CLA
  • Lint and unit tests pass locally with my changes
  • No documentation update needed (only stale links removed)

Copilot AI review requested due to automatic review settings July 26, 2026 02:18

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates inline documentation in the .NET client to remove an external appiumpro.com reference and keep the DirectConnect property docs self-contained.

Changes:

  • Remove the https://appiumpro.com/... URL from the DirectConnect XML documentation and replace it with a plain description.
  • Normalize the file header’s first line (removes the stray leading character/formatting artifact).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@KazuCocoa

Copy link
Copy Markdown
Member

We need to have alternative documentation for them instead of simple drop for individual pages

Copilot AI review requested due to automatic review settings July 26, 2026 04:36
@alaahong
alaahong force-pushed the chore/drop-appiumpro branch from f032e85 to a2b0e49 Compare July 26, 2026 04:36

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread src/Appium.Net/Appium/Service/AppiumClientConfig.cs
@alaahong

Copy link
Copy Markdown
Member Author

We need to have alternative documentation for them instead of simple drop for individual pages

Get the point. Caused that's the legacy link, seems no need to create new doc file into Appium docs? just replace with new link for same content reference.

@KazuCocoa
KazuCocoa merged commit 43f9001 into appium:main Jul 26, 2026
6 of 7 checks passed
@KazuCocoa KazuCocoa added the size:XS contribution size: XS label Aug 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Documentation size:XS contribution size: XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove appiumpro.com related links from our repos

3 participants