feat(feedback): capture app version + browser zone; Filed time in three zones #115

Merged

developer merged 2 commits from feature/feedback-version into development 2026-06-22 17:07:21 +00:00

Conversation 2 Commits 2 Files Changed 21

+191 -45

developer commented 2026-06-22 15:02:19 +00:00

Owner

Copy Link

Copy Source

Adds the app version to feedback (the build a report was sent from) and shows it in the operator console, and renders Filed in UTC plus the sender's time zone.

• Version rides the FlatBuffers FeedbackSubmitRequest (new version field; client attaches __APP_VERSION__) → gateway transcode → backend, persisted in feedback_messages.app_version (migration 00003, nullable/additive → image rollback DB-safe). Snapshotted like the interface language.
• Console detail shows the app version and the Filed time as UTC · local (zone) (the sender account's time_zone; the local part is dropped when the zone is UTC).
• Touches fbs + regenerated codegen, codec/transport, transcode/backendclient, store/service, admin view + template, docs (ARCHITECTURE §15, FUNCTIONAL +_ru).

Verified: feedback integration tests (migration + version round-trip), codec round-trip; check 0/0, unit 311, build green. Folds into the held v1.2.1 promotion (PR #114).

Adds the **app version** to feedback (the build a report was sent from) and shows it in the operator console, and renders **Filed** in UTC plus the sender's time zone. - **Version** rides the FlatBuffers `FeedbackSubmitRequest` (new `version` field; client attaches `__APP_VERSION__`) → gateway transcode → backend, persisted in `feedback_messages.app_version` (migration `00003`, nullable/additive → image rollback DB-safe). Snapshotted like the interface language. - **Console detail** shows the app version and the Filed time as `UTC · local (zone)` (the sender account's `time_zone`; the local part is dropped when the zone is UTC). - Touches fbs + regenerated codegen, codec/transport, transcode/backendclient, store/service, admin view + template, docs (ARCHITECTURE §15, FUNCTIONAL +_ru). Verified: feedback integration tests (migration + version round-trip), codec round-trip; check 0/0, unit 311, build green. Folds into the held v1.2.1 promotion (PR #114).

developer added 1 commit 2026-06-22 15:02:20 +00:00

feat(feedback): capture the app version; show version + local Filed time ... b78ce42922

Each feedback submission now carries the client app version (__APP_VERSION__),
snapshotted like the interface language: FlatBuffers FeedbackSubmitRequest gains
a version field → gateway transcode → backend, persisted in a new nullable
feedback_messages.app_version column (migration 00003, additive so an image
rollback stays DB-safe). The operator console detail shows the app version and
renders the Filed time in UTC plus the sender's time zone (fmtTimeIn).

Touches: fbs schema + regenerated Go/TS codegen, codec + transport (the client
attaches its build), gateway transcode + backendclient, feedback store/service,
admin view + template, docs (ARCHITECTURE §15, FUNCTIONAL + _ru). Verified:
feedback integration tests (migration + version round-trip), codec round-trip,
check/unit/build green.

owner approved these changes 2026-06-22 15:20:24 +00:00

Dismissed

developer added 1 commit 2026-06-22 16:06:57 +00:00

feat(feedback): capture the browser UTC offset; Filed time in three zones ... 004aca4e97

The account time zone defaults to UTC until a player saves a profile, so a
report's Filed time could only render in UTC even for a player clearly in
another zone. Capture the client's detected "±HH:MM" offset (browser_tz) with
each submission and show the Filed time in three zones in the operator console
— UTC, the browser offset detected at submit, and the sender's saved profile
zone — each shown "N/A" when not known, so the operator can tell what is
certainly known from what is merely defaulted.

- Thread browser_tz through the fbs envelope (+ Go/TS codegen), the gateway
  transcode + backend client, and the backend feedback service/store; add the
  column via migration 00004 (additive, image-rollback-safe).
- Fix fmtTimeIn to resolve "±HH:MM" offsets via account.ResolveZone;
  time.LoadLocation alone silently fell back to UTC for offset zones, which is
  the second reason a "+02:00" sender showed only UTC.
- Update ARCHITECTURE/FUNCTIONAL(+ru) docs and the feedback integration test.

developer dismissed owner's review 2026-06-22 16:06:57 +00:00

Reason:

New commits pushed, approval review dismissed automatically according to repository settings

developer changed title from feat(feedback): capture app version; Filed time in UTC + user zone to feat(feedback): capture app version + browser zone; Filed time in three zones 2026-06-22 16:07:20 +00:00

developer referenced this pull request 2026-06-22 16:59:10 +00:00

feat(account): seed the time zone from the client's detected offset at creation #116

owner approved these changes 2026-06-22 17:05:24 +00:00

developer merged commit 170a6ae9ef into development 2026-06-22 17:07:21 +00:00

developer deleted branch feature/feedback-version 2026-06-22 17:07:21 +00:00

Sign in to join this conversation.

Reviewers

No Reviewers

owner

Labels

No items

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

owner (Ilia) developer

No Assignees

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: developer/scrabble-game#115