Skip to content
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.

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

More intuitive folder structure naming #64

Merged
merged 1 commit into from
Jul 5, 2022

Conversation

rotorman
Copy link
Member

@rotorman rotorman commented Jul 1, 2022

This PR, instead of taranis-x7, taranis-x9, horus and nv14, uses the following subfolder naming:

  • bw128x64 (old taranis-x7)
  • bw212x64 (old taranis-x9)
  • c480x272 (old horus)
  • c320x480 (old NV14, applies also to EL18)

* bw128x64 (old taranis-x7)
* bw212x64 (old taranis-x9)
* c480x272 (old horus)
* c320x480 (old NV14, applies also to EL18)
@pfeerick
Copy link
Member

pfeerick commented Jul 2, 2022

Looks good. Needs co-ordination with both Flasher and Buddy to ensure they don't get broken due to the change in names. Or at least confirmation they are both exclusively using sdcard.json to determine filenames programatically rather than hard-coded references ;)

@rotorman
Copy link
Member Author

rotorman commented Jul 2, 2022

@CoderElectronics and @freshollie both confirmed sdcard.json change should be all it takes for the change to not break the compatibility:
Buddy: https://discord.com/channels/839849772864503828/839856795781431317/992365576590741554
Flasher: https://discord.com/channels/839849772864503828/839856795781431317/992480001364602910

@rotorman rotorman merged commit 5d45140 into EdgeTX:master Jul 5, 2022
@rotorman rotorman deleted the DispTypeResRename branch July 5, 2022 09:48
@pfeerick pfeerick mentioned this pull request Aug 11, 2022
pfeerick added a commit to pfeerick/flasher that referenced this pull request Aug 11, 2022
@pfeerick
Copy link
Member

pfeerick commented Aug 11, 2022

The names are actually hard-coded into Flasher so this actually broke Flasher's ability to SD card prep. 😆

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