Skip to content

[Enforce]: Multiple fixes & improvements. - #221

Merged
ltzmax merged 2 commits into
ltzmax:masterfrom
Evanroby:enforce
Mar 17, 2026
Merged

[Enforce]: Multiple fixes & improvements.#221
ltzmax merged 2 commits into
ltzmax:masterfrom
Evanroby:enforce

Conversation

@Evanroby

Copy link
Copy Markdown
Contributor

I like to sneak into code files before adding anything to my bot, found a few things worth fixing such as:

antispam.stamp() was never called so rate limiting on tos prompts did nothing
red_delete_data_for_user silently did nothing for uncached users, get_user() returns None, so I switched to user_from_id()
resetsettings was calling clear_all() instead of clear_all_globals(), accidentally wiping everyone's acceptance
tos_prompt_antispam dict grew forever, so I added a periodic clear + immediate pop on accept

also removed an unused datetime import, redundant @commands.is_owner() on set_link subcommand whose parent already restricts it, wrong docstring on AcceptView (view isn't persistent as it times out) and a type hint on reset_user

@ltzmax
ltzmax merged commit 619fd74 into ltzmax:master Mar 17, 2026
3 checks passed
@ltzmax

ltzmax commented Mar 17, 2026

Copy link
Copy Markdown
Owner

Thank you!

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

Successfully merging this pull request may close these issues.

2 participants