Thanks for publishing the updated script!
Looking back at your earlier notes in #25, I see two mismatches that are now solved in your version:
- I mentioned that
ICS_NAMESPACE
wasn’t supported — but the new script does include it, with a clear priority order (--namespace
→ICS_NAMESPACE
→ derived from feed URL/filename). - I also had
DEFAULT_TAGS
in my.env
, but the script actually expectsDISCOURSE_DEFAULT_TAGS
. That’s an easy fix in the environment file.
The other environment/cron steps are unchanged and still work fine; just remember to pass the feed as --ics-url "$ICS_SOURCE"
(or --ics-file
), since the script doesn’t implicitly read ICS_SOURCE
by itself.
All in all, these are small adjustments, and with them the instructions line up perfectly with the new code.