-
Notifications
You must be signed in to change notification settings - Fork 91
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
[CP] Add Thumbnail support for the API #1142
Conversation
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## main #1142 +/- ##
==========================================
+ Coverage 33.55% 33.91% +0.36%
==========================================
Files 350 174 -176
Lines 39235 19813 -19422
Branches 12312 6267 -6045
==========================================
- Hits 13164 6720 -6444
+ Misses 25831 12973 -12858
+ Partials 240 120 -120
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
681dcdb
to
3e575e3
Compare
@@ -62,12 +62,17 @@ service /api/am/backoffice on ep0 { | |||
} | |||
return apiDefinition; | |||
} | |||
|
|||
isolated resource function get apis/[string apiId]/thumbnail(@http:Header string? 'if\-none\-match, @http:Header string? accept = "application/json") returns http:Response|commons:APKError { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shall we add them to the same level as previous commented out methods, otherwise with open-api bal gen command again these methods location will change
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
@@ -232,6 +232,9 @@ dependencies = [ | |||
{org = "ballerina", name = "jballerina.java"}, | |||
{org = "ballerina", name = "lang.int"} | |||
] | |||
modules = [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add these same changes to dependencies.template.toml file as well
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added.
@@ -24,6 +24,7 @@ import ballerina/io; | |||
import wso2/apk_common_lib as commons; | |||
import ballerina/log; | |||
|
|||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
extra line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
b660f92
to
6683eae
Compare
6683eae
to
688d63d
Compare
Purpose
This will add the thumbnail support for the API
This will implement the following resources in back office and devportal ds.
Backoffice
Devportal
Fixes