feat(reflection): add secular readings with a Settings toggle - #8
Open
kazo0 wants to merge 1 commit into
Open
Conversation
Ship a full year of secular daily readings alongside the A.A. text, scraped from the Beyond Belief EPUB (Joe C., Rebellion Dogs Publishing), and let the reader switch between the two sets from Settings. The readings go into the existing DailyReflections table behind a new IsSecular column, so DateLookup widens to UNIQUE(Month, Day, IsSecular) and both sets can share a date. GetReflection falls back to the A.A. reading when a secular row is missing, and each set now carries its own copyright line — a secular reading must not claim the A.A. attribution. The reflection feed combines the date state with SettingsModel.SecularReadings (feed composition, matching SobrietyTimeModel) so flipping the toggle reloads the current day. Replace the extracted database in LocalApplicationData whenever its size differs from the embedded resource. The version-gated RefreshDatabaseFile only fires for upgrades from below its thresholds, which no longer matches any shipping install, so an existing install would otherwise keep querying a schema that has no IsSecular column and fail every read. Also closes an unterminated YearVisible attribute in DailyReflectionPage.xaml that broke the app head build. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01VBHcfMrn7fcyttfmZkAkJV
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Ship a full year of secular daily readings alongside the A.A. text, scraped from the Beyond Belief EPUB (Joe C., Rebellion Dogs Publishing), and let the reader switch between the two sets from Settings.
The readings go into the existing DailyReflections table behind a new IsSecular column, so DateLookup widens to UNIQUE(Month, Day, IsSecular) and both sets can share a date. GetReflection falls back to the A.A. reading when a secular row is missing, and each set now carries its own copyright line — a secular reading must not claim the A.A. attribution.
The reflection feed combines the date state with SettingsModel.SecularReadings (feed composition, matching SobrietyTimeModel) so flipping the toggle reloads the current day.
Replace the extracted database in LocalApplicationData whenever its size differs from the embedded resource. The version-gated RefreshDatabaseFile only fires for upgrades from below its thresholds, which no longer matches any shipping install, so an existing install would otherwise keep querying a schema that has no IsSecular column and fail every read.
Also closes an unterminated YearVisible attribute in DailyReflectionPage.xaml that broke the app head build.
Claude-Session: https://claude.ai/code/session_01VBHcfMrn7fcyttfmZkAkJV