Skip to content

Comments

Deprecate suffix VFS plugin#11365

Merged
TheOneRing merged 5 commits intomasterfrom
work/suffix
Nov 22, 2023
Merged

Deprecate suffix VFS plugin#11365
TheOneRing merged 5 commits intomasterfrom
work/suffix

Conversation

@TheOneRing
Copy link
Contributor

@TheOneRing TheOneRing commented Nov 14, 2023

No description provided.

@TheOneRing TheOneRing added this to the Desktop 6.0 milestone Nov 14, 2023
@TheOneRing TheOneRing linked an issue Nov 14, 2023 that may be closed by this pull request
Deprecate suffix vfs in 6.0 #11325
Closed
@TheOneRing TheOneRing requested a review from a team November 14, 2023 13:04
TheOneRing
TheOneRing commented Nov 14, 2023
View reviewed changes
@fmoc fmoc changed the title Deprecate suffx vfs Deprecate suffix VFS plugin Nov 14, 2023
fmoc
fmoc suggested changes Nov 14, 2023
View reviewed changes
@TheOneRing TheOneRing force-pushed the work/suffix branch 3 times, most recently from 00cf1ea to c1e1a42 Compare November 15, 2023 12:46
@TheOneRing TheOneRing marked this pull request as draft November 15, 2023 12:50
@TheOneRing TheOneRing requested review from a team and fmoc November 16, 2023 13:47
@TheOneRing TheOneRing marked this pull request as ready for review November 16, 2023 13:47
TheOneRing
TheOneRing commented Nov 16, 2023
View reviewed changes
fmoc
fmoc suggested changes Nov 22, 2023
View reviewed changes
fmoc
fmoc approved these changes Nov 22, 2023
View reviewed changes
Copy link
Contributor

@fmoc fmoc left a comment

Choose a reason for hiding this comment

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

Please see my suggestion. We might want to create an issue for it.

@TheOneRing TheOneRing enabled auto-merge (rebase) November 22, 2023 16:25
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 22, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

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

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@TheOneRing TheOneRing disabled auto-merge November 22, 2023 19:08
@TheOneRing TheOneRing merged commit ae29e2b into master Nov 22, 2023
@delete-merged-branch delete-merged-branch bot deleted the work/suffix branch November 22, 2023 19:24
@saw-jan saw-jan mentioned this pull request Nov 23, 2023
Closed
2 tasks
@Salipa-Gurung Salipa-Gurung mentioned this pull request Nov 23, 2023
Merged
@saw-jan saw-jan mentioned this pull request Apr 5, 2024
Merged
@DeepDiver1975 DeepDiver1975 mentioned this pull request Jun 2, 2025
Merged
DeepDiver1975 added a commit that referenced this pull request Jun 4, 2025
@DeepDiver1975
[DC-53] feat: remove suffix plugin (#12174)
8695b84 
Suffix VFS was deprecated in late 2023 for desktop client 6.0. Now with 7.0 we finally remove it.

#11365
@dsonck92
Copy link

dsonck92 commented Sep 20, 2025

I've just noticed this change with the upgrade to 6.0.0, is there any equivalent planned for the future? I've been using the suffix based VFS driver for a long time since it works more elegantly than the conditional syncing of folders.

I have a lot of files that I only need partly available but aren't put into folders. A similar setup like Windows would be very welcome, but the suffix based variant worked decently enough so it's a bit of a surprise it got removed. I see that it was already done in 2023, but only now in v6 is it prominently mentioned. I think it would have been nice to know where I could have seen this coming in 2023.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@fmoc fmoc fmoc approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

Desktop 6.0

Development

Successfully merging this pull request may close these issues.

Deprecate suffix vfs in 6.0

3 participants

@TheOneRing @dsonck92 @fmoc