Improvement: Remove "streamdal" action
plumber contains a streamdal action that allows you to login, logout, search etc. on the old Streamdal (event-driven) SaaS platform.
Since streamdal moved into the open source data quality space, the plumber streamdal ... action no longer makes sense.
To remove this, you will need to update the protobuf definitions available here: batchcorp/schemas
^ This is because the plumber CLI is defined as protobuf (because the CLI options are highly coupled with the actions and params for the actions that plumber takes like plumber read rabbitmq ...).
This task is:
- "Easy" if you have worked with protobuf and Go
- "Medium" if you have worked with
protobufbut not in Go - "Hard" if you have little to no experience with protobuf
Hi @dselans I'm very interested in contributing to Plumber and this issue caught my eye! I have experience with Go and Protobuf and I'd love to give this a shot for Hacktoberfest, which starts tomorrow. Would it be alright if I take a closer look and reach out with any initial questions I have before diving into the implementation?
Yeah absolutely, go for it!
Hi @dselans, I've created a draft PR. Could you please review it to ensure it fully addresses the requirement to remove Streamdal?
I'm also having trouble testing it. I've tried copying the newly generated plumber-schemas to the plumber > vendor > [path to] > plumber-schemas > build > go > protos directory, but I'm encountering numerous undefined Streamdal errors. This is likely due to the ongoing use of Streamdal in several plumber code areas. Is this expected behavior?