Closed mengwong closed 8 years ago
I can do this. Looks like it's just founderagreement.xml that's affected given you had grep '?>%' ALL?
I forgot to say a few other things in the original issue description …
Since this is something that affects the production service, this work should be committed to a development branch.
Issues #82 and #94 go together, in the sense that if we change one without changing the other, we will get output that has either no % signs or two %% signs everywhere, lol.
To add complication, there are two repositories. So, in the google-app repository there is now a branch called currencyformats
. In the legalese-io.github.io repository I have just created a branch called currencyformats
.
If you do your work in those branches, and test them using a development environment that is separate from the production environment for both the JS google-app code and the XML templates, then it should be possible to resolve both 82 and 94 at the same time and push both sets of changes in both branches to the master branch with confidence.
The second thing I forgot to include in the original issue description was the full output of the grep. I will fix that now.
At present, templates have
<?= data.percentage ?>%
the getDisplayValues trick described in issue #82 would have the % character as part of data.percentage.
So, go through every template and remove a % as part of fixing #82.