search

cwrc / Islandora-CWRC-Writer

Islandora wrapper for the CWRC-Writer.
https://github.com/cwrc/CWRC-Writer
GNU General Public License v3.0
0 stars 3 forks source link

Consolidate CWRC-Writer button config #32

Open jefferya opened 6 years ago

jefferya commented 6 years ago

The goal: refactor code to set all CWRC-Writer button config in one location. At present, the config is spread across multiple files:

Message from Andrew - Slack In order to simplify my own testing, I've kept my JS that modifies buttons1 ( https://github.com/cwrc/Islandora-CWRC-Writer/blob/master/js/islandora_cwrc_writer.js#L23-L26 ) but I'm thinking that this should be consolidated at some point

SusanBrown commented 6 years ago

@ajmacdonald Can we have an estimate on this pls?

ajmacdonald commented 6 years ago

@SusanBrown I'm hesitant to guess. I'm not sure it's a good idea for me to tackle this on my own as it involves PHP coding and parts of the Islandora module that I'm not familiar with.

SusanBrown commented 6 years ago

@nikathone Another one we think you could tackle. Consult with @ajmacdonald and @jefferya as needed about specifics of the CWRC-Writer integration module.