Skip to content

Docs updates from workspace "testing" - #98

Open
AndersHoglund wants to merge 16 commits into
rotorflight:mainfrom
AndersHoglund:testing
Open

Docs updates from workspace "testing"#98
AndersHoglund wants to merge 16 commits into
rotorflight:mainfrom
AndersHoglund:testing

Conversation

@AndersHoglund

@AndersHoglund AndersHoglund commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Small rephrase -(docEditor)

@pkaig

pkaig commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

As discussed. Will hold till ready.

@mtrussotto

Copy link
Copy Markdown

Looks right to me.

@pkaig

pkaig commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

It does need to retain the

sidebar_position: 50

This sets the auto position in the sidebar

@AndersHoglund

Copy link
Copy Markdown
Contributor Author

It does need to retain the

sidebar_position: 50

This sets the auto position in the sidebar

I did not (intentionally) change that.? And now when re-opening I get a merge conflict on it, with no way I can see to resolve. What is going on?
/A
Screenshot 2026-08-24 140824

@AndersHoglund

Copy link
Copy Markdown
Contributor Author

Manually, outside doc editor, reverting that commit seems to resolve conflict. Will retry the intended change using doc-editor.
/A

@AndersHoglund

Copy link
Copy Markdown
Contributor Author

Oooops, all my changes in this PR seems to be reverted when viewing with doc-editor. Not so when viewing local git repo branch or viewing diff in github PR. Doc-editor local workspace out of sync it seems.

@AndersHoglund

Copy link
Copy Markdown
Contributor Author

Re-did the intended change in emacs, looks good on github PR and local git repo.
Doc-editor did not synchronize that update. Restarting did not help either.

@mtrussotto

Copy link
Copy Markdown

Needs to mention somewhere that
set srxl2_unit_id = 0
is necessary for full size receivers.

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.

3 participants