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

[InviteTracker] Fix. #31

Closed
wants to merge 7 commits into from
Closed

Conversation

Honkertonken
Copy link
Contributor

Hey there, made quite a lot of changes in invitetracker the cog should be working as expected right now.
Changes include -

  1. Dropped the on_member_remove listener as it was not accurate and getting the invite on leave was going to be messy/ not accurate you can read more abut it here.
  2. As leave tracking was not a thing anymore removed invset leaveenable.
  3. Merged invset joinenable and invset enable into 1 command.
  4. Fixed the userinvites command.
  5. Some minor changes which includes using f strings, removed unused stuff in the config, grammar fixes etc (not very prominent).

Ps: Haven't updated the cog readme / version.

Do let me know if you have any suggestions.
Thanks in advance.

@Honkertonken Honkertonken closed this by deleting the head repository Oct 17, 2024
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.

1 participant