-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Extend the shared library to provide more data for bitbucket prs (and use it here) #4141
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
Extend the shared library to provide more data for bitbucket prs (and use it here) #4141
Conversation
|
While we are waiting for the shared library to apply and publish the changes, I'm converting this PR to a draft. |
Integrates changes from gitkraken/provider-apis-package-js#223 Solves #4128 (#4141)
1845881 to
edfc7b5
Compare
Integrates changes from gitkraken/provider-apis-package-js#223 Solves #4128 (#4141)
Integrates changes from gitkraken/provider-apis-package-js#223 Solves #4128 (#4141)
edfc7b5 to
62d7e9d
Compare
Integrates changes from gitkraken/provider-apis-package-js#223 Solves #4128 (#4141)
Integrates changes from gitkraken/provider-apis-package-js#223 Solves #4128 (#4141)
62d7e9d to
4b6f9d6
Compare
Integrates changes from gitkraken/provider-apis-package-js#223 Solves #4128 (#4141)
Integrates changes from gitkraken/provider-apis-package-js#223 Solves #4128 (#4141)
Integrates changes from gitkraken/provider-apis-package-js#223 Solves #4128 (#4141)
4b6f9d6 to
ed49037
Compare
Integrates changes from gitkraken/provider-apis-package-js#223 Solves #4128 (#4141)
Integrates changes from gitkraken/provider-apis-package-js#223 Solves #4128 (#4141)
ed49037 to
fee015e
Compare
Integrates changes from gitkraken/provider-apis-package-js#223 Solves #4128 (#4141)
fee015e to
14a8312
Compare
Integrates changes from gitkraken/provider-apis-package-js#223 Solves #4128 (#4141)
Integrates changes from gitkraken/provider-apis-package-js#223 Solves #4128 (#4141)
Integrates changes from gitkraken/provider-apis-package-js#223 Solves #4128 (#4141)
14a8312 to
c4e8ca2
Compare
|
Hi @axosoft-ramint |
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.
Looks good, though we will want to talk through paging soon since for some providers we return everything, and for some we only return one "page" of results, however many that may be.
Integrates changes from gitkraken/provider-apis-package-js#223 Solves #4128 (#4141)
Description
Checklist
Fixes $XXX -orCloses #XXX -prefix to auto-close the issue that your PR addresses