[Minor] Job Offer UX improvise (#15042)
* job offer ux improv
* Update job_offer.js
diff --git a/erpnext/hr/doctype/job_offer/job_offer.js b/erpnext/hr/doctype/job_offer/job_offer.js
index f90feae..367ce38 100755
--- a/erpnext/hr/doctype/job_offer/job_offer.js
+++ b/erpnext/hr/doctype/job_offer/job_offer.js
@@ -13,13 +13,22 @@
},
refresh: function (frm) {
- if ((!frm.doc.__islocal) && (frm.doc.status == 'Accepted') && (frm.doc.docstatus === 1)) {
+ if ((!frm.doc.__islocal) && (frm.doc.status == 'Accepted')
+ && (frm.doc.docstatus === 1) && (!frm.doc.__onload.employee)) {
frm.add_custom_button(__('Make Employee'),
function () {
erpnext.job_offer.make_employee(frm);
}
);
}
+
+ if(frm.doc.__onload.employee) {
+ frm.add_custom_button(__('Show Employee'),
+ function () {
+ frappe.set_route("Form", "Employee", frm.doc.__onload.employee);
+ }
+ );
+ }
}
});
diff --git a/erpnext/hr/doctype/job_offer/job_offer.py b/erpnext/hr/doctype/job_offer/job_offer.py
index 813b442..7e3014b 100644
--- a/erpnext/hr/doctype/job_offer/job_offer.py
+++ b/erpnext/hr/doctype/job_offer/job_offer.py
@@ -7,7 +7,9 @@
from frappe.model.mapper import get_mapped_doc
class JobOffer(Document):
- pass
+ def onload(self):
+ employee = frappe.db.get_value("Employee", {"job_applicant": self.job_applicant}, "name") or ""
+ self.set_onload("employee", employee)
@frappe.whitelist()
def make_employee(source_name, target_doc=None):