8000 Fixes for D-STAR radios by kk7ds · Pull Request #281 · kk7ds/chirp · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fixes for D-STAR radios #281

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
Dec 19, 2022
Merged

Fixes for D-STAR radios #281

merged 6 commits into from
Dec 19, 2022

Conversation

kk7ds
Copy link
Owner
@kk7ds kk7ds commented Dec 18, 2022
  • Tweak the port finder result dialog
  • Fix settings editor nesting issue
  • Make settings tree control use a min size
  • Make IcomCloneModeRadio expose D-STAR as settings
  • Handle DV props on radios that require call lists

kk7ds added 5 commits December 18, 2022 14:22
@kk7ds
Tweak the port finder result dialog
d7130c3
@kk7ds
Fix settings editor nesting issue
abe78bf 
The first element in the hierarchy would not properly nest children
underneath because the "if parent" test would fail for the first
parent (which is id zero).
@kk7ds
Make settings tree control use a min size
92fd444 
Initial fitment won't include collapsed children, and thus will be
too small.
@kk7ds
Make IcomCloneModeRadio expose D-STAR as settings
971e71c 
This is how we will edit D-STAR lists in the new UI. IIRC, we had
D-STAR support in chirp before generic settings support, and I expect
we would have just done this otherwise.

Any Icom (d-star) radio that exposes settings will need to properly
super() and append.
@kk7ds
Handle DV props on radios that require call lists
bfae3ba
@kk7ds
Implement banks for TH-D74
6a4e48c 
Also add to cpep8.manifest and fix a couple style issues.
@kk7ds kk7ds merged commit 4a4d65b into py3 Dec 19, 2022
@kk7ds kk7ds deleted the dstaredit branch December 19, 2022 00:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@kk7ds
0