-
Notifications
You must be signed in to change notification settings - Fork 11
feat: 🎸 CorporateBallots #1459
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
base: develop
Are you sure you want to change the base?
feat: 🎸 CorporateBallots #1459
Conversation
Also, you need to rebase the entire PR once on top of develop branch |
944a762
to
d770a63
Compare
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.
It seems you forgot to address couple of comments, but other than that LGTM
f5aca3d
to
29720e5
Compare
69579a4
to
7e2ca53
Compare
/** | ||
* Retrieve all Ballots associated to this Asset | ||
*/ | ||
public async get(): Promise<CorporateBallotWithDetails[]> { |
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.
This should accept optional pagination params
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.
don't think we can use it here - we need to get all of the corporate actions to be able to filter out the ballot type and then request details - so can't use pagination on that one
so if we'd add pagination we could then request details only for the paged entries or we'd ned to use sq to replace the first one..
|
Description
Breaking Changes
JIRA Link
Checklist