-
-
Notifications
You must be signed in to change notification settings - Fork 825
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[dev/core#1879] .ical files not populating correctly for sites with ACL's configured for events #17852
Conversation
(Standard links)
|
Can you give a little more info on the details for setting up the ACL because I tried the patch and then logged in as a user in the group the ACL applied to, and then tried to download the ical and it was still mostly empty. Does it depend on specific ACL settings? |
test this please |
I'd feel better with a test..... |
With this ACL (Everyone can edit the fall fundraiser): When logged in as an administrator with permissions to edit/view all events, ical files for events that do not have an ACL look like: even though this test event has a start and end date which should appear in the ical file. That said, I believe any ACL for an event should work. @eileenmcnaughton I will work on a test. |
@alifrumin I couldn't replicate this - but you might be able to build on #18038 |
Test - attempt to replicate #17852
Thank you @eileenmcnaughton!!! |
Closing in favor of Eileens fix. |
Overview
see https://lab.civicrm.org/dev/core/-/issues/1879