-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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 endpoint for all connections #8356
Add endpoint for all connections #8356
Conversation
Signed-off-by: abdullah1308 <abdullahrafi.1308@gmail.com>
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #8356 +/- ##
=========================================
- Coverage 5.20% 5.20% -0.01%
=========================================
Files 124 124
Lines 17628 17664 +36
=========================================
+ Hits 918 919 +1
- Misses 16538 16573 +35
Partials 172 172
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
@@ -39,7 +39,7 @@ type Connection struct { | |||
|
|||
// swagger:response ConnectionPage | |||
type ConnectionPage struct { | |||
Connection []Connection `json:"connection"` | |||
Connections []Connection `json:"connections"` |
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.
@abdullah1308 This will also require changes in UI, right?
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.
@MUzairS15 These endpoints aren't being used anywhere.
Merge conflict.. |
Spreadsheet updated? |
@senali-d is this what you need? |
Signed-off-by: abdullah1308 <abdullahrafi.1308@gmail.com>
Fixed merge conflict |
Yes I'm waiting for this. |
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.
LGTM
Notes for Reviewers
This PR fixes #
Added GET
/api/integrations/connections
Signed commits