coder / coder/coder

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

Aperta
#18,478 0 commenti 0 reazioni 1 assegnatario Rivendicata da @Emyrk Vedi su GitHub
Lingua principale
Go
Stelle
14.5k
Fork
1.5k
Merge medio
1g 20h
PR unite (30g)
601

Descrizione

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.

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.