summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLV-426 <lv-426@taproot.org.il>2015-01-18 17:05:36 +0200
committerLV-426 <lv-426@taproot.org.il>2015-01-18 17:08:05 +0200
commitddb4649b237183756dcb6615a708681bd1688895 (patch)
tree6fc3f61cb6f3774ca61bc61f8b523baae27aef7b
parent9f03ef3c55894ab4815b4d4ff13c9da78e185bff (diff)
rz_feedback: cleanup msg_body construction
-rw-r--r--src/server/rz_feedback.py25
1 files changed, 9 insertions, 16 deletions
diff --git a/src/server/rz_feedback.py b/src/server/rz_feedback.py
index 24b17e0a..555d6d86 100644
--- a/src/server/rz_feedback.py
+++ b/src/server/rz_feedback.py
@@ -40,21 +40,14 @@ def send_user_feedback__email():
# FIXME: should be async via celery (or another method)
session_user = session.get('username')
- feedback_body = """Feedback from user:
-%(user)s
-watching URL:
-%(url)s
+ msg_body = ['Feedback from user:',
+ '',
+ 'user: %s' % (session_user if session_user else "<not-logged-in>"),
+ 'user-agent: %s' % (u_feedback.user_agent),
+ 'watching URL: %s' % (u_feedback.url),
+ 'user-note: %s' % (u_feedback.note),
+ ''
+ ]
+ msg_body = '\n'.join(msg_body)
-Left note:
-%(note)s
-""" % dict(url=url, note=note, user=session_user if session_user else "Not logged in")
- send_message(recipients=[current_app.rz_config.feedback_recipient],
- subject="User feedback from Rhizi",
- body=feedback_body,
- attachments=[
- ('feedback_screenshot.png', 'image/png', img),
- ('feedback_page.html', 'text/html', html),
- ])
-
- return "{}" # expects json encoded, contents discarded