Skip to content

docs: flysky st16 touchscreen modification - #7528

Merged
pfeerick merged 1 commit into
EdgeTX:mainfrom
SakuraHlh:flysky-st16-touch-support
Jul 10, 2026
Merged

docs: flysky st16 touchscreen modification#7528
pfeerick merged 1 commit into
EdgeTX:mainfrom
SakuraHlh:flysky-st16-touch-support

Conversation

@SakuraHlh

@SakuraHlh SakuraHlh commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

flysky st16 touchscreen modification related documentation

Summary by CodeRabbit

  • Documentation
    • Added a new hardware modification guide for enabling touch support on the Flysky ST16.
    • Includes step-by-step instructions, wiring details, part placement, and reassembly notes.
    • Updated the Hardware Mods navigation so the new guide is easier to find.

@coderabbitai

coderabbitai Bot commented Jul 4, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Adds a new documentation page describing a hardware modification to add touch screen support to the Flysky ST16 under edgetx, and registers this new page in the mkdocs navigation under Hardware Mods.

Changes

Documentation Addition

Layer / File(s) Summary
Flysky ST16 touch mod guide and nav entry
docs/mods/flysky-st16-touch-support.md, mkdocs.yml
Adds a new step-by-step guide for installing touch support on the Flysky ST16 (wiring, glass prep, assembly) and links it in the mkdocs navigation under Hardware Mods.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Related PRs: None identified.

Suggested labels: documentation

Suggested reviewers: None identified.

Poem
A rabbit taps a touch-glass screen,
Wires now dance where buttons had been,
A new page joins the nav tree's light,
ST16 fingers, touch just right! 🐰✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is too brief and misses the required template sections, including Fixes # and a Summary of changes block. Add the template sections, especially a Summary of changes and a Fixes # reference if applicable, and describe the documentation change more completely.
✅ Passed checks (4 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly matches the main change: added documentation for the Flysky ST16 touchscreen modification.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@SakuraHlh

Copy link
Copy Markdown
Contributor Author

@pfeerick Could you please check if there are any adjustments needed?

@pfeerick pfeerick changed the title flysky st16 touchscreen modification support documents docs: flysky st16 touchscreen modification Jul 5, 2026
@pfeerick pfeerick added the documentation 📝 Improvements or additions to documentation label Jul 5, 2026
@richardclli

richardclli commented Jul 7, 2026

Copy link
Copy Markdown
Member

@SakuraHlh You should make the update of the docs in this repository instead https://github.com/EdgeTX/edgetx.github.io

Clone the document repository, and organize your docs in "Hardware Mod" section. Then make a new PR in the docs repository and close this PR.

@SakuraHlh

Copy link
Copy Markdown
Contributor Author

@SakuraHlh You should make the update of the docs in this repository instead https://github.com/EdgeTX/edgetx.github.io

Clone the document repository, and organize your docs in "Hardware Mod" section. Then make a new PR in the docs repository and close this PR.

ok

@SakuraHlh

Copy link
Copy Markdown
Contributor Author

@richardclli I couldn't find the relevant location in this link.

@pfeerick

pfeerick commented Jul 10, 2026

Copy link
Copy Markdown
Member

@richardclli Huh? We're moving away from the wiki, to the mkdocs site, i.e. https://edgetx.org/edgetx/latest/, so this is correct. The wiki is completely unusable for anyone who is not a contributor to the project, due to how the permissions work with github.

@pfeerick pfeerick added this to the 3.0 milestone Jul 10, 2026
@pfeerick
pfeerick merged commit fc830bb into EdgeTX:main Jul 10, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation 📝 Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants