-
Notifications
You must be signed in to change notification settings - Fork 56
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
[CMS] feat: add proposalsowned and proposalbilling #2046
Conversation
Needs a rebase, then should be good to go |
This diff now allows users to select proposals by its name, instead of its token. Since getting proposals names require a batching request, and those requests are bounded by 20 proposals for each request, those requests have to be done more than once in case the token from the invoice (details and diff), or the proposal (new and edit actions) are not on the dropdown list.
0e1b329
to
bcd2286
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.
Hey mate, thanks for this PR, looking really nice. Left some minor comments.
Also, do you think we can add a <- go back
link on the proposal billing details page? Like we have on other details page. See:
Another thing I noticed is that the line items listed in this page are not getting cached, so it reloads when we leave then go back to it. Pretty minor though, so might be out of scope for this PR.
edit: travis seems to be bitching about some unused imported packages :b
Thanks for the review! Both your suggestions should be done in separate PRs. This one is for the Proposals Owned tab. I'll create issues for both. |
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.
ACK
invoice && | ||
invoice.input && | ||
invoice.input.lineitems && | ||
flow( |
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 PR adds the
Proposals Owned
tab if the user owns proposals and proposal billing details.Solution description
Added a new tab and
proposals/billing
endpoint.UI Changes Screenshot
Dependencies
Depends on decred/politeia#1242 and #2041
Question
What's the best option to get the proposal title? I'm currently showing the proposal token returned byproposalsowned
. Should we hitbatchProposals
on the FE or should the BE return an object of the format:{title, token}
forproposalsowned
instead of just thetoken
string?