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

Add option to sync the root folder from MEGA #271

Open
David7ce opened this issue Nov 7, 2024 · 2 comments
Open

Add option to sync the root folder from MEGA #271

David7ce opened this issue Nov 7, 2024 · 2 comments

Comments

@David7ce
Copy link

David7ce commented Nov 7, 2024

Right now it is only possible to sync a subfolder from MEGA cloud to Android path.

I want to sync the entire MEGA with "/" the root folder, and not creating a new folder to put all content under it.

@konnovdev
Copy link
Contributor

konnovdev commented Nov 8, 2024

It's possible to implement this. In fact, we intentionally disable the selection of MEGA root folder, you can see it in this line of code inside of MegaPickerViewModel.kt:

isSelectEnabled = isRootFolder(currentFolder).not()

This was a business decision. User selecting root MEGA folder would likely happen by mistake. And unlike Desktop, the user of a mobile device has a more constraint local storage and is unlikely to want to clutter their device with the entire MEGA cloud content.

But your comment shows that there is a desire to sync the entire MEGA cloud among some of the users, so perhaps we could reconsider the business decision and enable selecting the root MEGA folder. To avoid the situations of users choosing root folder by mistake, I guess we could simply show user a warning dialog.

I'll take this issue to the business for discussion. Thank you very much for your feedback!

@David7ce
Copy link
Author

David7ce commented Nov 8, 2024

Thank you very much for the clarification.

I wrote as an issue but I know it is not an issue per se, and now I know you manage as a way to reduce possible bad decisions made by users.

However I think most of the users understand what is the root folder, and a warning is something better that limiting the functionality of the app.

Anyways thank you for your work and quick response.

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

2 participants