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

Some minor cleanups #618

Draft

cmyui wants to merge 1 commit into master from some-minor-cleanups

Member

cmyui commented Feb 17, 2024

Describe your changes

Related Issues / Projects

Checklist

I've manually tested my code


          Some minor cleanups

f61107f

cmyui commented

View reviewed changes

app/commands.py

+                  target.donor_end = new_donor_end
+                  await players_repo.update(
+                      id=target.id,
+                      donor_end=new_donor_end,
                   )

Member Author

cmyui Feb 17, 2024

Needs manual test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

NiceAesth Awaiting requested review from NiceAesth NiceAesth will be requested when the pull request is marked ready for review NiceAesth is a code owner

tsunyoku Awaiting requested review from tsunyoku tsunyoku will be requested when the pull request is marked ready for review tsunyoku is a code owner

7mochi Awaiting requested review from 7mochi 7mochi will be requested when the pull request is marked ready for review 7mochi is a code owner

At least 1 approving review is required to merge this pull request.

Labels

None yet