coder / coder/coder

Refactor `templateVersionDynamicParameters` to remove control flag and send websocket errors

Đang mở
#18,478 0 bình luận 0 reaction 1 người được giao Được @Emyrk nhận Xem trên GitHub
Ngôn ngữ chính
Go
Star
14.5k
Fork
1.5k
Merge trung bình
1 ngày 20 giờ
Pull request đã merge (30 ngày)
601

Mô tả

This code currently operates with a control flag `listen`

https://github.com/coder/coder/blob/ae827703d8f75ebc77f07f436ad3200f48fa2ed1/coderd/parameters.go#L71-L112

This requires a `nolint` and should be fixed. Additionally, http errors are not well received by websocket based FE requests. These errors should be translated to the websocket message to be properly rendered on the FE.

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Đánh giá

Issue này chưa được đánh giá.

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.