vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 4

fix: Remove mandatory and save URL params to localstorage

Browse source 
- Saving URL params to localstorage to avoid re-entering the data.
Usually only 1 or 2 param(s) change is required to generate new link
This commit is contained in:
Suraj Shetty 2023-03-21 11:31:05 +05:30
parent 1abcd5a11a
commit 02b661bbb9
2 changed files with 10 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
frappe/email/doctype/newsletter/newsletter.json
View file

@ -244,7 +244,7 @@
"fieldtype": "Link",
"label": "Campaign",
"options": "Marketing Campaign",
"reqd": 1
"reqd": 0
}
],
"has_web_view": 1,

9
frappe/public/js/frappe/utils/utils.js
View file

@ -1619,11 +1619,13 @@ Object.assign(frappe.utils, {
fieldtype: "Data",
options: "URL",
reqd: 1,
default: localStorage.getItem("tracker_url:url"),
},
{
fieldname: "source",
label: __("Source"),
fieldtype: "Data",
default: localStorage.getItem("tracker_url:source"),
},
{
fieldname: "campaign",
@ -1631,23 +1633,30 @@ Object.assign(frappe.utils, {
fieldtype: "Link",
ignore_link_validation: 1,
options: "Marketing Campaign",
default: localStorage.getItem("tracker_url:campaign"),
},
{
fieldname: "medium",
label: __("Medium"),
fieldtype: "Data",
default: localStorage.getItem("tracker_url:medium"),
},
],
function (data) {
let url = data.url;
localStorage.setItem("tracker_url:url", data.url);
if (data.source) {
url += "?source=" + data.source;
localStorage.setItem("tracker_url:source", data.source);
}
if (data.campaign) {
url += "&campaign=" + data.campaign;
localStorage.setItem("tracker_url:campaign", data.campaign);
}
if (data.medium) {
url += "&medium=" + data.medium.toLowerCase();
localStorage.setItem("tracker_url:medium", data.medium);
}
frappe.utils.copy_to_clipboard(url);