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

[oc] Sync deathdate and anniversary to birthday calendar #1742

Merged
merged 1 commit into from
Oct 18, 2016

Conversation

rullzer
Copy link
Member

@rullzer rullzer commented Oct 14, 2016

* Sync deathdate and anniversary to birthday calendar (which should be renamed maybe)

* Sync deathdate and anniversary to birthday calendar (which should be renamed maybe)

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
@rullzer rullzer added 3. to review Waiting for reviews downstream labels Oct 14, 2016
@rullzer rullzer added this to the Nextcloud 11.0 milestone Oct 14, 2016
@mention-bot
Copy link

@rullzer, thanks for your PR! By analyzing the history of the files in this pull request, we identified @DeepDiver1975, @nickvergessen and @icewind1991 to be potential reviewers.

@tcitworld
Copy link
Member

Not sure the death date of someone is something people want to see in their calendars, just saying.

@MorrisJobke
Copy link
Member

Tested and works 👍

@MorrisJobke
Copy link
Member

Not sure the death date of someone is something people want to see in their calendars, just saying.

You already added it to the contact ... and in the end you could disable this calendar

@tcitworld
Copy link
Member

You already added it to the contact ... and in the end you could disable this calendar

Well, maybe you just got it from somewhere else... and you still enjoy having your contacts birthdays. It was just a remark though.

@LukasReschke
Copy link
Member

👍

@LukasReschke LukasReschke merged commit 0923d26 into master Oct 18, 2016
@LukasReschke LukasReschke deleted the us_sync-date-of-death branch October 18, 2016 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants