[LevelUp] Extend external API and add dynamic command cooldown and unlocking based on level #231
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Please contact me in the 3rd party server to discuss this |
Merge from fork
…guild owners to set channels to ignore level up notifications.
There was a problem hiding this comment.
Pull Request Overview
This PR extends the external API for user profiles and introduces dynamic level-based command cooldowns and unlocking.
- Added get_xp and get_level methods to retrieve user XP and level.
- Implemented new level and cooldown checks via a dedicated Checks class and integrated them into the bot's hook system.
- Introduced a set of admin and user commands for managing command cooldowns, level requirements, and notification preferences.
Reviewed Changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| levelup/shared/profile.py | Added get_xp and get_level methods for retrieving user XP and level. |
| levelup/shared/levelups.py | Updated notification logic to respect user notification preferences. |
| levelup/shared/checks.py | Introduced new Checks class for level and cooldown validations. |
| levelup/shared/init.py | Integrated Checks into SharedFunctions. |
| levelup/main.py | Registered and unregistered before_invoke hooks for new checks. |
| levelup/common/models.py | Added new configuration fields for command cooldowns and bypass roles. |
| levelup/commands/user.py | Added a toggle command for level up notifications and revised prestige logic. |
| levelup/commands/admin.py | Added commands for managing bypass members/roles, level cooldowns, and requirements. |
| levelup/README.md | Updated documentation with instructions for new commands and features. |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
Hey, I took a look through the PR and it seems good, could you bump the minor version number since this is a big change? Its under the main cog constructor class. |
|
Done |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I extended the functionality of the shared functions for user profiles, allowing external cogs to get xp and level of users. I also added a new system to LevelUp that allows guild owners to setup dynamic command lock/unlock based on their level, as well as cooldowns for any command based on level. This is done by creating hooks and injecting them into RedBot that will check user's level and send appropriate error messages if they do not meet the level requirement for a command, or are on cooldown for a command based on their level.