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

Revert "[stable28] fix/remote activity constructor" #1842

Merged
merged 1 commit into from
Nov 18, 2024

Conversation

juliusknorr
Copy link
Member

Reverts #1813 as the code in 28 does not have the original issue.

The bug was only introduced in c8248b0 for 29 and later

Copy link

cypress bot commented Nov 15, 2024

Activity    Run #2100

Run Properties:  status check passed Passed #2100  •  git commit bf49b6c7a2: Revert "[stable28] fix/remote activity constructor"
Project Activity
Branch Review revert-1813-backport/1811/stable28
Run status status check passed Passed #2100
Run duration 01m 43s
Commit git commit bf49b6c7a2: Revert "[stable28] fix/remote activity constructor"
Committer Julius Knorr
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 3
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 7
View all changes introduced in this branch ↗︎

@juliusknorr juliusknorr force-pushed the revert-1813-backport/1811/stable28 branch from fae456b to 14607f7 Compare November 15, 2024 13:09
@juliusknorr juliusknorr force-pushed the revert-1813-backport/1811/stable28 branch from 14607f7 to 4c79562 Compare November 15, 2024 13:18
@juliusknorr
Copy link
Member Author

php-cs also fails on other prs, seems unrelated (#1705)

@juliusknorr juliusknorr merged commit 981c0be into stable28 Nov 18, 2024
48 of 49 checks passed
@juliusknorr juliusknorr deleted the revert-1813-backport/1811/stable28 branch November 18, 2024 10:15
@joshtrichards joshtrichards added this to the Nextcloud 28.0.13 milestone Nov 24, 2024
@skjnldsv skjnldsv mentioned this pull request Nov 27, 2024
4 tasks
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.

3 participants