search

olliw42 / mLRS-docu

Documentation for the mLRS project
GNU General Public License v3.0
35 stars 11 forks source link

elrs-add-br3 #152

Closed jlpoltrack closed 6 months ago

jlpoltrack commented 6 months ago

Title, plus cleaned up the table a little.

olliw42 commented 6 months ago

lgtm

but it's confusing. It first is claimed to tell te supported receivers, but the selected cannot be found ...

olliw42 commented 6 months ago

D PA probably also needed

jlpoltrack commented 6 months ago

It first is claimed to tell te supported receivers, but the selected cannot be found ...

Could you clarify a little here? I don't fully follow.

olliw42 commented 6 months ago

well, the first chapter claims "these are supported", makes me look at this chapter and look into the json. even though not needed at all for the selected devices ...

I would think the logic would be rather: These are the selected de ices which are supported, and in addition these are also supported ...

bottom line: we need to get a good plan how to hanlde the various devices/ases, as discussed in another PR ...

so, I'm happy for this to be merged for sure, but the situation is confusing IMHO

jlpoltrack commented 6 months ago

Gotcha - will track here: https://github.com/olliw42/mLRS-docu/issues/153