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

Installed but not showing info #25

Closed
KevoM opened this issue Apr 6, 2022 · 5 comments
Closed

Installed but not showing info #25

KevoM opened this issue Apr 6, 2022 · 5 comments
Labels
documentation Improvements or additions to documentation enhancement New feature or request

Comments

@KevoM
Copy link

KevoM commented Apr 6, 2022

Hello, thanks for this new project
Install is fine through Docker, seems to work, but nothing except the background appears

 -------------------------------------------------------------
- Repopulating web root with application files.
- Check if backgrounds or sites volumes have been mounted.
   - Backgrounds directory is mapped... symlinking.
   - Sites directory is mapped... symlinking.
- All done! Starting nginx/php services now.
-------------------------------------------------------------
[06-Apr-2022 17:09:47] NOTICE: fpm is running, pid 17
[06-Apr-2022 17:09:47] NOTICE: ready to handle connections
192.168.1.82 - - [06/Apr/2022:17:10:03 +0000] "GET / HTTP/1.1" 200 1131 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/100.0.4896.60 Safari/537.36" "-" 0.091 0.090 . 

Capture d’écran 2022-04-06 à 19 22 49

@daledavies
Copy link
Owner

Howdy, could you post a screenshot of the console tab in your browser's Dev tools?

@KevoM
Copy link
Author

KevoM commented Apr 6, 2022

Yes, here it is.

and the json is as recommended in the configuration page.

Capture d’écran 2022-04-06 à 20 33 35

@daledavies
Copy link
Owner

Sorry I should have asked this in my first comment, if you look in the network tab for /api/weatherdata.php, what's the response back from that?

I think it is not liking the json returned from weatherdata.php, which could mean that it is producing an error rather than actual well formed json.

@KevoM
Copy link
Author

KevoM commented Apr 6, 2022

It was indeed because I didn't edit the API key...
Removing the OWMAPIKEY's line that make it work ;)

Sorry for pointing out something obvious as an issue. Maybe you can comment the docker compose file ('# change this Key or remove it if not necessary') cause the error didn't clearly say why it happened.

Status 200 in the log and the console

Thanks for you help !

@daledavies
Copy link
Owner

Excellent I'm glad you got it working! I think I should update the readme to be clear like you said, also weatherdata PHP should probably return something other than 200. I'll add it to my to-do list 😄

@daledavies daledavies added documentation Improvements or additions to documentation enhancement New feature or request labels Apr 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants