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.

Sign up for GitHub

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

Jump to bottom

Make rooms / resources automatically reply to invites #16615

Merged
merged 1 commit into from Aug 28, 2019
Merged

Make rooms / resources automatically reply to invites #16615

merged 1 commit into from Aug 28, 2019

Conversation

georgehrke
Copy link
Member

Signed-off-by: Georg Ehrke developer@georgehrke.com

@georgehrke georgehrke added the 2. developing Work in progress label Jul 30, 2019
@georgehrke georgehrke changed the title Make rooms / resoureces automatically reply to invites Make rooms / resources automatically reply to invites Jul 30, 2019
@georgehrke georgehrke force-pushed the feature/16518/rooms_resources_should_respond branch 2 times, most recently from 10ecd5c to d810cd0 Compare July 30, 2019 12:28
@rullzer rullzer added this to the Nextcloud 17 milestone Aug 11, 2019
@georgehrke georgehrke force-pushed the feature/16518/rooms_resources_should_respond branch from d810cd0 to cb17057 Compare August 14, 2019 15:10
@rullzer rullzer mentioned this pull request Aug 18, 2019
Merged
@georgehrke georgehrke force-pushed the feature/16518/rooms_resources_should_respond branch 2 times, most recently from c4ec109 to cd0e4a6 Compare August 21, 2019 15:17
@georgehrke georgehrke marked this pull request as ready for review August 21, 2019 15:18
@georgehrke georgehrke added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Aug 21, 2019
@georgehrke georgehrke force-pushed the feature/16518/rooms_resources_should_respond branch from cd0e4a6 to 9f65a91 Compare August 21, 2019 15:20
@georgehrke
Copy link
Member Author

@rullzer This is finally read for a code-review :)

@georgehrke
Copy link
Member Author

The calendar app does not show the response status at the moment:
You have to check the calendar-data of the organiser to check:
8321B77A-3ED5-48CA-B418-5E663B47D335
655B86D1-8B9E-4B6C-AD2B-5F37BA4CF4D4

rullzer
rullzer requested changes Aug 21, 2019
View reviewed changes
Copy link
Member

@rullzer rullzer left a comment

Choose a reason for hiding this comment

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

Looks good overal

apps/dav/lib/CalDAV/Schedule/Plugin.php Show resolved Hide resolved
apps/dav/lib/CalDAV/Schedule/Plugin.php Outdated Show resolved Hide resolved
@rullzer rullzer mentioned this pull request Aug 23, 2019
Merged
@georgehrke georgehrke added 2. developing Work in progress and removed 3. to review Waiting for reviews labels Aug 26, 2019
@georgehrke
Make rooms / resources automatically reply to invites
cc37c39 
Signed-off-by: Georg Ehrke <developer@georgehrke.com>
@georgehrke georgehrke force-pushed the feature/16518/rooms_resources_should_respond branch from 9f65a91 to cc37c39 Compare August 26, 2019 11:28
@rullzer rullzer added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Aug 28, 2019
@rullzer rullzer merged commit e99fa92 into master Aug 28, 2019
@rullzer rullzer deleted the feature/16518/rooms_resources_should_respond branch August 28, 2019 07:34
@georgehrke georgehrke mentioned this pull request Aug 31, 2019
Closed
@ChristophWurst ChristophWurst mentioned this pull request Nov 24, 2021
Open
@ChristophWurst ChristophWurst mentioned this pull request Dec 1, 2021
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rullzer rullzer rullzer approved these changes

@ChristophWurst ChristophWurst ChristophWurst approved these changes

@juliushaertl juliushaertl Awaiting requested review from juliushaertl

Assignees
No one assigned
Labels
3. to review Waiting for reviews
Projects
None yet
Milestone
Nextcloud 17
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@georgehrke @rullzer @ChristophWurst