You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Thanks for the bug report. You seem comfortable making code changes... do you want to tackle this one?
If I had to guess, I would guess it's an easy-ish fix. Probably something like calling a ToString on an enum's name instead of using a const string inside a TransparencyType class. (Admittedly, I haven't looked.)
happy to - looking at the AttendeeSerializer class in its current form, the staus is not currently being serialized. I believe you might be working on this as per issue #45. ideally I can uncomment the tests in the deserialize test groups checking that these are failing, fix and rerun, but given the current problems with serialization it seems worth postponing issues to do with letter case until serialization of the properties is occurring.
Finding my way to the attendeeSerializer class, I saw a handful of simple/minor optimisations which are included in a pull request.
First - huge thank you and kudos for getting this project under active development once again!
Setting the properties for an event instance, for example as below:
myEvt.Transparency = TransparencyType.Opaque;
myEvt.Status = EventStatus.Confirmed;
results in a serialised string with the following text:
STATUS:Confirmed
TRANSP:Opaque
this does not validate, as the string should be
STATUS:CONFIRMED
TRANSP:OPAQUE
The text was updated successfully, but these errors were encountered: