vue-responsive-calendar icon indicating copy to clipboard operation
vue-responsive-calendar copied to clipboard

Month view

Open erhansaydam opened this issue 7 years ago • 0 comments

Hi again,

It would be very nice if you can add the month view option.

Best regards.

erhansaydam avatar Nov 28 '17 08:11 erhansaydam

I fixed the problem by explicitely putting an $object->save() in my 'Add' callable before attaching the members. I hope this is a safe way to deal with it.

While debugging, I also found that a group create passes all members to the Add callable, while the update passes the members one by one. This makes the callable a bit complex because it needs to analyse the incoming $value structure to decide how to proceed. I think it s probably better to just pass the value array in the update operation as well, and remove the foreach loop that is currently there in https://github.com/arietimmerman/laravel-scim-server/blob/master/src/Http/Controllers/ResourceController.php#L265.

Any thoughts on this?

pieterdt avatar Jun 04 '23 23:06 pieterdt

With the latest release, the code you are talking about has been rewritten completely. I believe this also solves the issues you've found.

arietimmerman avatar Jul 05 '24 18:07 arietimmerman