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

Remove ratio from entry view #682

Merged
merged 2 commits into from
Apr 7, 2024
Merged

Remove ratio from entry view #682

merged 2 commits into from
Apr 7, 2024

Conversation

nobodyatroot
Copy link
Member

@nobodyatroot nobodyatroot commented Apr 7, 2024

We can close this PR removing ratio from the entry view if there is significant dissent, alternative is to calculate ratio on upvotes vs. downvotes instead of boosts as it is right now... but honestly I've ignored ratio until now when I noticed it always shows 0 without any boosts:

Does nothing on upvotes:
image

Boosts:
image

@nobodyatroot nobodyatroot added the needs feedback Requires a greater consensus to make an informed decision label Apr 7, 2024
@nobodyatroot nobodyatroot changed the title Remove ratio from entry view, open to feedback Remove ratio from entry view Apr 7, 2024
@nobodyatroot nobodyatroot self-assigned this Apr 7, 2024
@nobodyatroot nobodyatroot enabled auto-merge (squash) April 7, 2024 15:35
@nobodyatroot nobodyatroot merged commit f307fa7 into main Apr 7, 2024
7 checks passed
@nobodyatroot nobodyatroot deleted the remove_entry_ratio branch April 7, 2024 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs feedback Requires a greater consensus to make an informed decision
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants