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

refactor(db): refactor database schema #217

Merged
merged 6 commits into from
Dec 12, 2024
Merged

refactor(db): refactor database schema #217

merged 6 commits into from
Dec 12, 2024

Conversation

dev-737
Copy link
Member

@dev-737 dev-737 commented Dec 10, 2024

Renames methods in HubSettingsManager and related files to improve
clarity and consistency. Changes getSetting to has, getAllSettings
to getAll, and settingsEmbed to embed. Updates references in
other files accordingly. Adjusts import statements for JSON files to
use the new syntax. This enhances code readability and aligns method
names with their functionality.

Renames methods in HubSettingsManager and related files to improve 
clarity and consistency. Changes `getSetting` to `has`, `getAllSettings` 
to `getAll`, and `settingsEmbed` to `embed`. Updates references in 
other files accordingly. Adjusts import statements for JSON files to 
use the new syntax. This enhances code readability and aligns method 
names with their functionality.
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 13254 lines exceeds the maximum allowed for the inline comments feature.

@dev-737 dev-737 merged commit 96445c0 into main Dec 12, 2024
5 of 7 checks passed
@dev-737 dev-737 deleted the d-branch-2 branch December 12, 2024 07:07
Copy link

sentry-io bot commented Dec 13, 2024

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ HTTPError: Service Unavailable MessageInfo.execute(messageInfo.ts) View Issue
  • ‼️ DiscordAPIError[10062]: Unknown interaction MessageInfo.handleModals(messageInfo.ts) View Issue
  • ‼️ **PrismaClientKnownRequestError: ** default(Profanity.ts) View Issue

Did you find this useful? React with a 👍 or 👎

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