| 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 | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Crowdfunding goal: | 0 | 
| Patch complexity: | Trivial patch | Documentation contact: | |
| Documentation submission: | Text to go in the release notes: | ||
| Version(s) released in: | Circulation function: | ||
| 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. |