maps icon indicating copy to clipboard operation
maps copied to clipboard

Ability to change the location of the Maps default folder

Open louim-lbs opened this issue 2 years ago • 2 comments

How to use GitHub

  • Please use the 👍 reaction to show that you are affected by the same issue.
  • Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
  • Subscribe to receive notifications on status change and new comments.

Is your feature request related to a problem? Please describe. I would like to be able to change the location of the Maps default folder, to have a clearer folder list at root.

Describe the solution you'd like Like in Talk, or Calendar apps.

louim-lbs avatar Jun 22 '23 10:06 louim-lbs

I think about taking a shot at this. Could someone please summarize what one roughly has to do?

From my first looks I would need to

  • Find all occurences of the /Maps path and replace it with a variable or function
  • Find all mentions of /Maps in the translations and replace them
  • Implement a configuration or .env variable (This is where I'm least sure what the intended way in Nextcloud would be)

nougatbyte avatar Mar 10 '24 17:03 nougatbyte

First of all thanks for your initiative. If you have any questions or problems feel free to ask for more help. I give my best to give you some hints. But I'm on the phone write now and looking up the relavent links is cumbersome, so excuse me if some don't work.

First of all you need to decide if the variable should be user depended or the same for all users on the server.

Then You can use an app config variable (for all users) or userValue for a user depended value with

https://stable27--nextcloud-server.netlify.app/classes/ocp-iconfig#method_setAppValue

https://stable27--nextcloud-server.netlify.app/classes/ocp-iconfig#method_getAppValue

IConfig additionally has the getAppValue method. Probably you should provide a default value in the getAppValue to prevent issues when the variable is not set, alternatively you must write a transaction setting the value when installing the update.

Then you can use the occ command https://docs.nextcloud.com/server/latest/admin_manual/configuration_server/occ_command.html#config-commands-label To set the values. Probably the user:setting can edit the user values.

Last you could extend the user and/or admin setting GUI to have a GUI for configuration.

Manually updating the translation files is not needed, they are generated by transifex. Updating the translation has to happen there.

tacruc avatar Mar 11 '24 18:03 tacruc