Skip to content

Weekly update + QTPS value - #34

Merged
dewabisma merged 2 commits into
mainfrom
beast/new-weekly-update
Aug 19, 2026
Merged

Weekly update + QTPS value#34
dewabisma merged 2 commits into
mainfrom
beast/new-weekly-update

Conversation

@dewabisma

Copy link
Copy Markdown
Collaborator

Summary

  • Add new 08-19-2026 weekly update
  • Update QTPS value and refactor to make value as constant so easy to adjust.

@dewabisma
dewabisma requested a review from n13 August 19, 2026 14:59

@n13 n13 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewer model: GPT Sol

APPROVE — no blocking findings at 394ef7d2.

Reviewed the complete 5fb4ce4...394ef7d2 diff. The shared QTPS constant is applied consistently across the home page, wallet page, ticker, and all nine locales; every localized weekly post and both new WebP assets render in the static output.

Validation:

  • Frozen-lock Bun dependency install: passed with browser download skipped locally.
  • Astro production build with a localhost site URL: passed; all nine localized blog routes were generated.
  • SEO metadata check, JSON parsing and translation-structure checks, rendered QTPS/post checks, image validation, and git diff --check: passed.
  • The repository-wide Prettier check remains red on multiple files already unformatted at the base, plus a non-blocking formatting delta in SolutionSection.astro. No GitHub checks are reported for this branch.

No blocking correctness, compatibility, or content findings.

@dewabisma
dewabisma merged commit 704c1ad into main Aug 19, 2026
@dewabisma
dewabisma deleted the beast/new-weekly-update branch August 19, 2026 15:11
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