diff --git a/app/components/SimpleEditor/SimpleEditor.jsx b/app/components/SimpleEditor/SimpleEditor.jsx
index b86431cce9965bdcf78b4d49a8d0e3b46b38b829..59fca1bee4aabbd61442d56d56ee83ea9d739631 100644
--- a/app/components/SimpleEditor/SimpleEditor.jsx
+++ b/app/components/SimpleEditor/SimpleEditor.jsx
@@ -140,7 +140,6 @@ export default class SimpleEditor extends React.Component {
     const el = ReactDOM.findDOMNode(this)
 
     const { book, fileUpload, fragment, history, onSave, update, user } = this.props
-
     const { configurator, documentSession } = this.createSession()
 
     if (!fragment) return
@@ -152,6 +151,7 @@ export default class SimpleEditor extends React.Component {
     const updateTrackChangesStatus = this.updateTrackChangesStatus
     const disabled = !this.state.canEdit
 
+    // TODO -- justify
     this.setState({
       config: configurator.config
     })
@@ -171,6 +171,7 @@ export default class SimpleEditor extends React.Component {
       updateTrackChangesStatus,
       user
     }, el)
+    // TODO -- pull all above into its own function
 
     window.addEventListener('beforeunload', this._releaseLock)
   }
@@ -225,6 +226,7 @@ export default class SimpleEditor extends React.Component {
 
 SimpleEditor.propTypes = {
   book: React.PropTypes.object.isRequired,
+  canEdit: React.PropTypes.bool, // needed?
   fragment: React.PropTypes.object,
   history: React.PropTypes.object.isRequired,
   onSave: React.PropTypes.func.isRequired,
diff --git a/app/components/SimpleEditor/SimpleEditorWrapper.jsx b/app/components/SimpleEditor/SimpleEditorWrapper.jsx
index 1bdb3c9fff604c05aa6bcf4cdfd23236900ad49f..e80343c18f249d7424fae6036fc11a8c79682e64 100644
--- a/app/components/SimpleEditor/SimpleEditorWrapper.jsx
+++ b/app/components/SimpleEditor/SimpleEditorWrapper.jsx
@@ -84,13 +84,14 @@ export class SimpleEditorWrapper extends React.Component {
   }
 }
 
+// TODO -- review required props
 SimpleEditorWrapper.propTypes = {
   actions: React.PropTypes.object.isRequired,
   book: React.PropTypes.object.isRequired,
   fragment: React.PropTypes.object,
   history: React.PropTypes.object.isRequired,
-  update: React.PropTypes.func,
-  user: React.PropTypes.object.isRequired
+  user: React.PropTypes.object.isRequired,
+  update: React.PropTypes.func
 }
 
 const mapStateToProps = (state, ownProps) => {