Skip to content

Conversation

@modSpike
Copy link
Contributor

implemented the menu - the actions need more testing but so far it seems to work fine

also fixed a bug in folderman related to incorrect unsynced count when user adds folder
@modSpike modSpike self-assigned this Dec 19, 2025
@update-docs
Copy link

update-docs bot commented Dec 19, 2025

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@modSpike modSpike marked this pull request as draft December 19, 2025 10:51
a few more tweaks and cleanup
started adding some handling for the enable/disable vfs action. this has led to some extra refactoring so I want to commit this controller related stuff as a separate step - it may not build but better than munging all of this stuff into one commit I think.
killed deadwood
renamed Vfs::checkAvailability and changed return type to simple string
updated callers and added copious todo's related to this topic of having local paths that don't support vfs even when vfs is otherwise available. See DC-219 for that
missed folderman on last push
unused function.
also improved the docs for _vfsIsReady and cleaned up some random stuff
still needs to be tested on windows
also added some todo's related to handling Theme::showVirtualFilesOption which needs work
@modSpike modSpike requested a review from erikjv January 14, 2026 16:09
@modSpike modSpike marked this pull request as ready for review January 14, 2026 16:09
same as the title
@modSpike modSpike requested a review from erikjv January 15, 2026 14:56
@erikjv erikjv merged commit 5151db2 into master Jan 15, 2026
12 of 14 checks passed
@erikjv erikjv deleted the refactor/DC-216 branch January 15, 2026 16:36
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