Skip to content

Update FSAF + fix handle local thread json parsing exception in another place#773

Merged
Adamantcheese merged 2 commits into
multi-featurefrom
fsaf-and-local-thread-fixes
Mar 4, 2020
Merged

Update FSAF + fix handle local thread json parsing exception in another place#773
Adamantcheese merged 2 commits into
multi-featurefrom
fsaf-and-local-thread-fixes

Conversation

@K1rakishou

@K1rakishou K1rakishou commented Feb 29, 2020

Copy link
Copy Markdown

Closes #714, #772 and couple of more fixes for #720 and more logs to fix this #638 issue (again) (I kinda have a hunch why this thing happens but need confirmation)

  • Update FSAF to v1.0-alpha39 which has couple of fixes related to files listed in a directory and file containing more than one period at the end of it.
  • Local thread json parsing exception was not handled in all places where it could be called. Now it should be handled in all places and the error should be propagated down to the ThreadPresenter where we will show it instead of infinite spinner.
  • Use default external storage dir path when couldn't figure out where the initial position of SaveLocationController should be.

…er place

- Update FSAF to v1.0-alpha39 which has couple of fixes related to files listed in a directory and file containing more than one period at the end of it.
- Local thread json parsing exception was not handled in all places where it could be called. Now it should be handled in all places and the error should be propagated down to the ThreadPresenter where we will show it instead of infinite spinner.
- Use default external storage dir path when couldn't figure out where the initial position of SaveLocationController should be.
…shes sometimes with "root is already FileRoot" exception
@Adamantcheese Adamantcheese merged commit 56a2917 into multi-feature Mar 4, 2020
@K1rakishou K1rakishou deleted the fsaf-and-local-thread-fixes branch March 4, 2020 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants