Hi @petefoth are there any updates on this? If there is anything that needs to be changed I can take care of it

Hi @petefoth are there any updates on this? If there is anything that needs to be changed I can take care of it

There is another PR (#431) which achieves the same but in a more general way. I was thinking merging that PR rather than this one, because it is more flexible and future-proof. I'd be interested in your comments on that PR before I make a decision

I'll make a decision this evening on which PR to merge, so the merge can happen before the September build run starts

I decided to merge #431 instead, as it has more flexibility

Thanks for your contribution though.

There is another PR (#431) which achieves the same but in a more general way. I was thinking merging that PR rather than this one, because it is more flexible and future-proof. I'd be interested in your comments on that PR before I make a decision

Yup, fine for me, I opened this one as I had the perception that the other was too broad and you were more favorable about the specific functionality.

Thank you!