From abcde393535d162c8d20ab6fdaa590d5fbc2b181 Mon Sep 17 00:00:00 2001
From: Thomas Rittson <31796059+eliykat@users.noreply.github.com>
Date: Fri, 26 Jul 2024 07:15:30 +1000
Subject: [PATCH] Remove TODO from OrganizationLicense (#4553)

---
 src/Core/Models/Business/OrganizationLicense.cs | 15 +++++----------
 1 file changed, 5 insertions(+), 10 deletions(-)

diff --git a/src/Core/Models/Business/OrganizationLicense.cs b/src/Core/Models/Business/OrganizationLicense.cs
index cfc374ad86..d5c54ef3e9 100644
--- a/src/Core/Models/Business/OrganizationLicense.cs
+++ b/src/Core/Models/Business/OrganizationLicense.cs
@@ -350,16 +350,11 @@ public class OrganizationLicense : ILicense
                         organization.SmServiceAccounts == SmServiceAccounts;
             }
 
-            // Restore validity check when Flexible Collections are enabled for cloud and self-host
-            // https://bitwarden.atlassian.net/browse/AC-1875
-            // if (valid && Version >= 14)
-            // {
-            //     valid = organization.LimitCollectionCreationDeletion == LimitCollectionCreationDeletion;
-            // }
-            // if (valid && Version >= 15)
-            // {
-            //     valid = organization.AllowAdminAccessToAllCollectionItems == AllowAdminAccessToAllCollectionItems;
-            // }
+            /*
+             * Version 14 added LimitCollectionCreationDeletion and Version 15 added AllowAdminAccessToAllCollectionItems,
+             * however these are just user settings and it is not worth failing validation if they mismatch.
+             * They are intentionally excluded.
+             */
 
             return valid;
         }