From ae96409c1ced2525e53f1558af6bbaa4dcd92c8a Mon Sep 17 00:00:00 2001
From: Will Hunt <will@half-shot.uk>
Date: Tue, 21 Nov 2023 15:20:27 +0000
Subject: [PATCH] Apply suggestions from code review

---
 proposals/1929-admin-contact.md | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/proposals/1929-admin-contact.md b/proposals/1929-admin-contact.md
index ec08e393..a947689c 100644
--- a/proposals/1929-admin-contact.md
+++ b/proposals/1929-admin-contact.md
@@ -50,6 +50,8 @@ that the value be namespaced to prevent collisions. A value for `role` MUST be s
 `support_page` is an optional property to specify a affiliated page of the homserver to give users help
 specific to the homeserver, like extra login/registration steps.
 
+At least one valid key should be provided. This means `contacts` should have at least one entry, or the `support_page` should be defined. An empty object is not considered valid, however both `contacts `and `support_page` may be specified together. 
+
 ## Alternative solutions
 
 Hardcode a given user localpart that should be used as an admin address.