fix: support for multi-properties

[daveterra]

WIP: Add support for multi-properties in calendar parsing.

• Updated InnerComponent multi_property from Vec to BTreeMap<String, Vec>
• When converting into InnerComponent, split the properties by those that "May occur more than once" and all others
• Added a basic test as proof of concept
• Still to do is add filtering for other properties such as attach, cattegory, etc.

[daveterra]

Here is the basic outline, let me know what needs to change or any other feedback. I'm still pretty new to Rust so I've very likely missed something, especially around lifetimes.

[hoodie]

@daveterra I'll take this over if you don't mind

[daveterra]

@daveterra I'll take this over if you don't mind

Sounds good! Let me know if there is anything else I can help with.

[hoodie]

thank you so much for finding this and helping out with the fix!!!

[hoodie]

https://crates.io/crates/icalendar/0.15.9

[qwandor]

This should be a semver-breaking change, as it changes the signature of the public Component::multi_properties method.

[hoodie]

Oh no you're right. I had run semver checks but it didn't report that. I'll have that version.

[hoodie]

@qwandor thanks for ringing the alarm, it's 16.0.0 now 👏

[qwandor]

I assume you mean 0.16.0? Thanks!

[hoodie]

of course