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

Support for multiple parameters in the Java method invocation Step #626

Closed
a1shadows opened this issue Feb 15, 2024 · 6 comments
Closed
Assignees

Comments

@a1shadows
Copy link
Collaborator

@authorjapps
It looks like currently we can support only 1 or 0 argument methods for invocation through steps. I can look into this to support a variable number of arguments

@a1shadows a1shadows changed the title Support for multiple parameters in Java method invocation Support for multiple parameters in the Java method invocation Step Feb 15, 2024
@a1shadows
Copy link
Collaborator Author

@authorjapps Can you please assign the issues I raised to me? I'll look into them one by one

@authorjapps
Copy link
Owner

@authorjapps Can you please assign the issues I raised to me? I'll look into them one by one

Happy to assign @a1shadows .

Just checking what would be usecases for more than 1 params ?

Currently, if you pass an object to the function, it can accommodate 0 or 1 or more parameters as input to the function imo, which addresses most of the usecases.

@a1shadows
Copy link
Collaborator Author

@authorjapps I think that would be true for any method written specifically for use in Zerocode. However, for already existing methods, this necessitates creating wrapper methods and DTO classes for methods that could have been directly called without writing extra code

@authorjapps
Copy link
Owner

Got you @a1shadows, lets go for it if it makes testing process and dev/test life further easy.

Please make sure backward compatibility is maintained, as existing projects should not break. Of course you might know this already, just a note due to we all forget during the dev :-)

@authorjapps
Copy link
Owner

Closed by PR #628

@authorjapps
Copy link
Owner

This feature is available in v 1.0.36 and higher when released.

PENDING:

  • Documentation for this feature

@authorjapps authorjapps added this to the BELGIUM-MINOR-RELEASE milestone Mar 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants