manifoldmarkets / manifold

Manifold Markets: A market for every question
https://manifold.markets
MIT License
413 stars 156 forks source link

Trading ban #2853

Open SirSaltyy opened 3 weeks ago

vercel[bot] commented 3 weeks ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
dev ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 18, 2024 9:54pm
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 18, 2024 9:54pm
prod ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 18, 2024 9:54pm
SirSaltyy commented 3 weeks ago

@mantikoros if u think someone else should review let me know. Also, I might need help resolving the conflict as I think the conflict editor is being dumb (or I am) and I don't want to accidentally undo someone's changes.

SirSaltyy commented 3 weeks ago

Oh, I just did a trading ban (rather than sweepcash and mana-specific), because I couldn't think of a single scenario where we'd want to ban someone from one and not just both.

Admin accounts not being allowed to trade sweepcash is covered by other logic.

mantikoros commented 2 weeks ago

I can think of several cases where we'd only want to ban sweeps trading but not mana trading: Being an admin account is an overly restrictive condition. Future Manifold employees will not be admins. Non-employees who are connected to Manifold may also need to be banned. Users may also voluntarily request to be banned from trading as part of our responsible gaming program.