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

Limit negative balances #57

Open
lysogeny opened this issue Nov 5, 2023 · 3 comments
Open

Limit negative balances #57

lysogeny opened this issue Nov 5, 2023 · 3 comments

Comments

@lysogeny
Copy link
Contributor

lysogeny commented Nov 5, 2023

Allowing arbitrarily large negative balances can be problematic. It may be useful to introduce a limit beyond which the balance cannot go negative, like -5€.

@stapelberg
Copy link
Member

There was discussion to remove negative balances altogether. Maybe just reverting the commit that introduced them is sufficient for that, and to make this issue moot, too.

@Merovius
Copy link
Contributor

Merovius commented Nov 9, 2023

Personally, I would favor not having negative balances. It was a reasonable experiment, but to me, the fact that we ended with negative totals after the event means it failed, pretty much. The relatively small convenience of not having to top up when getting a drink doesn't outweigh the awkwardness of having to hunt down outstanding debt after the event.

@lysogeny
Copy link
Contributor Author

There was also discussion to limit it to something small where losses would not be too severe.

Personally I would not be opposed to going back to strictly positive balances either.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants