Skip to content

Add Timeline pull-to-sync with shared sync activity - #115

Merged
RCmerci merged 1 commit into
mainfrom
codex/timeline-pull-sync-20261024
Oct 24, 2026
Merged

RCmerci merged 1 commit into
mainfrom
codex/timeline-pull-sync-20261024

Conversation

@RCmerci

@RCmerci RCmerci commented Oct 24, 2026

Copy link
Copy Markdown
Contributor

Dragging down from the top of Timeline shows “Pull to sync” and “Release to sync”; releasing past the threshold enters the existing online-recovery/pull path. The existing loading indicator left of Options covers Connecting, Pulling and Submitting. Repeated requests retain one forced freshness intent behind the shared pull/submission owner, and completion remains tied to the response and required authoritative commit.

The List keeps its native scrolling and row identity, cancels below-threshold gestures, and clears the hint on release or page exit. Default accessibility copy is English (Sync, Files, Photos, Camera), as is the photo-add permission description; existing locale-selected translations are retained. No protected spec or Dune changes.

Validation:

  • dune build @all and full dune runtest --force passed, including the public Core single-flight/commit/admission cases and existing header/callback tests.
  • Existing Apple app-icon packaging test passed; Info.plist validates.
  • A135 iPhone 13/iOS 26.1: actual XCTest touch gestures passed for short cancel, threshold/release, online/offline, repeated busy pulls, page return including the first pull after return, deep-scroll non-trigger, and stable anchor. Actual system Reduce Motion test passed and the original setting was restored. English hints were checked in the recorded frames.
  • The decision document validates. Repository-wide spec-dev-tool checking still reports the five pre-existing main document failures.

Native acceptance uses the production List, Timeline, header and public sync Core in an isolated synthetic-IO fixture with no real graph/account. It does not establish live service/authentication acceptance. The maintained Simulator host-restamp manifest retains cross_abi_validated=false; no release/ABI or performance claim is made.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 24, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-24T10:59:59.376044Z f234dca PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@RCmerci
RCmerci merged commit bfd603e into main Oct 24, 2026
2 checks passed
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.

1 participant