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

Ban lengths should be more granular #59

Closed
kaguy4 opened this issue Aug 14, 2023 · 0 comments · Fixed by #60
Closed

Ban lengths should be more granular #59

kaguy4 opened this issue Aug 14, 2023 · 0 comments · Fixed by #60
Labels
idea For your feature ideas.

Comments

@kaguy4
Copy link
Member

kaguy4 commented Aug 14, 2023

At least on Heyuri, sometimes 1 day long ban is too much for minor stuff. The "ideal" would be like on vichan where you can enter combinations like "1w3d12h" (1 week 3 days 12 hours) (it allows putting only "2d3h" or "3d" too). After splitting the string, it can find the integer of how many hours that is in total by multiplying "w" with 168, "d" with 24, and add "h" to that. You can take this a step further and add support for minutes as well actually (and calculate ban's length in minutes at the end instead of hours). Of course, you can just ask ChatGPT-san to generate a PHP function too (the challenge here probably isn't the conversion anyways).

...If that's not easy as it sounds to implement in practice, it can be simply in hours instead of days too.

So on adminban panel, it should either say "Expires (ex: 1w1d1h): (string goes here)", or if that's hard to implement, "Expires (hours): (integer goes here)"

@kaguy4 kaguy4 added the idea For your feature ideas. label Aug 14, 2023
@kaguy4 kaguy4 changed the title Ban lenghts should be better adjustable Ban lenghts should be more granular Aug 14, 2023
@kaguy4 kaguy4 changed the title Ban lenghts should be more granular Ban lengths should be more granular Aug 14, 2023
@kaguy4 kaguy4 closed this as completed in #60 Sep 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
idea For your feature ideas.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant