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

Migrate to files:node:updated #4054

Open
juliusknorr opened this issue Apr 14, 2023 · 0 comments · May be fixed by #6427
Open

Migrate to files:node:updated #4054

juliusknorr opened this issue Apr 14, 2023 · 0 comments · May be fixed by #6427

Comments

@juliusknorr
Copy link
Member

After nextcloud/server#37726 we need to adjust the event that a file has been updated, however we should actually make sure to send a full Node object there not just the fileid.

Note: For the other apps that already started using the old files:file:updated syntax, we should migrate.
As those apps are shipped and have stablexx branches, I think it's safe to change it for master in both server and those apps. Only Notes would be an issue:

nextcloud/notes@f635c5c/src/components/NoteRich.vue#L63 @juliushaertl
nextcloud/text@01ccc7c/src/components/Editor.vue#L637 @juliushaertl

@github-project-automation github-project-automation bot moved this to 🧭 Planning evaluation (don't pick) in 📝 Office team Jun 19, 2023
@blizzz blizzz moved this from 🧭 Planning evaluation (don't pick) to 📄 To do (~10 entries) in 📝 Office team Jun 19, 2023
@luka-nextcloud luka-nextcloud self-assigned this Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 👀 In review
Development

Successfully merging a pull request may close this issue.

4 participants