Skip to content

Commit 113f773

Browse files
committed
fix(models): better error messages than the built in Django ones for a few of the fields
Fixes #848 Fixes #849
1 parent 537b90b commit 113f773

2 files changed

Lines changed: 12 additions & 2 deletions

File tree

rootfs/api/models/domain.py

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,12 @@
77
class Domain(AuditedModel):
88
owner = models.ForeignKey(settings.AUTH_USER_MODEL, on_delete=models.PROTECT)
99
app = models.ForeignKey('App', on_delete=models.CASCADE)
10-
domain = models.TextField(blank=False, null=False, unique=True)
10+
domain = models.TextField(
11+
blank=False, null=False, unique=True,
12+
error_messages={
13+
'unique': 'Domain is already in use by another application'
14+
}
15+
)
1116
certificate = models.ForeignKey(
1217
'Certificate',
1318
on_delete=models.SET_NULL,

rootfs/api/models/key.py

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,12 @@ class Key(UuidAuditedModel):
2121

2222
owner = models.ForeignKey(settings.AUTH_USER_MODEL, on_delete=models.CASCADE)
2323
id = models.CharField(max_length=128)
24-
public = models.TextField(unique=True, validators=[validate_base64])
24+
public = models.TextField(
25+
unique=True, validators=[validate_base64],
26+
error_messages={
27+
'unique': 'Public Key is already in use'
28+
}
29+
)
2530
fingerprint = models.CharField(max_length=128, editable=False)
2631

2732
class Meta:

0 commit comments

Comments
 (0)