Skip to content

Websocket params should be a struct, not positional #871

@adamchalmers

Description

@adamchalmers

Currently the modeling API's websocket has a lot of parameters:

Image

It's very annoying to update these, because each time we add an argument, the consumers have to look up the old order and the new order, and make sure all the arguments are correct.

IMO this would be much easier if the websocket endpoint accepted a single parameter of type struct ModelingWsParams with named fields.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions