nextcloud / weather

⛅️ Weather app for Nextcloud
GNU Affero General Public License v3.0
47 stars 27 forks source link

Save button doesn't work in admin settings #35

Closed Stibila closed 7 years ago

Stibila commented 7 years ago

Steps to reproduce

  1. Go to Admin -> Additional settings
  2. Provide OpenWeatherMap API Key
  3. Click on Save button

Expected behaviour

API Key being saved

Actual behaviour

Nothing

Server configuration

Operating system: CentOS 7.3.1611

Web server: Nginx 1.12.0

Database: MariaDB 5.5.52

PHP version: 7.1.4

Nextcloud version: (see Nextcloud admin page) 11.0.3

Logs

All logs are empty.

nerzhul commented 7 years ago

oh sorry it was accidentally released. This feature is not finished yet... you should use personnal setting atm

tavinus commented 7 years ago

Heh!
Took me a few hours for me to get here too...

Does the app store update from master?
It would be nice to have the admin setting working, but maybe disable the option in the master while it is not yet implemented?

nerzhul commented 7 years ago

Yes i just miss some time i'm working on other free projects at the same time, i hope some can help us to contribute or having a second developer on the project can be nice :)

kylerchin commented 7 years ago

Yep.... Me too.... image

xXSTrikeXx commented 7 years ago

Has someone got a solution for saving it globally in admin setts?

MariusBluem commented 7 years ago

Has someone got a solution for saving it globally in admin setts?

This is not implemented yet ;)

nerzhul commented 7 years ago

Yes sorry, i should patch it but i'm very busy , you can also propose a pr to drop Old user setting and create handler for admin setting

nerzhul commented 7 years ago

Working on it

hofmannmartin commented 7 years ago

As this might be related I post it here: There is no field to provide the api key in my personal settings.

Server configuration

Operating system:

Debian 9.0

Web server:

Apache 2.4.25

Database:

MariaDB 10.1

PHP version:

7.0

Nextcloud version: (see Nextcloud admin page)

12.0.0

nerzhul commented 7 years ago

master is not stable, if you are trying to use it.

hofmannmartin commented 7 years ago

I am using the version 1.4.2

nerzhul commented 7 years ago

Solved in 1.5.0

dps27049 commented 6 years ago

I have the same issue. Using 1.5.1. Still says key is invalid.

andriusign commented 5 years ago

When is this gonna be fixed? Still not working. C'mon guys, what's the problem?