ical.js
ical.js copied to clipboard
Refactor ical, relocating parsing code into new module suitable for non-Node.js environments
Hi, thanks for the helpful project. I'm having an issue using this in a non-Node.js environment (Google Apps scripting), and there's an easy fix, because this project's i/o features are the only issues:
- Move all ical parsing code (
parseICS
) into a new npm module@peterbraden/ical-parser
(scope required because unscoped packageical-parser
already exists). - Refactor
ical
to depend on@peterbraden/ical-parser
for the parsing logic, but retain thefromURL
andparseFile
methods for backward compatibility.
If needed, I have already separated the parsing logic myself, and can give it to you in order for you to create @peterbraden/ical-parser
. Just let me know if you want it.
NB: Looks like https://github.com/peterbraden/ical.js/issues/80 is related to this.
Can I suggest an alternate approach? We could drop fromUrl and parseFile - they never should have been part of this module, as they are out of scope.
I'd be happy to release a major version removing these functions.
That would work, for sure, I just thought you'd want to preserve backward compatibility. Your project, do what you want, but just make sure to rev the major version to remain semver compliant. Thanks!
Did this ever get resolved?