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

upgrade app to be compatible with Nextcloud 28 (issue #159) #166

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

christian-baumann
Copy link

Upgrade of the app to the new event API to make it work with Nextcloud 28.

https://help.nextcloud.com/t/symfony-event-dispatcher-update-in-nextcloud-server-28-breaking-changes/167550

Signed-off-by: Christian Baumann <christianbaumann@gmx.at>
@Sblop
Copy link

Sblop commented Feb 20, 2024

Please approve this modification.
@ArtificialOwl @artificial-owl.com
Can you assist?

@devnoname120
Copy link

@susnux Would you mind ✅'ing this PR? Thanks

@@ -24,7 +24,7 @@
<screenshot>https://raw.githubusercontent.com/nextcloud/files_downloadactivity/master/docs/screenshot.png</screenshot>

<dependencies>
<nextcloud min-version="24" max-version="26" />
<nextcloud min-version="26" max-version="28" />
Copy link
Member

Choose a reason for hiding this comment

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

Should add 29 then as well?

@si458 si458 mentioned this pull request Jun 14, 2024
@devnoname120
Copy link

@nickvergessen Any chance this could be merged? This is spamming my logs.

@nickvergessen
Copy link
Member

nickvergessen commented Jul 17, 2024

If anyone is interested in taking over maintainership we can do so.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants