Open nsarupr opened 22 hours ago
The pull request introduces modifications to the AppsmithAiPlugin
class and related services, primarily updating method signatures to include an additional parameter, ExecuteActionDTO
. This change affects the executeCommon
method of AppsmithAiPluginExecutor
and the createQuery
methods across several service implementations. The updates maintain existing logic while enhancing the method interfaces to support more complex action execution scenarios.
File | Change Summary |
---|---|
app/server/appsmith-plugins/appsmithAiPlugin/src/main/java/com/external/plugins/AppsmithAiPlugin.java |
Updated executeCommon method signature to include ExecuteActionDTO . |
app/server/appsmith-plugins/appsmithAiPlugin/src/main/java/com/external/plugins/services/AiFeatureService.java |
Modified createQuery method signature to accept ExecuteActionDTO . |
app/server/appsmith-plugins/appsmithAiPlugin/src/main/java/com/external/plugins/services/features/ImageCaptioningServiceImpl.java |
Updated createQuery method to include ExecuteActionDTO . |
app/server/appsmith-plugins/appsmithAiPlugin/src/main/java/com/external/plugins/services/features/ImageClassificationServiceImpl.java |
Updated createQuery method to include ExecuteActionDTO . |
app/server/appsmith-plugins/appsmithAiPlugin/src/main/java/com/external/plugins/services/features/ImageEntityExtractionServiceImpl.java |
Updated createQuery method to include ExecuteActionDTO . |
app/server/appsmith-plugins/appsmithAiPlugin/src/main/java/com/external/plugins/services/features/TextClassificationServiceImpl.java |
Updated createQuery method to include ExecuteActionDTO . |
app/server/appsmith-plugins/appsmithAiPlugin/src/main/java/com/external/plugins/services/features/TextEntityExtractionServiceImpl.java |
Updated createQuery method to include ExecuteActionDTO . |
app/server/appsmith-plugins/appsmithAiPlugin/src/main/java/com/external/plugins/services/features/TextGenerationServiceImpl.java |
Updated createQuery method to include ExecuteActionDTO . |
app/server/appsmith-plugins/appsmithAiPlugin/src/main/java/com/external/plugins/services/features/TextSummarizationServiceImpl.java |
Updated createQuery method to include ExecuteActionDTO . |
app/server/appsmith-plugins/appsmithAiPlugin/src/test/java/com/external/plugins/services/features/ImageCaptioningServiceImplTest.java |
Updated test methods to include ExecuteActionDTO in createQuery calls. |
app/server/appsmith-plugins/appsmithAiPlugin/src/test/java/com/external/plugins/services/features/ImageClassificationServiceImplTest.java |
Updated test methods to include ExecuteActionDTO in createQuery calls. |
app/server/appsmith-plugins/appsmithAiPlugin/src/test/java/com/external/plugins/services/features/ImageEntityExtractionServiceImplTest.java |
Updated test methods to include ExecuteActionDTO in createQuery calls. |
app/server/appsmith-plugins/appsmithAiPlugin/src/test/java/com/external/plugins/services/features/TextClassificationServiceImplTest.java |
Updated test methods to include ExecuteActionDTO in createQuery calls. |
app/server/appsmith-plugins/appsmithAiPlugin/src/test/java/com/external/plugins/services/features/TextEntityExtractionServiceImplTest.java |
Updated test methods to include ExecuteActionDTO in createQuery calls. |
app/server/appsmith-plugins/appsmithAiPlugin/src/test/java/com/external/plugins/services/features/TextGenerationServiceImplTest.java |
Updated test methods to include ExecuteActionDTO in createQuery calls. |
app/server/appsmith-plugins/appsmithAiPlugin/src/test/java/com/external/plugins/services/features/TextSummarizationServiceImplTest.java |
Updated test methods to include ExecuteActionDTO in createQuery calls. |
Enhancement
, Bug
, Task
, ok-to-test
, JS Objects
, Javascript Product
, Query & JS Pod
🎉 In the land of code, where changes flow,
New parameters added, to help us grow.
ExecuteActionDTO
, a friend so true,
Enhancing our queries, making them new!
With tests updated, we march ahead,
In the realm of plugins, where progress is spread! 🌟
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Description
Fixes https://github.com/appsmithorg/appsmith/issues/37637
Automation
/ok-to-test tags="@tag.Sanity"
:mag: Cypress test results
Communication
Should the DevRel and Marketing teams inform users about this change?