Skip to content

Comments

Update app to handle riseset module changes#2

Merged
exploded merged 1 commit intomasterfrom
claude/update-riseset-module-b5NfI
Feb 22, 2026
Merged

Update app to handle riseset module changes#2
exploded merged 1 commit intomasterfrom
claude/update-riseset-module-b5NfI

Conversation

@exploded
Copy link
Owner

The riseset module (v1.0.1-0.20260220080739-24891d86367a) added two new boolean fields to RiseSet: AlwaysAbove and AlwaysBelow, which distinguish "object never sets" (midnight sun) from "object never rises" (polar night). Previously both cases returned "-" for Rise and Set with no way to tell them apart.

Changes:

  • go.mod/go.sum: upgrade to new riseset pseudo-version with the new fields
  • moon.go: use named constants riseset.Moon and riseset.Sun instead of magic numbers 1 and 2; new JSON fields are included automatically
  • calendar.html: render "Always above" / "Always below" in the table instead of showing "-" for polar day/night conditions
  • static/script.js: handle AlwaysAbove and AlwaysBelow in the /gettimes JSON response, displaying a descriptive message in the Rise/Set fields

https://claude.ai/code/session_01T2dm9m2XwVaWyvAx6CDfhM

The riseset module (v1.0.1-0.20260220080739-24891d86367a) added two new
boolean fields to RiseSet: AlwaysAbove and AlwaysBelow, which distinguish
"object never sets" (midnight sun) from "object never rises" (polar night).
Previously both cases returned "-" for Rise and Set with no way to tell them apart.

Changes:
- go.mod/go.sum: upgrade to new riseset pseudo-version with the new fields
- moon.go: use named constants riseset.Moon and riseset.Sun instead of
  magic numbers 1 and 2; new JSON fields are included automatically
- calendar.html: render "Always above" / "Always below" in the table
  instead of showing "-" for polar day/night conditions
- static/script.js: handle AlwaysAbove and AlwaysBelow in the /gettimes
  JSON response, displaying a descriptive message in the Rise/Set fields

https://claude.ai/code/session_01T2dm9m2XwVaWyvAx6CDfhM
@exploded exploded merged commit 80dd3ba into master Feb 22, 2026
1 check passed
@exploded exploded deleted the claude/update-riseset-module-b5NfI branch February 23, 2026 03:09
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.

2 participants