Skip to content
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

Allow only events with public visibility for Guest User #2253

Merged
merged 1 commit into from
Sep 4, 2024

Conversation

Rup-Narayan-Rajbanshi
Copy link
Collaborator

@Rup-Narayan-Rajbanshi Rup-Narayan-Rajbanshi commented Sep 3, 2024

Addresses

Changes

  • Allow only events with public visibility for Guest User

Checklist

Things that should succeed before merging.

  • Updated/ran unit tests
  • Updated CHANGELOG.md

Release

If there is a version update, make sure to tag the repository with the latest version.

@Rup-Narayan-Rajbanshi Rup-Narayan-Rajbanshi force-pushed the fix/user-guest-permission-event branch from 167a23d to 4f5ef4e Compare September 4, 2024 05:20
@samshara samshara requested a review from thenav56 September 4, 2024 05:30
@samshara samshara requested review from thenav56 and removed request for thenav56 September 4, 2024 05:31
@samshara samshara self-assigned this Sep 4, 2024
@samshara samshara marked this pull request as ready for review September 4, 2024 05:31
@samshara samshara changed the title Fix getting event with visibility=RCRC for guest user. Allow only events with public visibility for Guest User Sep 4, 2024
@samshara samshara assigned szabozoltan69 and unassigned thenav56 and samshara Sep 4, 2024
@szabozoltan69 szabozoltan69 merged commit 063eea3 into develop Sep 4, 2024
2 checks passed
@szabozoltan69 szabozoltan69 deleted the fix/user-guest-permission-event branch September 4, 2024 08:11
@szabozoltan69
Copy link
Contributor

Deployed to Staging

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants