AutarkLabs / open-enterprise

A suite of apps that includes allocation, dot voting, issue curation, and other planning tools so organizations can collectively budget and design custom reward & bounty systems.
GNU General Public License v3.0
92 stars 54 forks source link

Overdue bounties correct status info #1829

Open rkzel opened 4 years ago

rkzel commented 4 years ago

Additionally IssueDetails (Bounty widget) and Issue Card needed to notice bounty's expiration and show appropriate information, as per Figma design. Screens of the Issue Card:

Not completed:

Screenshot from 2019-12-21 12-16-57

Bounty in progress, number information added (1 bounty is the default and only option right now):

Screenshot from 2019-12-21 12-18-30

Bounty completed (reduction in number of items):

Screenshot from 2019-12-21 12-24-28

Bounty widget past due date without submissions:

Screenshot from 2019-12-21 19-26-19

Bounty widget past due date with submissions (panels open in RO mode):

Screenshot from 2019-12-21 19-32-14

BountyContextMenu for expired bounties reverts to 'Fund issue'. It opens "bounty update" form (alternative solution: it could perform removing bounty first, one additional transaction). Currently it's returning "not enough permissions" upon submitting - not sure if it is my setup, or... in any case it needs some further testing.

rkzel commented 4 years ago

There is more to this. Namely: what exactly should and can happen after a bounty expires. What should happen: the bounty can be edited and deadline extended, possibly with changed parameters like funding and exp level. I have spent too much time getting old updateBounty function back into life, but that function does not emit appropriate event (so can't update frontend automatically) and it can only change limited set of details of a bounty, so it is of not much use to us as it is right now. That makes quarter of Funding Panel obsolete by the way - I'm happy to clean up.. The other possibility is to remove that bounty and create new funding. Two transactions, but it looks like a more viable way.