Skip to content
This repository was archived by the owner on Jun 22, 2026. It is now read-only.

feat: refresh feeds hourly and repurpose header refresh#3

Merged
phatpham9 merged 2 commits into
mainfrom
feat/hourly-refresh-no-manual-refresh
Jun 21, 2026
Merged

feat: refresh feeds hourly and repurpose header refresh#3
phatpham9 merged 2 commits into
mainfrom
feat/hourly-refresh-no-manual-refresh

Conversation

@phatpham9

@phatpham9 phatpham9 commented Jun 21, 2026

Copy link
Copy Markdown
Member

Summary

Update feedreader to refresh upstream sources hourly by default and keep the header refresh button as a lightweight backend re-fetch control.

What changed

  • change the default scheduler cadence from every 3 hours to every 1 hour
  • make the scheduler honor FEEDREADER_REFRESH_INTERVAL_HOURS when calculating the next wall-clock refresh boundary
  • keep the header refresh button, but repurpose it to re-fetch the current feed view from /api/items
  • do not use the header button to trigger upstream source refreshes
  • keep on-demand upstream source refresh available through the CLI (feedreader fetch) and POST /api/refresh
  • update the README to distinguish backend view refresh from upstream source refresh
  • refresh the checked-in README screenshot asset to show the restored header button
  • bump the app JS query-string version and service-worker shell cache key so deployed clients do not keep stale cached refresh behavior

Verification

  • docker run --rm -v "$PWD":/src -w /src golang:1.24-bookworm go test ./...
  • verified in a live local preview on http://127.0.0.1:18082
  • confirmed in browser automation that clicking the header refresh button issued:
    • GET /api/items?...
    • 0 requests to /api/refresh
  • confirmed the README screenshot asset was regenerated from the updated working-tree preview

Notes

  • the scheduler remains aligned to Asia/Ho_Chi_Minh / UTC+7 wall-clock boundaries
  • the default interval is now 1 hour, but FEEDREADER_REFRESH_INTERVAL_HOURS still overrides it

@phatpham9 phatpham9 changed the title feat: refresh feeds hourly and remove header refresh feat: refresh feeds hourly and repurpose header refresh Jun 21, 2026
@phatpham9 phatpham9 merged commit 292005d into main Jun 21, 2026
1 check passed
@phatpham9 phatpham9 deleted the feat/hourly-refresh-no-manual-refresh branch June 21, 2026 18:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant