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

Add API Chat and Marketplace Assistant feature implementations #12357

Merged
merged 15 commits into from
Mar 23, 2024

Conversation

ashera96
Copy link
Contributor

@ashera96 ashera96 commented Mar 23, 2024

@ashera96 ashera96 changed the title Add AI features Add API Chat and Marketplace Assistant feature implementations Mar 23, 2024
@Arshardh Arshardh mentioned this pull request Mar 23, 2024
Copy link

codecov bot commented Mar 23, 2024

Codecov Report

Attention: Patch coverage is 6.83572% with 845 lines in your changes are missing coverage. Please review.

Project coverage is 41.93%. Comparing base (e94b4fd) to head (6984829).
Report is 3 commits behind head on master.

❗ Current head 6984829 differs from pull request most recent head 9bc1f11. Consider uploading reports for the commit 9bc1f11 to get more accurate results

Files Patch % Lines
...ava/org/wso2/carbon/apimgt/impl/utils/APIUtil.java 0.00% 88 Missing ⚠️
...fier/MarketplaceAssistantApiPublisherNotifier.java 0.00% 69 Missing ⚠️
...imgt/rest/api/store/v1/dto/ApiChatResponseDTO.java 0.00% 67 Missing ⚠️
...mgt/rest/api/store/v1/impl/ApisApiServiceImpl.java 0.00% 66 Missing ⚠️
...re/v1/impl/MarketplaceAssistantApiServiceImpl.java 3.70% 52 Missing ⚠️
...pimgt/rest/api/store/v1/dto/ApiChatRequestDTO.java 0.00% 46 Missing ⚠️
...st/api/store/v1/dto/ApiChatRequestResponseDTO.java 0.00% 46 Missing ⚠️
...t/api/store/v1/dto/ApiChatResponseResourceDTO.java 0.00% 38 Missing ⚠️
...t/api/store/v1/dto/MarketplaceAssistantApiDTO.java 0.00% 38 Missing ⚠️
...est/api/store/v1/dto/ApiChatRequestApiSpecDTO.java 0.00% 30 Missing ⚠️
... and 20 more
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #12357      +/-   ##
============================================
- Coverage     47.62%   41.93%   -5.69%     
- Complexity     4192    15691   +11499     
============================================
  Files          1840     1849       +9     
  Lines        136532   136841     +309     
  Branches      19767    19823      +56     
============================================
- Hits          65019    57386    -7633     
- Misses        63857    72232    +8375     
+ Partials       7656     7223     -433     
Flag Coverage Δ
integration_tests 41.93% <6.83%> (-0.49%) ⬇️
unit_tests ?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

ashera96 and others added 2 commits March 23, 2024 23:32
Fix error flow error message propagate issue
Fix test failures and address review comments
@Arshardh Arshardh merged commit 1d952de into master Mar 23, 2024
5 of 9 checks passed
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

Successfully merging this pull request may close these issues.

4 participants