Summary: | Remove some obsolete references to Greybox in some templates | ||
---|---|---|---|
Product: | Koha | Reporter: | Owen Leonard <oleonard> |
Component: | Templates | Assignee: | Owen Leonard <oleonard> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | minor | ||
Priority: | P5 - low | CC: | f.demians, hector.hecaxmmx, kyle |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Bug Depends on: | |||
Bug Blocks: | 8290, 16811, 16812 | ||
Attachments: |
Bug 16600 - Remove some obsolete references to Greybox in some templates
[SIGNED-OFF]Bug 16600: Remove some obsolete references to Greybox in some templates Bug 16600 - Remove some obsolete references to Greybox in some templates |
Description
Owen Leonard
2016-05-26 13:32:40 UTC
Created attachment 51839 [details] [review] Bug 16600 - Remove some obsolete references to Greybox in some templates This patch removes unused Greybox markup from two templates. To test, apply the patch and go to Authorities. - Perform an authority search which will return multiple results. - Select two records to merge. - On the "merging records" page, an earlier version used Greybox to show a preview of the selected authority records. Confirm that authority number link for the two records triggers a pop-up window instead. - From the authorities home page, choose 'New from Z39.50' - Perform a search which will return multiple results. - Confirm that there is only a 'MARC' preview link. A forever-commented-out 'Card' link has been removed. Created attachment 52867 [details] [review] [SIGNED-OFF]Bug 16600: Remove some obsolete references to Greybox in some templates This patch removes unused Greybox markup from two templates. To test, apply the patch and go to Authorities. - Perform an authority search which will return multiple results. - Select two records to merge. - On the "merging records" page, an earlier version used Greybox to show a preview of the selected authority records. Confirm that authority number link for the two records triggers a pop-up window instead. - From the authorities home page, choose 'New from Z39.50' - Perform a search which will return multiple results. - Confirm that there is only a 'MARC' preview link. A forever-commented-out 'Card' link has been removed. Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com> Works as advertised. Obsolete comment on 'Card' removed Created attachment 52875 [details] [review] Bug 16600 - Remove some obsolete references to Greybox in some templates This patch removes unused Greybox markup from two templates. To test, apply the patch and go to Authorities. - Perform an authority search which will return multiple results. - Select two records to merge. - On the "merging records" page, an earlier version used Greybox to show a preview of the selected authority records. Confirm that authority number link for the two records triggers a pop-up window instead. - From the authorities home page, choose 'New from Z39.50' - Perform a search which will return multiple results. - Confirm that there is only a 'MARC' preview link. A forever-commented-out 'Card' link has been removed. Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com> Works as advertised. Obsolete comment on 'Card' removed Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Pushed to master for 16.11, thanks Owen! Pushed in 16.05. Will be in 16.05.02. |