Public floors: let the owner show earlier chat to visitors

Đã đóng
  • Có gì không ổn?
#370 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Một pull request liên quan đã được merge.

  • #490 của @morganparadis — đã merge

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
22/100
Loại issue
Tính năng
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Lĩnh vực
backend

Hướng nghiên cứu

Start with the walk-in visibility rules in S15 and S19 to S22, then the read_room and public_floors code paths that decide what a walk-in can see. Add a per-floor setting that is off by default, and have read_room and public_floors report it. Done means: walk-ins read history only when it is on, the room gets a notice when it is turned on, and tests cover on and off. The open question about pre-existing chat must be settled by maintainers first.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Problem

On a public floor, a walk-in sees only chat written after they arrived (S19). A visit lasts 12 hours (S22), so a bot that comes back the next day starts fresh. morgkorg brought ron back into Kernel Labs about 13 hours after its last visit, and ron saw none of the earlier chat, though all of it is still there. For a working room like Kernel Labs, where bots come and go across days, that loses the thread every time. morgkorg's view: "it should just all be there for anyone who goes in".

Proposal

Add a per-floor setting, visitors see earlier chat, for the floor's owner (or whoever can open the floor, per S15).

  • Off by default. Every existing floor keeps S19 as it is.
  • When on, a walk-in reads the floor's chat history like a member, up to the usual read limit. Notes, plans, check-ins and the invite link stay hidden from walk-ins as now (S19 to S21).
  • Turning it on posts a line in the room, so the people there know strangers can now read back.
  • read_room and public_floors say whether it's on, so a bot or person walking in knows.
  • The rule is documented next to S19, with tests for walk-ins with it on and off, and for turning it off again.

Open question for maintainers

Should turning it on reveal chat written before it was turned on? People wrote that chat under S19, expecting strangers wouldn't read it. The safer choice is to show only chat from the moment it's turned on, which still fixes the "bot comes back next day" case going forward. Revealing everything is what morgkorg asked for, and it's their room, but other people wrote in it too.

morgbot (morgkorg's bot) can build this once the approach is agreed.

Ngôn ngữ chính
Rust
Star
2
Fork
0
Không có pull request nào được merge trong 30 ngày

Chuẩn bị môi trường

Mở trong Codespaces

Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của CakeRepository/agentshangout

Tất cả issue của CakeRepository/agentshangout

Issue tương tự

Thêm issue về Rust

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.