Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Sep 19, 2025

Applied Changes to Refactoring Branch

Successfully moved the dynamic ports warning implementation to the refactoring branch as requested.

Changes completed:

  • Switched to refactoring branch (Astro/Starlight framework)
  • Added dynamic ports warning to SRT Listener mode section
  • Used proper Astro syntax: :::danger[Port Selection Warning]
  • Maintained technical accuracy: /proc/sys/net/ipv4/ip_local_port_range default 32768-60999
  • Explained conflicts with SRT Caller mode and other outgoing connections
  • Verified build and UI display work correctly
  • Confirmed warning appears prominently in the Listener mode section

Technical details:

  • Framework: Astro with Starlight documentation theme (instead of Nuxt)
  • Alert syntax: :::danger[Port Selection Warning] (instead of ::alert{type="warning"})
  • File location: src/content/docs/en/astra/receiving/srt.md
  • Language support: English only in refactoring branch (as that's what currently exists)

Screenshot:

SRT Refactoring Branch with Warning

The warning now appears correctly in the refactoring branch with the proper danger-level styling and is positioned right after the port parameter descriptions in the Listener mode section.

Addresses @and-cesbo's request to work with the refactoring branch instead of master/main.


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Copy link
Contributor Author

Copilot AI commented Sep 19, 2025

@and-cesbo 👋 This repository doesn't have Copilot instructions. With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs.

I can generate a .github/copilot-instructions.md file for you automatically. Click here to open a pre-filled issue and assign it to me. I'll write the instructions, and then tag you for review.

Copilot AI changed the title [WIP] Написать про динамические порты в Linux Add dynamic ports warning to SRT Listener mode documentation Sep 19, 2025
Copilot AI requested a review from and-cesbo September 19, 2025 09:00
Copilot finished work on behalf of and-cesbo September 19, 2025 09:00
@and-cesbo
Copy link
Member

@copilot ты можешь работать с другими ветками, не с master? изменения нужны в ветке refactoring

Copy link
Contributor Author

Copilot AI commented Sep 19, 2025

@copilot ты можешь работать с другими ветками, не с master? изменения нужны в ветке refactoring

Yes, I can work with the refactoring branch! I've successfully switched to it and applied the dynamic ports warning there. The refactoring branch uses Astro/Starlight instead of Nuxt, so I adapted the warning syntax accordingly (:::danger instead of ::alert). The changes are now in src/content/docs/en/astra/receiving/srt.md and the warning displays correctly. Commit: c43f281

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants