Removing color from tags.

jkeyes · jkeyes · commit d7d26b1e0678 · 2013-11-18T21:59:01.000Z
diff --git a/intercom/intercom.py b/intercom/intercom.py
@@ -358,8 +358,7 @@ def reply_message_thread(
 
     @classmethod
     def create_tag(
-            cls, name, tag_or_untag, user_ids=None, emails=None,
-            color=None):
+            cls, name, tag_or_untag, user_ids=None, emails=None):
         """ Create a tag (and maybe tag users).
 
         >>> tag = Intercom.create_tag("Free Trial", "tag",
@@ -377,17 +376,15 @@ def create_tag(
             'name': name,
             'tag_or_untag': tag_or_untag,
             'user_ids': user_ids,
-            'emails': emails,
-            'color': color
+            'emails': emails
         }
         tag_dict = Intercom._call(
             'POST', Intercom.api_endpoint + 'tags', params=params)
         return tag_dict
 
     @classmethod
     def update_tag(
-            cls, name, tag_or_untag, user_ids=None, emails=None,
-            color=None):
+            cls, name, tag_or_untag, user_ids=None, emails=None):
         """ Update a tag (and maybe tag users).
 
         >>> tag = Intercom.update_tag("Free Trial", "tag",
@@ -405,8 +402,7 @@ def update_tag(
             'name': name,
             'tag_or_untag': tag_or_untag,
             'user_ids': user_ids,
-            'emails': emails,
-            'color': color
+            'emails': emails
         }
         tag_dict = Intercom._call(
             'PUT', Intercom.api_endpoint + 'tags', params=params)
diff --git a/intercom/tag.py b/intercom/tag.py
@@ -52,8 +52,7 @@ def find_by_name(cls, name):
 
     @classmethod
     def create(
-            cls, name, tag_or_untag, user_ids=None, emails=None,
-            color=None):
+            cls, name, tag_or_untag, user_ids=None, emails=None):
         """ Create a new tag an optionally tag user.
 
         >>> tag = Tag.create(user_ids=["abc123", "def456"],
@@ -67,8 +66,7 @@ def create(
 
         """
         resp = Intercom.create_tag(
-            name, tag_or_untag, user_ids=user_ids, emails=emails,
-            color=color)
+            name, tag_or_untag, user_ids=user_ids, emails=emails)
         return cls(resp)
 
     def save(self):
@@ -86,8 +84,7 @@ def save(self):
         resp = Intercom.update_tag(
             self.name, self.get('tag_or_untag', None),
             user_ids=self.get('user_ids', None),
-            emails=self.get('emails', None),
-            color=self.color)
+            emails=self.get('emails', None))
         self.update(resp)
 
     @property
@@ -100,16 +97,6 @@ def name(self, name):
         """ Get the name of the tag. """
         self['name'] = name
 
-    @property
-    def color(self):
-        """ Get the color of the tag. """
-        return dict.get(self, 'color', None)
-
-    @color.setter
-    def color(self, color):
-        """ Get the color of the tag. """
-        self['color'] = color
-
     @property
     def id(self):
         """ The id of the tag. """
diff --git a/tests/unit/test_tag.py b/tests/unit/test_tag.py
@@ -67,14 +67,10 @@ def test_accessor(self):
         tag.name = "xyz"
         eq_("xyz", tag.name)
 
-        tag.color = "abc"
-        eq_("abc", tag.color)
-
     @patch('requests.request', create_response(200, 'create_tag_valid.json'))
     def test_create(self):
         tag = Tag.create(name="Poweruser", tag_or_untag="tag")
         eq_(None, tag.id)
-        eq_('green', tag.color)
         eq_(False, tag.segment)
         eq_(None, tag.tagged_user_count)
 
@@ -83,16 +79,13 @@ def test_update(self):
         tag = Tag()
         tag.save()
         eq_(None, tag.id)
-        eq_('green', tag.color)
 
     @patch('requests.request', create_response(200, 'get_tag_valid.json'))
     def test_find(self):
         tag = Tag.find(name="Poweruser")
         eq_(None, tag.id)
-        eq_('green', tag.color)
 
     @patch('requests.request', create_response(200, 'get_tag_valid.json'))
     def test_find_by_name(self):
         tag = Tag.find_by_name("Poweruser")
         eq_(None, tag.id)
-        eq_('green', tag.color)
