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

Update Vue #4258

Merged
merged 3 commits into from
Feb 20, 2025
Merged

Update Vue #4258

merged 3 commits into from
Feb 20, 2025

Conversation

haslinghuis
Copy link
Member

@haslinghuis haslinghuis commented Nov 22, 2024

Attempting to reconstruct and apply basic migration from 4169 to keep it alive

yarn remove @vitejs/plugin-vue2
yarn remove @panter/vue-i18next
yarn remove vue-template-compiler

yarn add @vitejs/plugin-vue
yarn add vue
yarn add tiny-emitter
yarn add i18next-vue

# yarn add vite --dev
yarn add vue-template-compiler --dev
yarn add @vue/compiler-sfc --dev  // no longer needed

Issue is using Vue v3 elements are not updated. Changing options won't reflect for manual, virtual mode. Does not show DFU mode etc. Like the events don't fire or don't reach...

See comment: #4169 (comment)

EDIT: last commit solves the issue with reactive behavior - major thanks to @McGiverGim

@haslinghuis haslinghuis added the dependencies Pull requests that update a dependency file label Nov 22, 2024
@haslinghuis haslinghuis added this to the 11.0 milestone Nov 22, 2024
@haslinghuis haslinghuis self-assigned this Nov 22, 2024
@haslinghuis haslinghuis marked this pull request as draft November 22, 2024 22:16
Copy link

netlify bot commented Nov 22, 2024

Deploy Preview for origin-betaflight-app ready!

Name Link
🔨 Latest commit 1d59a55
🔍 Latest deploy log https://app.netlify.com/sites/origin-betaflight-app/deploys/67b729f9b159df0008725c48
😎 Deploy Preview https://deploy-preview-4258.dev.app.betaflight.com
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@haslinghuis haslinghuis force-pushed the vue3 branch 4 times, most recently from 5856f0d to 8582462 Compare December 7, 2024 00:00
@haslinghuis haslinghuis force-pushed the vue3 branch 2 times, most recently from 4e7d763 to f76d591 Compare December 17, 2024 17:41
@haslinghuis haslinghuis force-pushed the vue3 branch 6 times, most recently from 9cab0fa to d7cdf38 Compare January 4, 2025 23:37
@haslinghuis haslinghuis force-pushed the vue3 branch 2 times, most recently from 69626de to 878857a Compare January 7, 2025 15:51
@haslinghuis haslinghuis force-pushed the vue3 branch 3 times, most recently from 5fa885e to e215b1e Compare February 18, 2025 18:21
@haslinghuis haslinghuis marked this pull request as ready for review February 20, 2025 12:49
haslinghuis and others added 3 commits February 20, 2025 14:10
The biggest problem is with arrays. Vue3 is not detecting well
reactivity in some arrays because we have the view and the logic in
different files, so we can't access the reactivity proxy in a clean way.

I've added reactivity in the definition of the modules affected as a
workaround, but I'm sure there must be a better way.
@nerdCopter
Copy link
Member

  • everything seems to work testing unsoldered SKYSTARSF405AIO -- from reset permissions cleared cache to re-flashed fully configured FC, saved configs, dumps, pasted diffs, presets.
  • nice to see the device selector populated instantly. also shows when USB connector is flaky, because it will appear/disappear.

@haslinghuis haslinghuis merged commit c2b3789 into betaflight:master Feb 20, 2025
9 checks passed
@haslinghuis haslinghuis deleted the vue3 branch February 20, 2025 14:57
@nerdCopter
Copy link
Member

nerdCopter commented Feb 20, 2025

  • post-merge. not a big deal, but i noticed if FC plugged in first, still has legacy behavior of empty device.
    image
  • refreshing URL or plugging in after URL-loaded seems to work well.
  • maybe not worth fixing.

@haslinghuis
Copy link
Member Author

@nerdCopter yes there seems to be a difference between local or deployment preview and online version.

Added a new PR to improve logging.

@McGiverGim
Copy link
Member

I can't reproduce it. If I have the FC connected, and go to https://master.dev.app.betaflight.com/ the FC appears as selected in the combo and if I have autoconnect enabled it connects.

@nerdCopter
Copy link
Member

nerdCopter commented Feb 21, 2025

I can't reproduce it. If I have the FC connected, and go to https://master.dev.app.betaflight.com/ the FC appears as selected in the combo and if I have autoconnect enabled it connects.

reset permissions & clear cache then close Chrome. plugin FC. start Chrome/app. i can't find my device, select. should result in blank list.

@McGiverGim
Copy link
Member

It's difficult to me to reproduce. I've reproduced it only connecting to https://master.dev.app.betaflight.com/ and not always but there I've not too much information. In local I haven't reproduce it.
I've observed that it seems to happen only when in the device appears "vinculado" (linked, connected, associated, etc.)
image

I'm not too sure what this implies.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
Development

Successfully merging this pull request may close these issues.

3 participants