Skip to content

release: v1.43.4 - #1238

Merged
MBombeck merged 3 commits into
mainfrom
fix/v1434-illness-symptoms
Oct 17, 2026
Merged

MBombeck merged 3 commits into
mainfrom
fix/v1434-illness-symptoms

Conversation

@MBombeck

Copy link
Copy Markdown
Owner

Patch release: an illness shows only the symptoms filed under it; the illness link is editable on a symptom (Refs #1237).

…s time

The episode detail listed every symptom occurrence that fell inside the
episode's onset-to-recovery span as well as the ones filed against it, so a
long-running condition showed the symptoms of every illness overlapping it,
and picking an illness in the entry sheet changed nothing on the page. The
occurrence edit sheet carried no illness link at all, so once logged the
assignment could be neither seen nor changed.

- The episode card reads only the occurrences filed against the episode
  (`SymptomEvent.episodeId`), with a guard against any other.
- The edit sheet loads the link and edits it like the time and intensity;
  an unchanged link is not re-sent, so a link the list cannot show stays.
- One rule decides which episodes may take an occurrence
  (`isEpisodeLinkableAt`): an open one, or a resolved one whose span holds
  the occurrence's time. The server checks it against the time after the
  edit, and the entry and edit sheets offer exactly what it admits, so a
  backdated occurrence can be filed with an illness that has since ended.
- The illness page names the illness on each occurrence row.
- The day view opens a symptom's illness only while that episode exists.

The stored links were always written correctly, so no data repair is needed.

Refs #1237
@MBombeck
MBombeck merged commit 9db0f09 into main Oct 17, 2026
25 checks passed
@MBombeck
MBombeck deleted the fix/v1434-illness-symptoms branch October 17, 2026 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant