Fix a bit of code rot in the Python SDK

Summary:
Addresses Evan's comments in T3498. Notably:
- add a `client.account` property.
- fix the README and the example app.

Test Plan: Tested manually.

Reviewers: spang

Reviewed By: spang

Subscribers: evan

Maniphest Tasks: T3498

Differential Revision: https://phab.nylas.com/D2042

Author: khamidou

README.md

@@ -49,7 +49,7 @@ from nylas import APIClient
 def index():
     redirect_url = "http://0.0.0.0:8888/login_callback"
     client = APIClient(APP_ID, APP_SECRET)
-    return redirect(client.authentication_url(redirect_uri))
+    return redirect(client.authentication_url(redirect_url))
 
 ```
 
@@ -76,8 +76,8 @@ implementing the auth flow.
 client = APIClient(APP_ID, APP_SECRET, token)
 
 # Print out the email address and provider (Gmail, Exchange)
-print client.email_address
-print client.provider
+print client.account.email_address
+print client.account.provider
 ```
 
 

nylas/client/client.py

@@ -9,7 +9,8 @@
 from .util import url_concat, generate_id
 from .restful_model_collection import RestfulModelCollection
 from .restful_models import (Calendar, Contact, Event, Message, Thread, File,
-                             Account, APIAccount, Tag, Folder, Label, Draft)
+                             Account, APIAccount, SingletonAccount, Tag,
+                             Folder, Label, Draft)
 from .errors import (APIClientError, ConnectionError, NotAuthorizedError,
                      InvalidRequestError, NotFoundError, MethodNotSupportedError,
                      ServerError, ServiceUnavailableError, ConflictError,
@@ -159,6 +160,10 @@ def is_opensource_api(self):
 
         return False
 
+    @property
+    def account(self):
+        return self._get_resource(SingletonAccount, '')
+
     @property
     def accounts(self):
         if self.is_opensource_api():

nylas/client/restful_models.py

@@ -7,10 +7,10 @@
 
 class NylasAPIObject(dict):
     attrs = []
-    # The Nylas API holds most objectsfor an acount, but some of
-    # them are under '/a' (mostly the account-management and billing code).
-    # api_root is a tiny metaprogramming hack to let us use the same
-    # code for both.
+    # The Nylas API holds most objects for an account directly under '/',
+    # but some of them are under '/a' (mostly the account-management
+    # and billing code). api_root is a tiny metaprogramming hack to let
+    # us use the same code for both.
     api_root = 'n'
 
     def __init__(self, cls, api):
@@ -443,10 +443,6 @@ def child_collection(self, cls, **filters):
 
 
 class Account(NylasAPIObject):
-    # The Nylas API holds most objects under '/n/', but some of
-    # them are under '/a' (mostly the account-management and billing code).
-    # api_root is a tiny metaprogramming hack to let us use the same
-    # code for both.
     api_root = 'a'
 
     attrs = ["account_id", "trial", "trial_expires", "sync_state",
@@ -482,3 +478,8 @@ def __init__(self, api):
     def as_json(self):
         dct = NylasAPIObject.as_json(self)
         return dct
+
+
+class SingletonAccount(APIAccount):
+    # This is an APIAccount that lives under /account.
+    collection_name = 'account'