-
Notifications
You must be signed in to change notification settings - Fork 4
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
feat: implement organisation filter #748
base: master
Are you sure you want to change the base?
Conversation
it was corrupted on merging the PR for some reason
## [2.35.1](v2.35.0...v2.35.1) (2024-12-17) ### Bug Fixes * update yarn.lock ([126d001](126d001))
ps: PR against changelog branch for now |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## feat/changelog-improvement #748 +/- ##
==============================================================
- Coverage 68.12% 67.80% -0.32%
==============================================================
Files 29 29
Lines 571 584 +13
Branches 132 138 +6
==============================================================
+ Hits 389 396 +7
- Misses 171 177 +6
Partials 11 11 ☔ View full report in Codecov by Sentry. |
@@ -69,6 +69,24 @@ module.exports = [ | |||
return orgs | |||
}, | |||
}, | |||
{ | |||
method: 'GET', | |||
path: '/v2/organisations-with-apps', |
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.
not sure about the route name - the API is not RESTful anyhow, so went for simplicity
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.
I would expect this to be under /organisations
. And maybe a resource under that?
0a23e3d
to
8144651
Compare
ed516a0
to
60cdfae
Compare
implements HUB-161
organisations-list-with-show-more.webm