j4ym0 / nextcloud-tv-show-namer

GNU Affero General Public License v3.0
5 stars 1 forks source link

fix issue #12 #13

Closed j4ym0 closed 1 year ago

j4ym0 commented 1 year ago

non admins can not scan or rename

j4ym0 commented 1 year ago

I did toy with the non admin on settings.

But as the settings are for all users. Original it was designed that all users would use the same API key. Maybe there needs to be an option for a user to customize there own settings.

What do you think?

wuppiwuppi commented 1 year ago

An API-Key is always bound to an account.

Now it depends on your view: does the admin of the next-cloud instance offer that to all users or is every user responsible for his account + api key (I would prefer that ...)

j4ym0 commented 1 year ago

After much consideration most of the settings should be per user like the file name structure. This is a users preference although group folders could be an issue.

I still believe that an admin CAN share there api key with the whole Nextcloud instance.

I have decided that all users should have there own settings and to be implemented in the future should be Nextcloud wide default settings for api key.