Remove TODO item which is no more valid (#21591)

project-chip · Jan 9, 2024 · 2301208 · 2301208
1 parent d969d96
commit 2301208
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 4 deletions.
diff --git a/src/app/clusters/operational-credentials-server/operational-credentials-server.cpp b/src/app/clusters/operational-credentials-server/operational-credentials-server.cpp
@@ -739,8 +739,6 @@ bool emberAfOperationalCredentialsClusterAddNOCCallback(app::CommandHandler * co
             (void) groupDataProvider->RemoveFabric(newFabricIndex);
         }
 
-        // TODO(#19898): All ACL work done within AddNOC does not trigger ACL cluster updates
-
         (void) Access::GetAccessControl().DeleteAllEntriesForFabric(newFabricIndex);
 
         MatterReportingAttributeChangeCallback(commandPath.mEndpointId, OperationalCredentials::Id,

diff --git a/src/app/server/Server.h b/src/app/server/Server.h
@@ -438,9 +438,7 @@ class Server
             // Remove access control entries in reverse order (it could be any order, but reverse order
             // will cause less churn in persistent storage).
 
-            // TODO(#19898): The fabric removal not trigger ACL cluster updates
             // TODO(#19899): The fabric removal not remove ACL extensions
-
             CHIP_ERROR aclErr = Access::GetAccessControl().DeleteAllEntriesForFabric(fabricIndex);
             if (aclErr != CHIP_NO_ERROR)
             {