Now respecting XDG_DOCUMENTS_DIR variables

[danilown]

Using the correct way to reference the /home/user/Documents folder that was previously hardcoded.

[flathubbot]

Started test build 42916

[flathubbot]

Build 42916 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/41285/us.zoom.Zoom.flatpakref

[motorto]

when will this push be accepted ? Its pretty annoying for me having a Documents folder in my system when I dont have the locale in english ...

[danilown]

You can achieve the same results of this commit using Flatseal. You then go to the "Filesystem" section and toggle "filesystem=home" on.

However, the "Documents" folder will still be created if you try to save something from Zoom into your machine due to a Zoom / Portals conflict (for the lack of a better word). More about that on this discussion.

[motorto]

Thanks, wasnt aware of that ! Really helpfull ! Nevertheless accepting this pull request should be done , because people shouldn' have to play with Flatseal :)

…

On 2021/04/14 12:41, Danilo Weber Nunes wrote: You can achieve the same results of this commit using [Flatseal](https://flathub.org/apps/details/com.github.tchx84.Flatseal). You then go to the "Filesystem" section and toggle "filesystem=home" on. However, the "Documents" folder will still be created if you try to save something from Zoom into your machine due to a Zoom / Portals conflict (for the lack of a better word). More about that on [this discussion](#259 (comment)). -- You are receiving this because you commented. Reply to this email directly or view it on GitHub: #262 (comment)

[alice-mkh]

You sure this is correct? See #208

[ramcq]

Perhaps the launcher script could also check if XDG_DOCUMENTS_DIR is not Documents, and create a symlink for Zoom?

[yajo]

Then how about removing the create part?

[alice-mkh]

Perhaps the launcher script could also check if XDG_DOCUMENTS_DIR is not Documents, and create a symlink for Zoom?

Could work, indeed.

Then how about removing the create part?

Well, it would mean it won't work until you manually create the dir. I mean zoom already sucks, no need to break it more. :p