Skip to content
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

Add Contested Proposals filtering #84

Merged
merged 5 commits into from Feb 22, 2023
Merged

Add Contested Proposals filtering #84

merged 5 commits into from Feb 22, 2023

Conversation

JSKitty
Copy link
Member

@JSKitty JSKitty commented Feb 21, 2023

Abstract

To help "dust away" and disincentivise spam or malicious proposals, this PR ensures proposals with less than 25% net consensus (i.e; 75% downvotes) are considered Contested Proposals.

The only exception is proposals with less than 100 votes, to prevent "downvote bombing" on young proposals.

Contested Proposals are effectively "punished" by:

  • Being dropped to the lowest part of the screen in a "Contested Proposals" section.
  • Being given a slightly lower opacity, to move the users' eyes to higher proposals.

This is purely a display change, there's no change in voting functionality, the user still has full and total freedom to vote on proposals they wish; the PR does not intend to harm the average proposal, but act as a 'net' for proposals that are most likely to be spam or otherwise malicious.

image

As usual, related code was also cleaned up and simplified slightly, as well as given JSDoc coverage and translation templates.

What does this PR address?

It addresses possible spam or ill-intent proposals by lowering their presence in the Governance menu - the PR intends to balance "Freedom" and "Censoring" in a way that clearly discourages spam proposals, without impairing the freedom of users.

What features or improvements were added?

A "Contested Proposals" table to the Governance menu, which displays exclusively proposals with a net of 25% or less Net Upvotes or equally 75% Downvotes.

How does this benefit users?

This should help guide users' attention away from clearly-malicious proposals, such as spam, which clears up the primary Governance Proposals section for well-vetted proposals by the DAO.

To help "dust away" and disincentivise spam or malicious proposals, this commit ensures proposals with less than 25% net consensus (i.e; 75% downvotes) are considered Contested Proposals, and put in to a lower display state on MPW, with slightly lowered opacity and stuffed at the bottom of the Governance menu, along with an explainer sub-text to educate the user on these proposals.
@JSKitty JSKitty added the Enhancement New feature or request label Feb 21, 2023
@JSKitty JSKitty self-assigned this Feb 21, 2023
Copy link
Member

@panleone panleone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good idea, only one small comment

scripts/global.js Outdated Show resolved Hide resolved
A minimum requirement for votes has been added before a proposal will be considered for the Contested category (100 votes) to prevent young proposals being abused in the DAO, and the filtering for 'finished' proposals was lightly refactored to include filtering directly, instead of post-call manually.
Copy link
Member

@Duddino Duddino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Last nitpick as far as i'm concerned

scripts/masternode.js Outdated Show resolved Hide resolved
JSKitty and others added 3 commits February 22, 2023 17:54
As suggested by Duddino

Co-authored-by: Duddino <47313600+Duddino@users.noreply.github.com>
@JSKitty JSKitty requested a review from Duddino February 22, 2023 17:59
Copy link
Member

@Duddino Duddino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tACK, code is clean and works

Copy link
Member

@panleone panleone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tACK

@JSKitty JSKitty merged commit 1d2783c into PIVX-Labs:master Feb 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants