Merge lp://qastaging/~sam92/openlp/bug-1695620 into lp://qastaging/openlp
Status: | Merged | ||||||||
---|---|---|---|---|---|---|---|---|---|
Merged at revision: | 2854 | ||||||||
Proposed branch: | lp://qastaging/~sam92/openlp/bug-1695620 | ||||||||
Merge into: | lp://qastaging/openlp | ||||||||
Diff against target: |
724 lines (+284/-155) 6 files modified
openlp/core/lib/serviceitem.py (+5/-3) openlp/core/ui/printserviceform.py (+5/-5) openlp/plugins/songs/lib/mediaitem.py (+40/-36) openlp/plugins/songs/lib/songstab.py (+72/-37) openlp/plugins/songs/songsplugin.py (+55/-5) tests/functional/openlp_plugins/songs/test_mediaitem.py (+107/-69) |
||||||||
To merge this branch: | bzr merge lp://qastaging/~sam92/openlp/bug-1695620 | ||||||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Tim Bentley | Pending | ||
Raoul Snyman | Pending | ||
Tomas Groth | Pending | ||
Review via email:
|
This proposal supersedes a proposal from 2019-02-21.
Commit message
Make footer configurable
Description of the change
Make footer configurable
- Make use of Mako for footer generation, configurable in song settings
- removed now obsolete and via template better configurable options to display "songbook", "written by" and "copyright" information in footer
- added explanation box for so far used settings as Mako placeholders
- added songs configuration setting for template including reset button
- added default template replacing currently existing configuration as best as possible (should be backwards compatible or at least be adaptable to correspond to former settings)
- write and adapt tests for new and removed functionality
- Added some more available fields in the footer: Alternate Title, CCLI Number, Topics, Authors (all music, all words)
You need to run CI on this to show the success of tests.
Not sure why but this request did a full download when I did a check out. Also cannot run due to chord issues.
Cannot run this but there seem to be lots of line breaks in the footer display. As the footer area is small this will not be displayed and lead to forum comments.
You need some validation do defend against too long footers!