diff --git a/app/assets/javascripts/discourse/mixins/ajax.js b/app/assets/javascripts/discourse/mixins/ajax.js
index b9f84255f51..c32abf8b180 100644
--- a/app/assets/javascripts/discourse/mixins/ajax.js
+++ b/app/assets/javascripts/discourse/mixins/ajax.js
@@ -63,6 +63,8 @@ Discourse.Ajax = Em.Mixin.create({
         xhr.jqTextStatus = textStatus;
         xhr.requestedUrl = url;
 
+        // TODO is this sequence correct? we are calling catch defined externally before
+        // the error that was defined inline, it should probably be in reverse
         Ember.run(null, reject, xhr);
         if (oldError) oldError(xhr);
       };