comments + clean

tracim/lib/core/group.py

@@ -17,7 +17,6 @@ class GroupApi(object):
             session: Session,
             current_user: typing.Optional[User],
             config: CFG
-
     ):
         self._user = current_user
         self._session = session

tracim/models/applications.py

@@ -25,7 +25,6 @@ class Application(object):
         self.main_route = main_route
 
 
-# TODO - G.M - 21-05-2018 Do not rely on hardcoded app list
 # default apps
 calendar = Application(
     label='Calendar',
@@ -77,6 +76,7 @@ pagehtml = Application(
     config={},
     main_route='/#/workspaces/{workspace_id}/contents?type=pagehtml',
 )
+# TODO - G.M - 08-06-2018 - This is hardcoded lists of app, make this dynamic.
 # List of applications
 applications = [
     pagehtml,

tracim/models/workspace_menu_entries.py

@@ -24,7 +24,6 @@ class WorkspaceMenuEntry(object):
         self.hexcolor = hexcolor
         self.icon = icon
 
-
 dashboard_menu_entry = WorkspaceMenuEntry(
   slug='dashboard',
   label='Dashboard',
@@ -40,7 +39,8 @@ all_content_menu_entry = WorkspaceMenuEntry(
   icon="",
 )
 
-
+# TODO - G.M - 08-06-2018 - This is hardcoded default menu entry,
+#  of app, make this dynamic (and loaded from application system)
 def default_workspace_menu_entry(
     workspace: Workspace,
 )-> typing.List[WorkspaceMenuEntry]:

tracim/views/core_api/workspace_controller.py

@@ -79,8 +79,8 @@ class WorkspaceController(Controller):
 
     def bind(self, configurator: Configurator) -> None:
         """
-        Create all routes and views using pyramid configurator
-        for this controller
+        Create all routes and views using
+        pyramid configurator for this controller
         """
 
         # Applications