remove invocation email reporting TODO comments which are resolved

Co-authored-by: Marius van den Beek <m.vandenbeek@gmail.com>

Author: ahmedhamidawan

client/src/components/WorkflowInvocationState/WorkflowInvocationState.vue

@@ -141,8 +141,6 @@ const invocationStateSuccess = computed(() => {
     );
 });
 
-// TODO: Decide if the entire tab should only be rendered if it is the user's own workflow run.
-// We could check ownership by comparing the invocation history user ID in the `WorkflowInvocationFeedback` component?
 const canSubmitFeedback = computed(
     () =>
         invocationAndJobTerminal.value && (invocationState.value === "failed" || Boolean(stateCounts.value?.errorCount))

lib/galaxy/workflow/errors.py

@@ -122,14 +122,6 @@ def create_report(self, user, email="", message="", redact_user_details_in_bugre
             f"/published/workflow?id={stored_workflow_id_encoded}&version={workflow_version}", qualified=True
         )
 
-        # TODO: We could maybe also include the invocation messages, but I believe they often link steps
-        # which aren't neccessarily the ones where the first jobs fail.
-        # invocation_messages = self.invocation.messages
-        # # Format messages for display
-        # if invocation_messages:
-        #     messages_str = "\n".join([f"- {msg[0].reason}" for msg in invocation_messages])
-        # else:
-        #     messages_str = "No messages"
 
         # Build the email message
         if redact_user_details_in_bugreport: