-
Notifications
You must be signed in to change notification settings - Fork 78
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
poll variant : Condorcet method support #3897
Comments
Hello, i was looking as well to get a feature like this one to be implement. I will look at it, i've found than php as already an library https://www.condorcet.io who can help. The easiest maybe it's have an option ranking in the text type poll configuration tab who let people insert text instead of checkbox. |
@vinimoz Nice. But please take into account, that the The Would be happy to see your implementation and further discussion in your upcoming PR. If you need help, don't care to contact me. Let's see, how we can adopt it then for the |
@dartcafe it's almost finished. If u want i can post it. like that maybe
u will be happy :) Some improvement maybe will be to adapt correctly box
size for display type of vote like star vote, etc. I can do more testing
later. To avoid branch 8 disturbing.
…On 21/03/2025 13:08, René Gieling wrote:
@vinimoz <https://github.com/vinimoz> Nice. But please take into
account, that the |main| branch is changing a lot.
The |master-7| branch is the current branch for v7.x and will die once
v8 is stable.
Would be happy to see your implementation and further discussion in
your upcoming PR. If you need help, don't care to contact me. Let's
see, how we can adopt it then for the |main| branch.
—
Reply to this email directly, view it on GitHub
<#3897 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AW7C5TTTUWKWNX3FIVQARNT2VSLYLAVCNFSM6AAAAABYQYH542VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDONBUGY2TGOBXGE>.
You are receiving this because you were mentioned.Message ID:
***@***.***>
dartcafe*dartcafe* left a comment (nextcloud/polls#3897)
<#3897 (comment)>
@vinimoz <https://github.com/vinimoz> Nice. But please take into
account, that the |main| branch is changing a lot.
The |master-7| branch is the current branch for v7.x and will die once
v8 is stable.
Would be happy to see your implementation and further discussion in
your upcoming PR. If you need help, don't care to contact me. Let's
see, how we can adopt it then for the |main| branch.
—
Reply to this email directly, view it on GitHub
<#3897 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AW7C5TTTUWKWNX3FIVQARNT2VSLYLAVCNFSM6AAAAABYQYH542VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDONBUGY2TGOBXGE>.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
after branch 8 is released what could be done :
|
Since you are basing on 7.4.1, you can open a PR against the A PR does no harm. |
PR has been proceed, i've never done it hope it work. Let me know if u get it |
Where? Not here... Create a PR from your branch and point it to https://github.com/nextcloud/polls/tree/master-7 |
I think i've tried but i wasn't allowed, its look like i need to be a poll developer. So i do a fork, i put the modification on the fork i've created https://github.com/vinimoz/polls |
I added a PR from your branch #3930 Find comments in the review comments there. |
Strange. Until now there were a lot of PRs from foreign branches. I am confused too. I cannot add myself as a reviewer or assign you to the PR. Maybe you can change it. |
Describe the goal you'd like to achieve
Support condorcet method in poll.
This method will require to set priorities or an ordering between poll entries.
Possibility to prioritize responses could be reused for other voting variant ( by example #3472 ).
This feature is part of voting variant and some requests exists already over this topic.
Describe possible solutions
I will start the study of this feature as mandated by my client.
Additional context
Astrolabe cooperative considers founding this request to get it done.
The text was updated successfully, but these errors were encountered: