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.

Sign up for GitHub

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

Jump to bottom

feat!(db): database configuration for per-guild settings #33

Merged
merged 15 commits into from
Nov 27, 2023
Merged

feat!(db): database configuration for per-guild settings #33

merged 15 commits into from
Nov 27, 2023

Conversation

JacobCoffee
Copy link
Owner

Pull Request Checklist

  • New code has 100% test coverage
  • (If applicable) The prose documentation has been updated to reflect the changes introduced by this PR
  • (If applicable) The reference documentation has been updated to reflect the changes introduced by this PR
  • Pre-Commit Checks were ran and passed
  • Tests were ran and passed

Description

Close Issue(s)

@railway-app Railway App
Copy link

railway-app bot commented Nov 26, 2023 •
edited
Loading

This PR is being deployed to Railway 🚅

byte: ◻️ REMOVED

@JacobCoffee
Copy link
Owner Author

JacobCoffee commented Nov 26, 2023 •
edited
Loading

Migrations do not autogenerate currently for some reason

cofin
cofin reviewed Nov 26, 2023
View reviewed changes
src/server/lib/db/utils.py Outdated Show resolved Hide resolved
@JacobCoffee JacobCoffee changed the title feat!(db): initial work towards #9 feat!(db): initial work towards guild settings (#9) Nov 27, 2023
@JacobCoffee JacobCoffee changed the title feat!(db): initial work towards guild settings (#9) feat!(db): database configuration for per-guild settings Nov 27, 2023
@JacobCoffee JacobCoffee requested a review from Alc-Alc November 27, 2023 05:21
@JacobCoffee JacobCoffee merged commit 326dda2 into main Nov 27, 2023
2 of 3 checks passed
@JacobCoffee JacobCoffee deleted the daterbase branch November 27, 2023 05:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cofin cofin cofin left review comments

@Alc-Alc Alc-Alc Awaiting requested review from Alc-Alc

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Enhancement: Add guild-specific settings
3 participants
@JacobCoffee @cofin @Alc-Alc