Let the candidate undo Continue without camera in the preflight
Maintainer thường phản hồi trong vòng 1 ngày
Đánh giá
Issue này chưa được đánh giá.
Mô tả
Observed behavior
In the media preflight of an unrecorded interview, clicking "Continue without camera" marks the camera step as done ("Not used (no_device)" or "Not used (denied)") and hides the button. Nothing on the page brings the camera back before the interview starts. A candidate who clicks it by mistake, or who plugs in or frees a camera afterwards, can only reload the page or go back to the lobby and start over.
Expected behavior
Until "Start interview" is pressed, the candidate can switch back to using the camera, and the camera step returns to its normal check.
Reproduction
- Run
codetrial weband start an unrecorded interview. - In the media preflight, click "Continue without camera".
- Look for a way to use the camera again before starting.
Environment
OS: Windows 10
Browser: Google Chrome 152
Initial analysis
The skip handler in web/interview.js sets cameraSkipped, calls pool.disable("video") and writes the Meet presentation choice back to off. refresh() then keeps #camera-skip hidden with recordingEnabled || cameraSkipped, a line tests/browser/dom-contract.test.js pins. createDevicePool in web/devices.js has disable(kind) but no counterpart, so a disabled device is never requested again. The skip is only read when the preflight resolves, so undoing it before Start would not affect any report.
Proposed change
Turn the skip button into a toggle ("Use camera" once skipped), add an enable(kind) to the device pool that clears disabled and requests the device again, and restart the face check when the new track arrives. The Meet presentation box stays unchecked after an undo rather than restoring a choice the candidate never made again. Tests: a device pool test that a re-enabled device is requested again, and a preflight test that skipping and then undoing leaves the camera step pending until a live track arrives. I would like to work on this if the direction looks right.
- Ngôn ngữ chính
- Rust
- Star
- 160
- Fork
- 64
- Merge trung bình
- 1 ngày 10 giờ
- Pull request đã merge (30 ngày)
- 135
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của sysprog21/codetrial
-
Audio output can be confirmed without playing a test soundCó thể đã có người làm @ChenYingHan1121 đã nhận 1 ngày trước. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
sysprog21/codetrial#359 · 3 bình luận · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Add a Copy code button to the final reportCó thể đã có người làm @yansawu đã nhận 1 ngày trước. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 80/100
sysprog21/codetrial#318 · 5 bình luận · 2 reaction · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
C tests accept a wrong next pointer on the last node in Org Chart Row NavigationCó thể đã có người làm @winter77777 đã nhận hôm nay. Đang mở
sysprog21/codetrial#370 · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
"Let me think about it for a moment" does not start a thinking holdCó thể đã có người làm @PeteChiang1211 đã nhận hôm nay. Đang mở
sysprog21/codetrial#369 · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
sysprog21/codetrial#368 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của sysprog21/codetrial
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
swc-project/swc#12576 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
good first issue gpu serve
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
biomejs/biome#12475 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 80/100
Maintainer thường phản hồi trong vòng 1 ngày