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

Stuck in loading files #137

Merged
merged 1 commit into from
Nov 5, 2024
Merged

Stuck in loading files #137

merged 1 commit into from
Nov 5, 2024

Conversation

murilopolese
Copy link
Contributor

Problem:

In order to use the editor there must be a folder setup for saving/loading local files.
This is the initial condition at first loading of the app.
If the selected folder gets renamed, deleted or moved, the app gets stuck loading files.

Issue: #135

How to reproduce

  1. Open Arduino Lab for MicroPython
  2. Select local folder if needed
  3. Exit Arduino Lab for MicroPython
  4. Rename selected local folder
  5. Open Arduino Lab for MicroPython again

This should make the app stuck in loading files.

Fast solution:

  1. Open the file explorer and paste %APPDATA% on the address bar
  2. Remove the folder Arduino Lab for MicroPython

VirtualBox_WinDev2311Eval_05_11_2024_10_10_58

VirtualBox_WinDev2311Eval_05_11_2024_10_14_05

Bugfix solution:

If refresh local files fail, make user pick a folder again.

@murilopolese murilopolese requested a review from ubidefeo November 5, 2024 13:14
@murilopolese murilopolese self-assigned this Nov 5, 2024
Copy link
Collaborator

@ubidefeo ubidefeo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tested branch on Win 11.

  • selected local folder
  • delete folder
  • quit editor
  • restarted editor

works as expected: the user is prompted to select a local folder

@murilopolese murilopolese merged commit 169a01a into main Nov 5, 2024
1 check passed
@murilopolese murilopolese deleted the bugfix/stuck-in-loading branch November 5, 2024 17:52
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