+
{{ _("Full Name") }}
+
{{ _("Detailed feedback") }}
@@ -53,7 +55,6 @@
this.rating = get_url_arg("rating") || 0;
// set ratings
- console.log(this.rating)
this.set_ratings_icon(this.rating)
this.bind_events();
@@ -81,6 +82,9 @@
if(!$('.star-icon.fa-star').length) {
frappe.msgprint(__("Please give a rating."));
return;
+ } else if(!$('.fullname').val().length){
+ frappe.msgprint(__("Please give a fullname."));
+ return;
} else if(!$('.feedback-text').val().length){
frappe.msgprint(__("Please give a detailed feebdack."));
return;
@@ -94,7 +98,8 @@
reference_name: me.reference_name,
reference_doctype: me.reference_doctype,
feedback: $('.feedback-text').val(),
- rating: $('.star-icon.fa-star').length
+ rating: $('.star-icon.fa-star').length,
+ fullname: $('.fullname').val()
},
callback: function(r) {
if(r.message) {
diff --git a/frappe/www/feedback.py b/frappe/www/feedback.py
index 7e75f8f0ac..70d6c0efae 100644
--- a/frappe/www/feedback.py
+++ b/frappe/www/feedback.py
@@ -6,6 +6,10 @@ no_cache = True
def get_context(context):
reference_doctype = frappe.form_dict.get("reference_doctype")
reference_name = frappe.form_dict.get("reference_name")
+
+ if not all([reference_name, reference_doctype]):
+ return {}
+
communications = frappe.get_all("Communication", filters={
"reference_doctype": reference_doctype,
"reference_name": reference_name,
@@ -20,7 +24,7 @@ def get_context(context):
}
@frappe.whitelist(allow_guest=True)
-def accept(key, sender, reference_doctype, reference_name, feedback, rating):
+def accept(key, sender, reference_doctype, reference_name, feedback, rating, fullname):
""" save the feedback in communication """
if not reference_doctype and not reference_name:
frappe.throw("Invalid Reference Doctype, Reference Name")
@@ -36,13 +40,14 @@ def accept(key, sender, reference_doctype, reference_name, feedback, rating):
communication = frappe.get_doc({
"rating": rating,
- "sender": sender,
"status": "Closed",
- "feedback": feedback,
+ "content": feedback,
"doctype": "Communication",
+ "sender": sender or "Guest",
"sent_or_received": "Received",
"communication_type": "Feedback",
"reference_name": reference_name,
+ "sender_full_name": fullname or "",
"feedback_request": feedback_request,
"reference_doctype": reference_doctype,
"subject": "Feedback for {0} {1}".format(reference_doctype, reference_name),