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

[REFACTOR] Improve WorkflowRequest Class #823

Open
joshpalis opened this issue Aug 5, 2024 · 2 comments
Open

[REFACTOR] Improve WorkflowRequest Class #823

joshpalis opened this issue Aug 5, 2024 · 2 comments
Labels
enhancement New feature or request good first issue Good for newcomers v2.18.0 Issues targeting release v2.18.0

Comments

@joshpalis
Copy link
Member

Is your feature request related to a problem?

Currently the Workflow Request class is used to handle several parameters such as provision, use_case, reprovision, etc, which is becoming unnecessarily complex and can lead to backwards compatibility issues with previous versions when serializing the requests.

What solution would you like?

Create subclasses of the WorkflowRequest class to handle each parameter

@joshpalis joshpalis added enhancement New feature or request untriaged backlog Good to have functionality not critical for next release labels Aug 5, 2024
@dbwiddis dbwiddis added good first issue Good for newcomers and removed untriaged labels Aug 6, 2024
@dbwiddis dbwiddis added v2.18.0 Issues targeting release v2.18.0 and removed backlog Good to have functionality not critical for next release labels Sep 18, 2024
@boooby19
Copy link

boooby19 commented Oct 8, 2024

Hello ! I am new to open-source projects. I am working as junior dev in c#/React, also have 3 year experience with Java. Can I work on this issue? Can you give me some more info? Thanks ! :)

@dbwiddis
Copy link
Member

dbwiddis commented Oct 8, 2024

Hello over here too, @boooby19! I see you posted on #905 so let's get started on that one for your first commit(s) as this one is a bit more involved with the OpenSearch codebase serialization mechanism.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers v2.18.0 Issues targeting release v2.18.0
Projects
None yet
Development

No branches or pull requests

3 participants