add TODOs

Author: lobsterkatie

src/sentry/interfaces/stacktrace.py

@@ -93,6 +93,9 @@ def get_context(lineno, context_line, pre_context=None, post_context=None):
 
 
 def is_newest_frame_first(event):
+    # TODO: Investigate if we should keep this special-casing for python, since we don't special
+    # case it anywhere else we check stacktrace order.
+    # (See https://github.com/getsentry/sentry/pull/96719 for more context.)
     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"] = {