search

jovandeginste / workout-tracker

A workout tracking web application for personal use (or family, friends), geared towards running and other GPX-based activities
Other
958 stars 30 forks source link

After upgraded DB workout needs to manually refreshed #98

Closed miszterx closed 7 months ago

miszterx commented 7 months ago

Hi,

I have started my activity with release version 10.5. I have imported 5800 workouts. Worked fine.

The Next release what I used as an upgrade was 12.1. Start automatically did the upgrade. The application loads perfectly and even lists the workouts and charts. However, If I click on one particular workout for details I got:

{"message":"Internal Server Error"}

In the log I can see:

8:21PM ERR template: workouts_show.html:65:45: executing "workouts_show.html" at <.StatisticsPer>: error calling StatisticsPer: runtime error: invalid memory address or nil pointer dereference app=workout-tracker version=v0.12.3 sha=6a69273150cab3138749ee5979d462bf959e58ee module=webserver request.time="2024-04-13 20:21:34.73549021 +0200 CEST" request.method=GET

If I click on the workout list view I can make a "refresh". It fixes it. However, I don't want to do it on more than 5000 entries.

Is there a way to trigger a refresh for all workouts? or is it a bug? Now I tried 12.3 release as well, but it is the same.

jovandeginste commented 7 months ago

This seems to be a bug ...

However, you can refresh all workouts via your profile page. This will take a while, of course, and will run in the back ground. You can tail the log file to follow.

image

miszterx commented 7 months ago

Great! Thanks for the recommendation. So the same is responsible for the bulk refresh as well. Just triggered. Thanks for the solution.

Not sure about the startup process, when it detects the DB needs to be upgraded, but probably a step is missing there.

miszterx commented 7 months ago

fixed with workaround

jovandeginste commented 7 months ago

Database migrations are hard... But the workaround should help!