diff options
| author |  Sebastiaan Zeeff <[email protected]> | 2019-08-24 12:35:29 +0200 |
|---|---|---|
| committer |  GitHub <[email protected]> | 2019-08-24 12:35:29 +0200 |
| commit | a756fb69d30a1fb522ff0658f2e2843d1d283490 (patch) | |
| tree | e283c48864d6d5d71243f29dfbe523d50aa399e8 /pydis_site | |
| parent | Changing LogView to DetailView, as per Volcy's review (diff) | |
Apply suggestions from code review
Co-Authored-By: Johannes Christ <[email protected]>
Diffstat (limited to 'pydis_site')
| -rw-r--r-- | pydis_site/apps/api/models/bot/message.py | 3 | ||||
| -rw-r--r-- | pydis_site/apps/api/models/bot/tag.py | 2 |
2 files changed, 2 insertions, 3 deletions
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: |