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

Remove chrome references #2451

Merged
merged 1 commit into from Mar 28, 2021
Merged

Remove chrome references #2451

merged 1 commit into from Mar 28, 2021

Conversation

haslinghuis
Copy link
Member

Fixes: #818 by removing references to chrome and checks for ChromeOS.

@haslinghuis
Copy link
Member Author

Should we remove mentioning the version here too?

Screenshot from 2021-03-21 20-47-18

@mikeller
Copy link
Member

@haslinghuis: Yes, I think so. This was displayed in the Chrome Web App for debug purposes, as the Chrome version was not dictated by the app. For the standalone app the NW.js version is dictated by the app.

@mikeller mikeller added this to the 10.8.0 milestone Mar 22, 2021
@sonarcloud
Copy link

sonarcloud bot commented Mar 22, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@mikeller mikeller merged commit 06c8e50 into betaflight:master Mar 28, 2021
@haslinghuis haslinghuis deleted the fix_818 branch March 28, 2021 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update warnings on firmware flasher page
2 participants