Medication Enhancement - Allow editing / undoing Taken meds

[themanbornwithin]

Please allow editing the time a medication was taken, and unmark a medication as taken.

[ulsklyc]

Confirmed, and of everything currently open this is the one I am least comfortable leaving as it is.

server/routes/health/medications.js has POST /logs/:id/take and POST /logs/:id/skip, and neither a PATCH nor a DELETE on a log entry. So a mistap is permanent. In most modules that would be an annoyance; in a medication log it is worse, because the wrong entry does not stay in the app. It goes into /export/meds-logs, which is the file somebody prints or hands to a doctor.

The good news is that the data model is already right: taken_at is its own column, not derived from the row's creation time, so correcting a time is an update rather than a reconstruction. What is missing is the route and the way to reach it.

Both halves of your request are accepted:

• editing the time of a recorded dose
• unmarking a dose, which has to include the skipped state as well, since skipping by accident is exactly as easy as taking by accident

One thing I want to get right rather than fast: a medication log is a record, and silently rewriting records is its own hazard. My inclination is that a correction stays visible as a correction rather than replacing history invisibly. That is a small decision, not a blocker, and if you have a view on it as someone using the module daily I would rather hear it now than after it ships.

[ulsklyc]

Shipped in v2.7.0. Both halves of your request are in:

• Editing the time of a recorded dose.
• Taking it back - including a skip, since skipping by accident is exactly as easy as taking by accident. The entry goes back to "pending", as if nothing had been ticked.

Open the medication tab, unfold the dose log, and tap the entry you want to correct.

Two decisions in there that go beyond what you asked, and both are worth knowing:

• A dose that came from a schedule cannot be deleted, only taken back. The scheduler would recreate it on its next run, because the dose is still planned for that time - deleting it would look like a success and be a return on the instalment plan. A dose you recorded yourself, without a schedule, is deletable.
• The time travels with the status. Anything that is not "taken" clears the timestamp, because an entry that says not-taken while carrying a time it was taken at contradicts itself - and it would carry that contradiction into the CSV export.

I left you with a question in my first answer, whether a correction should stay visible as a correction rather than replacing history silently, and I want to be straight about how I resolved it: it does not. A corrected entry now simply shows the corrected value. I decided against a visible audit trail for the first version because the log is a personal record, not a clinical document, and a "was 20:18, now 07:05" line in a household app reads as an accusation more than as information. If you use this daily and find that you miss the trail, say so here - I would rather add it because someone needed it than because it sounded rigorous.

While building this, a second defect fell out that you did not report and probably would have run into: the log labelled every entry that was not skipped as "taken", including the ones still waiting for their time. Harmless while there was no way back out of a state, and directly contradictory next to a correction dialog. It shows three states now.

[themanbornwithin]

Thank you for the changes. I agree with your correction method - I was thinking about this on my way in to work, and as this isn't a professional health record, I don't believe modifications need to be audited.