Message refactor

- renamed Ok to Connected message (used after successful connection) - add Ok message as general purpose successful message type
ScalaConsultants · Jun 24, 2015 · 513cf4c · 513cf4c
1 parent 0e4af56
commit 513cf4c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/src/main/scala/io/scalac/slack/api/ApiActor.scala b/src/main/scala/io/scalac/slack/api/ApiActor.scala
@@ -10,7 +10,7 @@ import spray.json._
 import scala.util.{Failure, Success}
 
 /**
- * Created on 21.01.15 20:32
+ *  marioosh
  */
 class ApiActor extends Actor with ActorLogging {
 
@@ -27,7 +27,7 @@ class ApiActor extends Actor with ActorLogging {
       SlackApiClient.get[ApiTestResponse]("api.test", params) onComplete {
         case Success(res) =>
           if (res.ok) {
-            send ! Ok(res.args)
+            send ! Connected
           }
           else {
             send ! ApiTestError

diff --git a/src/main/scala/io/scalac/slack/api/Message.scala b/src/main/scala/io/scalac/slack/api/Message.scala
@@ -12,16 +12,14 @@ case object RegisterModules extends Message
 
 //API CALLS
 case class ApiTest(param: Option[String] = None, error: Option[String] = None) extends Message
-
 case class AuthTest(token: APIKey) extends Message
-
 case class RtmStart(token: APIKey) extends Message
 
 //API RESPONSES
-case class Ok(args: Option[Map[String, String]]) extends Message
+case object Ok extends Message
+case object Connected extends Message
 
 case class AuthData(url: String, team: String, user: String, teamId: String, userId: String) extends Message
-
 case class RtmData(url: String)
 
 object AuthData {