From a756fb69d30a1fb522ff0658f2e2843d1d283490 Mon Sep 17 00:00:00 2001 From: Sebastiaan Zeeff <33516116+SebastiaanZ@users.noreply.github.com> Date: Sat, 24 Aug 2019 12:35:29 +0200 Subject: Apply suggestions from code review Co-Authored-By: Johannes Christ --- pydis_site/apps/api/models/bot/message.py | 3 +-- pydis_site/apps/api/models/bot/tag.py | 2 +- 2 files changed, 2 insertions(+), 3 deletions(-) (limited to 'pydis_site') diff --git a/pydis_site/apps/api/models/bot/message.py b/pydis_site/apps/api/models/bot/message.py index 0713b9d2..ac4b40a3 100644 --- a/pydis_site/apps/api/models/bot/message.py +++ b/pydis_site/apps/api/models/bot/message.py @@ -1,6 +1,5 @@ from datetime import datetime -import pytz from django.contrib.postgres import fields as pgfields from django.core.validators import MinValueValidator from django.db import models @@ -58,7 +57,7 @@ class Message(ModelReprMixin, models.Model): def timestamp(self) -> datetime: """Attribute that represents the message timestamp as derived from the snowflake id.""" tz_naive_datetime = datetime.utcfromtimestamp(((self.id >> 22) + 1420070400000) / 1000) - tz_aware_datetime = timezone.make_aware(tz_naive_datetime, timezone=pytz.timezone("UTC")) + tz_aware_datetime = timezone.make_aware(tz_naive_datetime, timezone=timezone.utc) return tz_aware_datetime class Meta: diff --git a/pydis_site/apps/api/models/bot/tag.py b/pydis_site/apps/api/models/bot/tag.py index fc0db4e4..8e7643ce 100644 --- a/pydis_site/apps/api/models/bot/tag.py +++ b/pydis_site/apps/api/models/bot/tag.py @@ -30,7 +30,7 @@ def validate_tag_embed_fields(fields): raise ValidationError("Embed fields must be a mapping.") if not all(required_field in field for required_field in required_fields): - raise ValidationError("Embed fields must contain both a 'name' and a 'value' field.") + raise ValidationError(f"Embed fields must contain the following fields: {', '.join(required_fields)}.") for field_name, value in field.items(): if field_name not in field_validators: -- cgit v1.2.3