[Bug 18060] New: Add onboarding functionality to guide users through setting up labels for the first time
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 Bug ID: 18060 Summary: Add onboarding functionality to guide users through setting up labels for the first time Change sponsored?: --- Product: Koha Version: 17.05 Hardware: All OS: All Status: ASSIGNED Severity: enhancement Priority: P5 - low Component: Label/patron card printing Assignee: alexbuckley@catalyst.net.nz Reporter: alexbuckley@catalyst.net.nz QA Contact: testopia@bugs.koha-community.org To help make the process of creating a label for the first time easier this bug report requires the implementation of onboarding tool in the labels area of the staff client to be run the first time the user wants to create a label. The features of the onboarding tool includes tooltips and modal boxes to be displayed to the user to clarify the process to make a label. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |17855 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855 [Bug 17855] Updated Koha web installer and new onboarding tool feature to guide users through setting up Koha -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on|17855 |18002 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855 [Bug 17855] Updated Koha web installer and new onboarding tool feature to guide users through setting up Koha https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18002 [Bug 18002] Make onboarding tool include framework to be embedded in Acquisitions to help the users set up Acquisitions for the first time -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |veron@veron.ch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 --- Comment #1 from Alex Buckley <alexbuckley@catalyst.net.nz> --- Created attachment 63224 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63224&action=edit Bug 18060 - Initial label onboarding tool -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #63224|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 --- Comment #2 from Alex Buckley <alexbuckley@catalyst.net.nz> --- Created attachment 63225 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63225&action=edit Bug 18060 - Initial label onboarding tool -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #63225|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 --- Comment #3 from Alex Buckley <alexbuckley@catalyst.net.nz> --- Created attachment 63228 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63228&action=edit Bug 18060 - Initial label onboarding tool with working retrieval of layouts and templates -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #63228|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 --- Comment #4 from Alex Buckley <alexbuckley@catalyst.net.nz> --- Created attachment 63229 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63229&action=edit Bug 18060 - Initial label onboarding tool with working retrieval of layouts and templates -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 --- Comment #5 from Alex Buckley <alexbuckley@catalyst.net.nz> --- Created attachment 63587 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63587&action=edit Bug 18060 - Working label onboarding tool for creating a layout and a template -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 --- Comment #6 from Alex Buckley <alexbuckley@catalyst.net.nz> --- Created attachment 63588 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63588&action=edit Bug 18060 - Implemented redirect to label-edit-profile.pl after the user has created a template using the label onboarding tool -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 --- Comment #7 from Alex Buckley <alexbuckley@catalyst.net.nz> --- Created attachment 63589 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63589&action=edit Bug 18060 - Removed unused form in onboardinginclude3.inc -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 --- Comment #8 from Alex Buckley <alexbuckley@catalyst.net.nz> --- Test plan: 1. Make sure the creator_layouts, creator_templates and printers_profile database tables are empty 2. In the Koha staff interface navigate to Tools->Label-creator 3. Notice that a blank screen appears 4. Apply patches attached to this bug report 5. Refresh the label-home.pl page 6. Notice a form to create a layout appears 7. Fill out this layout creation form and submit it 8. Notice another form also on label-home.pl appears to create a template 9. Fill out the template creation form and submit the form 10. Notice you are redirected to label-edit-profile.pl to create a printer profile 11. Test if the label onboarding tool forms are displaying appropriately by clicking on Manage->Label templates and delete the template you just created 12. Now go to label-home.pl and notice that the template creation form is displayed 13. Fill out the form and submit it 14. Notice you are not redirected to label-edit-profile.pl because you have already created a profile which has not been deleted instead a blank label-home.pl is displayed -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #63229|1 |0 is patch| | Attachment #63229|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #63587|1 |0 is patch| | Attachment #63587|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #63588|0 |1 is obsolete| | Attachment #63588|1 |0 is patch| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #63589|0 |1 is obsolete| | Attachment #63589|1 |0 is patch| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 --- Comment #9 from Alex Buckley <alexbuckley@catalyst.net.nz> --- Created attachment 64733 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=64733&action=edit Bug 18060 - Label onboarding tool to be displayed when no labels or templates exist -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 Lee Jamison <ldjamison@marywood.edu> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |ldjamison@marywood.edu --- Comment #10 from Lee Jamison <ldjamison@marywood.edu> --- The 18002 dependency applies cleanly (tested during this bug as well as tested 18002 independently as well). However, this bug does not apply. See merge conflict message below. Applying: Bug 18060 - Label onboarding tool to be displayed when no labels or templates exist Using index info to reconstruct a base tree... M C4/Budgets.pm M acqui/acqui-home.pl M admin/aqbudgetperiods.pl M admin/aqbudgets.pl M admin/currency.pl M koha-tmpl/intranet-tmpl/prog/css/staff-global.css M koha-tmpl/intranet-tmpl/prog/en/includes/budgets-active-currency.inc M koha-tmpl/intranet-tmpl/prog/en/modules/acqui/acqui-home.tt M koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqbudgetperiods.tt M koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqbudgets.tt <stdin>:749: new blank line at EOF. + warning: 1 line adds whitespace errors. Falling back to patching base and 3-way merge... Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/acqui/acqui-home.tt CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/modules/acqui/acqui-home.tt Auto-merging koha-tmpl/intranet-tmpl/prog/en/includes/onboardinginclude3.inc CONFLICT (add/add): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/includes/onboardinginclude3.inc Auto-merging koha-tmpl/intranet-tmpl/prog/en/includes/onboardinginclude2.inc CONFLICT (add/add): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/includes/onboardinginclude2.inc Auto-merging koha-tmpl/intranet-tmpl/prog/en/includes/onboardinginclude1.inc CONFLICT (add/add): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/includes/onboardinginclude1.inc Auto-merging koha-tmpl/intranet-tmpl/prog/css/staff-global.css CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/css/staff-global.css Failed to merge in the changes. Patch failed at 0001 Bug 18060 - Label onboarding tool to be displayed when no labels or templates exist The copy of the patch that failed is found in: /home/vagrant/kohaclone/.git/rebase-apply/patch When you have resolved this problem run "git bz apply --continue". If you would prefer to skip this patch, instead run "git bz apply --skip". To restore the original branch and stop patching run "git bz apply --abort". Patch left in /tmp/Bug-18060---Label-onboarding-tool-to-be-displayed--p4jDwr.patch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #64733|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 --- Comment #11 from Alex Buckley <alexbuckley@catalyst.net.nz> --- Created attachment 64766 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=64766&action=edit Bug 18060 - Label onboarding tool to be displayed when no labels or templates exist Updated to apply cleanly with bug 18002 Please follow the test plan on comment 8 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA --- Comment #12 from Marc Véron <veron@veron.ch> --- Sorry, after applying the patch (on top of 18002, refresh memcahed and plack), I get an Internal error. Plack error log says: Variable "$layout" is not available at /home/marc/koha/labels/label-home.pl line 55. Can't call method "get_attr" on an undefined value at /home/marc/koha/labels/label-home.pl line 55. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 --- Comment #13 from Alex Buckley <alexbuckley@catalyst.net.nz> --- Created attachment 64951 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=64951&action=edit Bug 18060: Followup solving $layout error -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff --- Comment #14 from Alex Buckley <alexbuckley@catalyst.net.nz> --- Hi Marc Hopefully this attached followup patch solves the issue, it does on my machine -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 --- Comment #15 from Lee Jamison <ldjamison@marywood.edu> --- Hi Alex, Sorry, but even after applying your followup 64951 patch the same plack error Marc points out still occurs for me. I tested it on a fresh kohadevbox -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA CC| |katrin.fischer@bsz-bw.de --- Comment #16 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi Alex, applied this on top of 18002. QA tools first: FAIL C4/Creators/Layout.pm OK critic OK forbidden patterns OK git manipulation OK pod FAIL pod coverage POD is missing for 'retrieve_all' OK spelling OK valid FAIL C4/Creators/Profile.pm OK critic OK forbidden patterns OK git manipulation OK pod FAIL pod coverage POD is missing for 'retrieve_all' OK spelling OK valid FAIL C4/Creators/Template.pm OK critic OK forbidden patterns OK git manipulation OK pod FAIL pod coverage POD is missing for get_next_label_pos POD is missing for retrieve_all OK spelling OK valid FAIL koha-tmpl/intranet-tmpl/prog/en/modules/acqui/acqui-home.tt OK forbidden patterns OK git manipulation FAIL spelling seperate ==> separate OK tt_valid OK valid_template Other files came back all green. Code review: didn't spot anything. Testing: When I try to access Tools > Label Creator I get an "internal server error". In the plack logs this translates to: Variable "$layout" is not available at /home/vagrant/kohaclone/labels/label-home.pl line 56. Can't call method "get_attr" on an undefined value at /home/vagrant/kohaclone/labels/label-home.pl line 56. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #64951|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 --- Comment #17 from Alex Buckley <alexbuckley@catalyst.net.nz> --- Created attachment 67663 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67663&action=edit Bug 18060: Followup solving $layout error -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18060 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|alexbuckley@catalyst.net.nz |koha-bugs@lists.koha-commun | |ity.org -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org