builttoroam / device_calendar

A cross platform plugin for modifying calendars on the user's device
https://pub.dev/packages/device_calendar
BSD 3-Clause "New" or "Revised" License
267 stars 267 forks source link

Change attendance status #391

Closed thomassth closed 2 years ago

thomassth commented 2 years ago

Replaces #335.

cherry picked commit 87710e0f0e084624aac8b477b40d9f1bd42790a1 into the latest branch for convenience.

thomassth commented 2 years ago

Although the tests are passing, more tests are needed to make sure the function actually works.

@maksmaxx do you have any experience using your function?

thomassth commented 2 years ago

I've done all I can with an android device. Everything android side are working as intended.

I need someone with an iOS device to complete the rest.

sowens-csd commented 2 years ago

I'll put the iOS work for this next on my list @thomassth.

sowens-csd commented 2 years ago

The example continues to work fine on iOS. I can see attendees. What else should I be looking at?

thomassth commented 2 years ago

According to the commit, by adding a new IOS only view, you should be able to change attendance details, instead of just viewing them?

@maksmaxx any experience making this work?

sowens-csd commented 2 years ago

Okay, I tried out the iOS specific dialog in the example app. A couple of small changes were required to the example app, namely making the Android status nullable and adding a privacy description to the plist. If we want to support this functionality we'll need to change the readme to document this new plist key requirement. The attendee status dialog did work but I think that updating the entry changed the ID of the calendar event which means that the example app has errors after the update until you reload the calendar to get the new event ID. @maksmaxx could you comment on whether this is the behaviour you'd expect?

thomassth commented 2 years ago

I've updated the plist documents.

@sowens-csd was the attendance update successful? (did you see the updated details at other calendar apps?) If it does, we only need to update how the example app works.

sowens-csd commented 2 years ago

So I've just committed an ugly hack to the example that means it doesn't cause an error. It pops back out to the calendar list after editing an iOS entry. With that change the iOS specific behaviour does seem to be working properly.