Skip to content

Standardize config parameters and variable names #1

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

Merged
merged 1 commit into from
Mar 14, 2025

Conversation

NightSling
Copy link
Owner

This pull request includes changes to the configuration and command registration in the config-global.js and index.js files. The most important changes involve renaming configuration variables and updating the command registration logic to support multiple guild IDs.

Configuration updates:

  • config-global.js: Renamed PREFIX and GUILD_ID to ADMIN_PREFIX and GUILD_IDS respectively, and corrected the usage of PREFIX to match the configuration file.

Command registration updates:

  • index.js: Updated the import statement to use GUILD_IDS instead of GUILD_ID from config-global.js.
  • index.js: Modified the registerCommands function to iterate over GUILD_IDS for registering slash commands.

Other changes:

  • index.js: Removed an unnecessary newline before the self-invoking function that registers commands.

Breaking Change:
- `GUILD_ID` is no longer a valid configuration attribute, it's been
renamed to `GUILD_IDS`.
- `PREFIX` which was unused previously now the default prefix.
@NightSling NightSling merged commit e611e8b into setup-prettier Mar 14, 2025
@NightSling NightSling deleted the naming-convetions branch April 28, 2025 17:42
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