betaflight-configurator icon indicating copy to clipboard operation
betaflight-configurator copied to clipboard

Update NWjs to 0.83

Open haslinghuis opened this issue 1 year ago • 12 comments

Check to fix virus issues reported with current build on Windows. Previous we were not able to go up due to icon issue which seems solved now.

haslinghuis avatar Dec 26 '23 01:12 haslinghuis

Do you want to test this code? Here you have an automated build: Betaflight-Configurator-Android Betaflight-Configurator-Linux Betaflight-Configurator-macOS Betaflight-Configurator-Windows WARNING: It may be unstable and result in corrupted configurations or data loss. Use only for testing!

github-actions[bot] avatar Dec 26 '23 01:12 github-actions[bot]

AUTOMERGE: (FAIL)

  • github identifies PR as mergeable -> FAIL
  • assigned to a milestone -> PASS
  • cooling off period lapsed -> PASS
  • commit count less or equal to three -> PASS
  • Don't merge label NOT found -> PASS
  • at least one RN: label found -> PASS
  • Tested label found -> FAIL
  • assigned to an approver -> PASS
  • approver count at least three -> FAIL

blckmn avatar Dec 26 '23 03:12 blckmn

0.83 give same issue with worm in resourcer.exe, Also at my pc found more connections (my headset) AND the installer exe now has the right BF icon. BUT the installed application does not have the Icon (??)

image The worm issue when you build the debug and release application, not when you run the bf installer, nor the bf configurator after installation.

HThuren avatar Dec 26 '23 12:12 HThuren

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

sonarqubecloud[bot] avatar Dec 31 '23 19:12 sonarqubecloud[bot]

Do you want to test this code? Here you have an automated build: Betaflight-Configurator-Android Betaflight-Configurator-Linux Betaflight-Configurator-macOS Betaflight-Configurator-Windows WARNING: It may be unstable and result in corrupted configurations or data loss. Use only for testing!

github-actions[bot] avatar Dec 31 '23 19:12 github-actions[bot]

NWjs developers often acknowledge existence of false-positive virus-checks simply as a by-product of exe generation. :( Unsure about the icon issue. Maybe windows icon cache could be cleared as a troubleshooting step.

nerdCopter avatar Dec 31 '23 21:12 nerdCopter

NWjs developers often acknowledge existence of false-positive virus-checks simply as a by-product of exe generation. :( Unsure about the icon issue. Maybe windows icon cache could be cleared as a troubleshooting step.

Windows defender discover the signature from Win32/AutoRun!pz when resourcer.exe (who do the icon change) start. Defender stop resourcer.exe, report and you have no icon shift. Since resourcer.exe are from 2009, we might can compile a new one with today runtime libs, but then we need the source and the references regarding resourcer.exe are dead. I have wrote to Felicien, if he by luck have the source, but no responce yet. Otherwise we need a new way to change the icon, I suppose deals with windows, mac and linux.

HThuren avatar Jan 01 '24 10:01 HThuren

Did someone try 0.85?

ASDosjani avatar Mar 13 '24 10:03 ASDosjani

Did someone try 0.85?

not yet. a good test might be BBE as well to see if it works in OSX. 0.72.x was the last to work on certain Mac.

pardon the mistaken closure/reopening -- clicked the wrong button.

nerdCopter avatar Mar 13 '24 12:03 nerdCopter

Do you want to test this code? Here you have an automated build: Betaflight-Configurator-Android Betaflight-Configurator-Linux Betaflight-Configurator-macOS Betaflight-Configurator-Windows WARNING: It may be unstable and result in corrupted configurations or data loss. Use only for testing!

github-actions[bot] avatar Mar 13 '24 12:03 github-actions[bot]

Do you want to test this code? Here you have an automated build: Betaflight-Configurator-Android Betaflight-Configurator-Linux Betaflight-Configurator-macOS Betaflight-Configurator-Windows WARNING: It may be unstable and result in corrupted configurations or data loss. Use only for testing!

github-actions[bot] avatar Mar 13 '24 14:03 github-actions[bot]

Moving to PWA NWjs won't be needed anymore.

haslinghuis avatar May 15 '24 17:05 haslinghuis