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

Add SabreDav Patch for iTip Broker #1939

Merged
merged 1 commit into from
Sep 15, 2024

Conversation

SebastianKrupinski
Copy link
Contributor

No description provided.

@miaulalala
Copy link
Contributor

Is there an associated upstream PR for this? If so, please link it here :)

@SebastianKrupinski
Copy link
Contributor Author

SebastianKrupinski commented Sep 13, 2024

Is there an associated upstream PR for this? If so, please link it here :)

Yes.

Sabre: sabre-io/dav#1559

And there are several issues at the moment we are can not fix because we can not overwrite the broker. You want me to list some of them?

@ChristophWurst
Copy link
Member

Please open a server PR with this 3rdparty checkout. This is needed for a full CI run.

@SebastianKrupinski
Copy link
Contributor Author

Please open a server PR with this 3rdparty checkout. This is needed for a full CI run.

Done. nextcloud/server#47998

Copy link
Member

@st3iny st3iny left a comment

Choose a reason for hiding this comment

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

Clean

Signed-off-by: SebastianKrupinski <krupinskis05@gmail.com>
@SebastianKrupinski SebastianKrupinski merged commit 1a2aa63 into master Sep 15, 2024
7 checks passed
@SebastianKrupinski SebastianKrupinski deleted the fix/sabre-dav-itip-broker branch September 15, 2024 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ☑️ Done
Development

Successfully merging this pull request may close these issues.

4 participants