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

Closed
juliusknorr opened this issue Apr 14, 2023 · 0 comments · Fixed by #6427
Closed

Migrate to files:node:updated #4054

juliusknorr opened this issue Apr 14, 2023 · 0 comments · 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
@github-project-automation github-project-automation bot moved this from 👀 In review to ☑️ Done in 📝 Office team Dec 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants