refactor (breaking): rename Setup Wizard slide to Onboarding Slide

This commit is contained in:
Shivam Mishra 2019-12-02 13:03:39 +05:30
parent b6418ca059
commit d648c4e1b9
9 changed files with 49 additions and 12 deletions

View file

@ -1,7 +1,7 @@
// Copyright (c) 2019, Frappe Technologies and contributors
// For license information, please see license.txt
frappe.ui.form.on('Setup Wizard Slide', {
frappe.ui.form.on('Onboarding Slide', {
refresh: function(frm) {
frm.toggle_reqd('ref_doctype', frm.doc.slide_type!='Information');
frm.toggle_reqd('slide_module', frm.doc.slide_type=='Information');
@ -33,7 +33,7 @@ frappe.ui.form.on('Setup Wizard Slide', {
reqd: 1
});
$.each(fields, function(_i, data) {
let row = frappe.model.add_child(frm.doc, 'Setup Wizard Slide', 'slide_fields');
let row = frappe.model.add_child(frm.doc, 'Onboarding Slide', 'slide_fields');
row.label = data.label;
row.fieldtype = data.fieldtype;
row.fieldname = data.fieldname;

View file

@ -68,7 +68,7 @@
"fieldname": "slide_fields",
"fieldtype": "Table",
"label": "Slide Fields",
"options": "Setup Wizard Slide Field"
"options": "Onboarding Slide Field"
},
{
"fieldname": "section_break_10",
@ -160,7 +160,7 @@
"modified": "2019-11-30 13:58:05.688981",
"modified_by": "Administrator",
"module": "Desk",
"name": "Setup Wizard Slide",
"name": "Onboarding Slide",
"owner": "Administrator",
"permissions": [
{

View file

@ -8,22 +8,22 @@ import json
from frappe.model.document import Document
from frappe.modules.export_file import export_to_files
class SetupWizardSlide(Document):
class OnboardingSlide(Document):
def on_update(self):
if self.ref_doctype:
module = frappe.db.get_value('DocType', self.ref_doctype, 'module')
else:
module = self.slide_module
export_to_files(record_list=[['Setup Wizard Slide', self.name]], record_module=module)
export_to_files(record_list=[['Onboarding Slide', self.name]], record_module=module)
def get_onboarding_slides_as_list():
slides = []
slide_docs = frappe.get_all('Setup Wizard Slide',
slide_docs = frappe.get_all('Onboarding Slide',
filters={'slide_order': ('!=', 0)},
order_by='slide_order')
for entry in slide_docs:
# using get_doc because child table fields are not fetched in get_all
slide_doc = frappe.get_doc('Setup Wizard Slide', entry.name)
slide_doc = frappe.get_doc('Onboarding Slide', entry.name)
if frappe.scrub(slide_doc.app) in frappe.get_installed_apps():
slides.append(frappe._dict(
slide_type=slide_doc.slide_type,

View file

@ -6,5 +6,5 @@ from __future__ import unicode_literals
# import frappe
import unittest
class TestSetupWizardSlide(unittest.TestCase):
class TestOnboardingSlide(unittest.TestCase):
pass

View file

@ -66,7 +66,7 @@
"modified": "2019-11-25 16:50:53.994656",
"modified_by": "Administrator",
"module": "Desk",
"name": "Setup Wizard Slide Field",
"name": "Onboarding Slide Field",
"owner": "Administrator",
"permissions": [],
"sort_field": "modified",

View file

@ -0,0 +1,37 @@
{
"add_more_button": 0,
"app": "ERPNext",
"creation": "2019-11-22 13:25:42.892593",
"docstatus": 0,
"doctype": "Onboarding Slide",
"domains": [],
"help_links": [
{
"label": "Need Help?",
"video_id": "cKZHcx1znMc"
}
],
"idx": 0,
"image_src": "/assets/erpnext/images/illustrations/letterhead-onboard.png",
"max_count": 0,
"modified": "2019-12-02 12:57:41.353913",
"modified_by": "Administrator",
"name": "Company Letter Head",
"owner": "Administrator",
"ref_doctype": "Letter Head",
"slide_desc": "<p>The letter head will appear across all print formats and PDFs</p>\n<p class=\"text-muted\">Keep it web friendly as 1024px by 128px</p>",
"slide_fields": [
{
"align": "center",
"fieldname": "letterhead",
"fieldtype": "Attach Image",
"label": "Attach Letterhead",
"options": "image",
"reqd": 0
}
],
"slide_order": 20,
"slide_title": "Company Letter Head",
"slide_type": "Create",
"submit_method": ""
}

View file

@ -3,7 +3,7 @@
"app": "ERPNext",
"creation": "2019-11-22 13:25:42.892593",
"docstatus": 0,
"doctype": "Setup Wizard Slide",
"doctype": "Onboarding Slide",
"domains": [],
"help_links": [
{

View file

@ -88,7 +88,7 @@ frappe.Application = Class.extend({
}
this.show_update_available();
if (frappe.boot.is_first_startup) {
if (frappe.boot.is_first_startup || true) {
this.setup_onboarding_wizard();
}