MNT Remove TODO comments

silverstripe · Oct 13, 2023 · 678aaeb · 678aaeb
1 parent 44a9b34
commit 678aaeb
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 4 deletions.
diff --git a/src/Auth/Handler.php b/src/Auth/Handler.php
@@ -22,7 +22,6 @@ class Handler
      * @var array
      *
      * @internal Experimental config:
-     * @todo Move this to a per-schema configuration and refer to this schema from the current endpoint
      * @link https://github.com/silverstripe/silverstripe-graphql/issues/58
      * @link https://github.com/silverstripe/silverstripe-graphql/issues/52
      */

diff --git a/src/Middleware/QueryCachingMiddleware.php b/src/Middleware/QueryCachingMiddleware.php
@@ -112,7 +112,6 @@ protected function getCachedResponse(string $key): ?array
         // On cache success validate against cached classes
         foreach ($cached['classes'] as $class) {
             // Note: Could combine these clases into a UNION to cut down on extravagant queries
-            // Todo: We can get last-deleted/modified as well for versioned records
             $lastEditedDate = DataObject::get($class)->max('LastEdited');
             if (strtotime($lastEditedDate ?? '') > strtotime($cached['date'] ?? '')) {
                 // class modified, fail validation of cache

diff --git a/src/Schema/DataObject/InterfaceBuilder.php b/src/Schema/DataObject/InterfaceBuilder.php
@@ -65,8 +65,6 @@ public function createInterfaces(ModelType $modelType, array $interfaceStack = [
         )
             ->setTypeResolver([AbstractTypeResolver::class, 'resolveType']);
 
-        // TODO: this makes a really good case for
-        // https://github.com/silverstripe/silverstripe-graphql/issues/364
         $validPlugins = [];
         foreach ($modelType->getPlugins() as $name => $config) {
             $plugin = $modelType->getPluginRegistry()->getPluginByID($name);