Repository navigation
Status: pet names and photos reach their copies, and organizers' names #328
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -4,7 +4,26 @@ | |
| 审查目录里的 `evidence/ios-native/acceptance-status.md` 是**第一阶段验收矩阵**(按 | ||
| `SWIFT-CLIENT-ACCEPTANCE.md` 条目编号记证据),不是状态表;功能迁移进度只记在这里。 | ||
|
|
||
| **更新** 2026-10-14 傍晚 · **第三轮多人测试做完**:聚会取消、邀请码作废时别人正在操作都没问题;200 人抢名额时偶尔有人看到「内部错误」,已修好并单独上线 · 报名被拒的提示按手机语言显示(下次装机生效) · 手机上的审计包 `db2d69c` 签名到 10-20 · 没有发布到 App Store | ||
| **更新** 2026-10-14 晚 · **宠物改名同步上线**:宠物改名或换头像后,以前的帖子、关注列表、聚会名单、打卡都跟着换;组织者改名后聚会上的名字也跟着换 · 退出聚会和取消聚会也在测试项目上用 200 人测过 · 手机上的审计包 `db2d69c` 签名到 10-20 · 没有发布到 App Store | ||
|
|
||
| --- | ||
|
|
||
| ## 10-14 晚:宠物改名同步,和组织者的名字 | ||
|
|
||
| 时间都是太平洋时间。 | ||
|
|
||
| - **问题**:宠物改名或换头像以后,它以前的帖子、别人的关注列表、聚会名单和打卡记录还显示旧的名字和头像,网页版也一样。原因是这几处各存了一份,改宠物资料时不跟着改;人改名有专门的同步,宠物没有。你 10-14 晚同意修。 | ||
| - **修法(#325)**:新增 `onPetUpdated`,宠物的名字或头像一变,就把这 4 处换成新的。 | ||
| - 写完再看一眼宠物,期间又改了就再来一轮,所以连续改几次,最后都是最新的。 | ||
| - 本来就是最新的不写。 | ||
| - 写的时候正好被删掉的(比如有人取消了关注)不会被写回来。 | ||
| - **顺带修的(#326)**:一个人改名后,他组织的聚会上「由某某发起」也跟着换。以前要等编辑那个聚会才会更新。 | ||
| - **测试**: | ||
| - 服务器测试新增 6 条,全套 295 条通过。 | ||
| - 测试项目上 200 人:一只有 199 个关注者的宠物被连改两次名字和头像,202 处(2 条帖子、199 份关注列表、1 份聚会名单)全是最后一次的,帖子数不变(#327)。 | ||
| - **旧数据不用补**:只读数过,正式环境里带宠物的 8 条帖子都已经是最新的;关注列表、聚会名单和打卡里还没有带宠物的记录。 | ||
| - **上线(17:39 左右,你在自己的终端里跑的)**:正式环境从 67 个函数变成 68 个。新函数 `onPetUpdated` 状态正常,`onUserUpdated` 版本号加 1、配置没变;线上代码和部署包逐字节相同,之后没有报错。 | ||
| - **退出聚会和取消聚会**:你批准把这两个函数加到测试项目上(#323)后,云端 200 人也测过了:5 人同时退出,人数从 20 变 15;185 人抢空出的 5 个名额,正好进 5 个;名单上有 101 个参加者时取消,101 人都收到通知(#324)。 | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
In the 200-user scenario, the roster count includes the organizer, but Useful? React with 👍 / 👎. |
||
|
|
||
| --- | ||
|
|
||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Check-in documents do not contain a pet-avatar field:
checkInCallablestores onlypetIdandpetName, andsyncPetCopiesupdates check-ins with only{ petName: name }. Therefore an avatar-only change does not modify old check-in records, contrary to the claim here that both old names and avatars in all four locations are replaced. Distinguish the name-only check-in synchronization from the name-and-photo synchronization for posts, follow lists, and meetup rosters.Useful? React with 👍 / 👎.