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

revert: remove include #57

Merged
merged 2 commits into from
Nov 21, 2023
Merged

revert: remove include #57

merged 2 commits into from
Nov 21, 2023

Conversation

net-tech
Copy link
Owner

No description provided.

Signed-off-by: net-tech <hello@nettech.dev>
@net-tech net-tech added the wtd AI PR description label Nov 21, 2023
@net-tech net-tech self-assigned this Nov 21, 2023
Copy link

height bot commented Nov 21, 2023

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

Copy link

what-the-diff bot commented Nov 21, 2023

PR Summary

  • Refinement of Function Call in settings.ts
    The modifications in src/commands/settings.ts have eliminated an unnecessary piece of information that was previously needed when calling the 'getGuildData' function. This makes the function easier to use in the future.

  • Function Enhancement in database.ts
    The getGuildData function in src/util/database.ts was improved by automatically including the roles attribute in the information it returns. This change removes the need to specify roles each time the function is called. The result is a more streamlined function that provides all necessary role data by default.

Signed-off-by: net-tech <hello@nettech.dev>
@net-tech net-tech merged commit 8ce8225 into main Nov 21, 2023
3 checks passed
@net-tech net-tech deleted the revert/use-include branch November 21, 2023 21:30
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wtd AI PR description
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant