Skip to content

Replace all utcnow with just now #1302

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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Replace all utcnow with just now #1302

wants to merge 7 commits into from

Conversation

ajax146
Copy link
Contributor

@ajax146 ajax146 commented May 2, 2025

This aims to resolve the deprecation issue with .utcnow(), and solves the bot showing the wrong times on embeds if its run not in the UTC timezone.
I believe this doesn't break anything but will need testing to be sure.

@ajax146 ajax146 requested review from a team, TheKrol and GalacticalWolf and removed request for a team May 2, 2025 18:46
@ajax146 ajax146 added Waiting for Review PR: Want for next release This PR should be merged prior to the next release labels May 2, 2025
Copy link
Contributor

@dkay0670 dkay0670 left a comment

Choose a reason for hiding this comment

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

modmail thread title and database modified dates should stay utc (make it now(datetime.UTC))

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: Want for next release This PR should be merged prior to the next release Waiting for Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants