Add 'default_status' and 'allowed_statuses' #7
Conversation
Or you could wait to see if 'pelican' fixes their default status code. Without a fix here or there this plugin won't work. getpelican/pelican#2558
|
Would love to see this merged :) |
Except Pelican. Pelican v4 requires an update to pelican-comment-system. See Scheirle/pelican_comment_system#8 & Scheirle/pelican_comment_system#7
|
@MinchinWeb: If you want, you could submit a PR to address the issue upstream in Pelican itself, as I noted in a comment last month. |
Fixes getpelican#2558. Fixes issues encountered by comment plugins among others c.f. [1](bstpierre/pelican-comments#4), [2](Scheirle/pelican_comment_system#8), [3](Scheirle/pelican_comment_system#7)
|
@justinmayer : Pull Request (pelican#2609) has been created! Also, I cleaned up a few failing-on-Windows tests (pelican#2609). As well, I have two languishing pull requests: pelican#2431 that standardizes how paths in settings are built, and pelican#2428 that adds Windows testing to Travis-CI; can you let me know what's needed to move these last two along? Thanks! @Scheirle : Even with these pull requests submitted to Pelican, I think it would still be worthwhile to merge this one too. |
|
Thanks to @MinchinWeb, the fix for this issue has been merged upstream and has been automatically published as Pelican 4.1.1, courtesy of AutoPub. ✨ |
Or you could wait to see if 'pelican' fixes their default status code. Without a fix here or there this plugin won't work.
getpelican/pelican#2558