From be5d86ee468bdd04d4b777b42de00602d3b0c028 Mon Sep 17 00:00:00 2001
From: Sabina Talipova <sabina.talipova@silverstripe.com>
Date: Mon, 16 Oct 2023 16:05:41 +1300
Subject: [PATCH] MNT Remove TODO comments

---
 _config/extensions.yml                      | 7 -------
 src/Extension/CustomHtmlEditorField.php     | 4 ----
 src/Extension/LoginAttemptNotifications.php | 8 ++------
 src/Feed/CwpAtomFeed.php                    | 1 -
 4 files changed, 2 insertions(+), 18 deletions(-)

diff --git a/_config/extensions.yml b/_config/extensions.yml
index 98a7278..b37ecc8 100644
--- a/_config/extensions.yml
+++ b/_config/extensions.yml
@@ -1,12 +1,6 @@
 ---
 Name: cwpcoreextensions
 ---
-# @todo Disable the feature. LoginAttempt seems to be broken on bridging solution - logs every
-# request instead of logins!
-# Follow up issue: https://github.com/silverstripe/cwp-core/issues/41
-# SilverStripe\Admin\LeftAndMain:
-#   extensions:
-#     - CWP\Core\Extension\LoginAttemptNotifications
 
 SilverStripe\ORM\FieldType\DBField:
   extensions:
@@ -16,7 +10,6 @@ SilverStripe\Security\Group:
   extensions:
     - CWP\Core\Extension\CwpHtmlEditorConfig
 
-# @todo investigate replacement for this
 HtmlEditorField_Toolbar:
   extensions:
    - CWP\Core\Extension\CustomHtmlEditorFieldToolbar
diff --git a/src/Extension/CustomHtmlEditorField.php b/src/Extension/CustomHtmlEditorField.php
index 994c8b6..4e88700 100644
--- a/src/Extension/CustomHtmlEditorField.php
+++ b/src/Extension/CustomHtmlEditorField.php
@@ -6,10 +6,6 @@
 use SilverStripe\Forms\Form;
 use SilverStripe\View\Requirements;
 
-/**
- * @todo rename file to match class name
- * @todo investigate updated HTMLEditorField_Toolbar changes, and find new implementation
- */
 class CustomHtmlEditorFieldToolbar extends Extension
 {
 
diff --git a/src/Extension/LoginAttemptNotifications.php b/src/Extension/LoginAttemptNotifications.php
index 796b30d..6f1c78b 100644
--- a/src/Extension/LoginAttemptNotifications.php
+++ b/src/Extension/LoginAttemptNotifications.php
@@ -10,8 +10,6 @@
 use SilverStripe\View\Requirements;
 
 /**
- * TODO: describe.
- * TODO: bug when using default admin - always shows the message...
  * Requires Security::login_recording config to be set to true.
  *
  * @property LeftAndMain $owner
@@ -69,14 +67,12 @@ public function init()
 
                     $message .= "The attempts were from " . implode(', ', $IPs) . '. ';
 
-                    // TODO: add this call to action in a way that doesn't break out of the availabel space. Fix CSS?
-                    // $message .= "If you suspect somebody else might be trying to access
-                    //  . "your account, please contact support.";
+                    $message .= "If you suspect somebody else might be trying to access "
+                    . "your account, please contact support.";
                 }
             }
         } else {
             // New session - show last login attempt.
-            // TODO: this currently does NOT surface to the frontend in any way.
             $lastLoginAttempt = LoginAttempt::get()->filter([
                         'MemberID' => $member->ID
             ])->sort('Created DESC')->First();
diff --git a/src/Feed/CwpAtomFeed.php b/src/Feed/CwpAtomFeed.php
index 029e806..a2997a3 100644
--- a/src/Feed/CwpAtomFeed.php
+++ b/src/Feed/CwpAtomFeed.php
@@ -6,7 +6,6 @@
  * CwpAtomFeed class
  *
  * This class is used to create an Atom feed.
- * @todo Improve documentation
  * @package cwp-core
  */
 use SilverStripe\Control\Controller;