search

larskarbo / napchart-website

Full stack website utilizing the napchart library to create, save and share napcharts.
https://napchart.com/
30 stars 20 forks source link

Schedules made with API call are not opening and just display 502 Bad Gateway #2

Closed Theleruby closed 6 years ago

Theleruby commented 6 years ago

Schedules which were created by sending an API request to https://napchart.com/api/create do not seem to be loading. When I try to open them, they just display 502 Bad Gateway. Example broken schedules: https://napchart.com/9mnl8 https://napchart.com/f2jpt https://napchart.com/62phx https://napchart.com/ft88y

larskarbo commented 6 years ago

Oh, I forgot to write the new data structure in the docs. You probably have to adjust your app a little bit. The type system used before is no longer used. Instead you create a colorTag array with bindings

I have written a quick explanation of the data now here https://github.com/larskarbo/napchart/blob/master/README.md#data

Theleruby commented 6 years ago

Shouldn't this affect only new creations then? Those schedules were saved prior to the new site launching.

Theleruby commented 6 years ago

I figured it out... 👍

All the schedules submitted by the Nap God +create function since it was implemented on 15 May 2017 have apparently omitted metaInfo. For example if you go to https://napchart.com/api/get?chartid=rd6a7 you will see metaInfo is missing. Apparently this is causing 502 Bad Gateway. I verified this by intentionally omitting metaInfo from an API call; the create succeeded, but the page load then fails.

I've now corrected the error my end, so future schedules created with it will include the metaInfo, but this bug goes back to 15 May 2017 so all of the schedules created with the bot in the last 8 months before today are currently broken and fail to load. (Either you will have to edit them in the database to add the metaInfo, or patch the new website to load the database entry even if metaInfo is missing)

larskarbo commented 6 years ago

Oh thanks for noticing! Metainfo is not mandatory so I will patch the server asap

larskarbo commented 6 years ago

Should work now 👍 could you verify with the bot?

Theleruby commented 6 years ago

Seems fine now thanks!