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

Remove sharingGroupUUID from FileIndexClientUI and add fileGroupUUID #17

Open
crspybits opened this issue Jul 13, 2021 · 0 comments
Open

Comments

@crspybits
Copy link
Member

crspybits commented Jul 13, 2021

With the addition of file group move, the sharingGroupUUID in the FileIndexClientUI can be stale. i.e., the file group containing the file can no longer be in the sharing group.

Instead, I should really have the fileGroupUUID in the FileIndexClientUI-- since that can never change, and it can be joined with the FileGroup table to get the current sharing group if needed. I could too just not have a fileGroupUUID and join first against the FileIndex to get the file group.

This will need a server db migration, but it shouldn't be that big of a deal. The table can simply be flushed. It's a convenience for clients anyways-- to help them more accurately show download indicators.

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

No branches or pull requests

1 participant