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

Adding of an attribute "Public" or "Private" for each event on the calendar. #1214

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

BrienO17
Copy link

#1019

Hello,

My name is Rafik, I am an engineering student. With my comrade Milo we worked on the adding of an attribute "Public" or "Private" for each event on the calendar. For now you can specify it with a character string when you press "n" to create an event and the latter is displayed in the ICS file. For the future we would like to do it with a clickable button with an "x" displayed in the middle of the button that is clicked and which automatically adds the attribute "Public" or "Private" to the ICS file. We also want to limit the access to the private events. Therefore only a list of people added by the creator of the event could access to it.

If you have any question about our work do not hesitate to write to us !

Rafik.

@@ -493,7 +499,7 @@ def delete_instance(vevent: icalendar.Event, instance: dt.datetime) -> None:
vevent.pop('EXDATE')
vevent.add('EXDATE', exdates)
if 'RDATE' in vevent:
rdates = [one for one in _get_all_properties(vevent, 'RDATE') if one != instance]
rdates = [one for one in _get_allz_properties(vevent, 'RDATE') if one != instance]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo here?

``None``, then ``event.calendar`` must be defined.

:param event: the event to be inserted.
def insert(self, event: Event, collection: Optional[str]=None) -> Event:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This function does not return anything.

I think you started of an old version of main which had typing wrong, but then mis-merged somehow? I do recall this being broken in the past.

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.

2 participants