docfix/Add todo regarding user agreement

OpenBankProject · Sep 9, 2024 · 464e3c7 · 464e3c7
1 parent 4203be7
commit 464e3c7
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/obp-api/src/main/scala/code/users/UserAgreement.scala b/obp-api/src/main/scala/code/users/UserAgreement.scala
@@ -10,11 +10,13 @@ import net.liftweb.mapper._
 import net.liftweb.common.Box.tryo
 
 object MappedUserAgreementProvider extends UserAgreementProvider {
+  // TODO Change the function name
   override def createOrUpdateUserAgreement(userId: String, agreementType: String, agreementText: String): Box[UserAgreement] = {
     UserAgreement.find(
       By(UserAgreement.UserId, userId),
       By(UserAgreement.AgreementType, agreementType)
     ) match {
+      // TODO We should be adding an additional record. Not changing existing one.
       case Full(existingUser) =>
         Full(
           existingUser