Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

doc: Group connector and processor parameters #186

Closed
hariso opened this issue Nov 5, 2024 · 4 comments
Closed

doc: Group connector and processor parameters #186

hariso opened this issue Nov 5, 2024 · 4 comments

Comments

@hariso
Copy link
Contributor

hariso commented Nov 5, 2024

Currently, a connector's or processor's own parameters can get mixed with the common SDK parameters, e.g.

Screenshot from 2024-11-05 11-54-41

It would be good to group them so that we first list a connector's/processor's parameters and then the common parameters.

@raulb
Copy link
Member

raulb commented Nov 6, 2024

@hariso you mean, just positioning the ones for the specific connector on top and then the commons below maybe, or something else?

@hariso
Copy link
Contributor Author

hariso commented Nov 6, 2024

@raulb Exactly, it's just about re-ordering.

@lovromazgon
Copy link
Member

The team was talking about sorting the parameters based on whether they are required and then alphabetically.

@hariso hariso moved this from Triage to Todo in Conduit Main Nov 14, 2024
@hariso
Copy link
Contributor Author

hariso commented Jan 6, 2025

Done in ConduitIO/conduit-connector-sdk#209.

@hariso hariso closed this as completed Jan 6, 2025
@github-project-automation github-project-automation bot moved this from Todo to Done in Conduit Main Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

3 participants