add TODOs

Author: lobsterkatie

src/sentry/interfaces/stacktrace.py

@@ -93,6 +93,8 @@ def get_context(lineno, context_line, pre_context=None, post_context=None):
 
 
 def is_newest_frame_first(event):
+    # TODO: Why does python get a different default here, when it doesn't anywhere else that we
+    # check stacktrace order?
     newest_first = event.platform not in ("python", None)
 
     if env.request and env.request.user.is_authenticated:

src/sentry/users/api/serializers/user.py

@@ -194,7 +194,9 @@ def serialize(
             }
 
             stacktrace_order = int(
-                options.get("stacktrace_order", StacktraceOrder.DEFAULT) or StacktraceOrder.DEFAULT
+                options.get("stacktrace_order", StacktraceOrder.DEFAULT)
+                # TODO: This second `or` won't be necessary once we remove empty strings from the DB
+                or StacktraceOrder.DEFAULT
             )
 
             d["options"] = {