bstascavage / plexReport

Scripts to generate a weekly email of new additions to Plex
67 stars 17 forks source link

Ability to have a space in subject line #15

Closed xomnia closed 9 years ago

xomnia commented 9 years ago

I would like the ability to put a space after my custom subject line before the date. It currently shows up as "New This Week:12/07/2014". I tried adding single quotes but it breaks the scripts ability to parse the config file. I also tried   figuring it would maybe read the html in the subject but no luck.

If you could change the way the subject option is parsed to allow single quotes or even just add a space on the end by default that would fix it for me.

bstascavage commented 9 years ago

Um, you can just a space at the end. I do for my emails:

In config.yaml: subject: 'Felannisport Update: Week of '

In the email: Felannisport Update: Week of 12/05/2014

xomnia commented 9 years ago

Yea I tried that and got thrown an error about parsing the yaml. Bug? I'll try it again when I get a chance and submit the log

On Sunday, December 7, 2014, Brian notifications@github.com wrote:

Um, you can just a space at the end. I do for my emails:

In config.yaml: subject: 'Felannisport Update: Week of '

In the email: Felannisport Update: Week of 12/05/2014

— Reply to this email directly or view it on GitHub https://github.com/bstascavage/plexReport/issues/15#issuecomment-65947092 .

xomnia commented 9 years ago

Nevermind, my apologies, single quotes are working fine.

bstascavage commented 9 years ago

I am going to reopen this. It doesn't make sense for the user to have to add a space in his/her config. It should be added normally via code.