-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
fix(files): standardize files:node:event
event syntax
#37726
Conversation
Signed-off-by: John Molakvoæ <[email protected]>
For notes we can add a version check then |
Needed after: nextcloud/server#37726 Signed-off-by: Louis Chemineau <[email protected]>
Needed after: nextcloud/server#37726 Signed-off-by: Louis Chemineau <[email protected]> Signed-off-by: nextcloud-command <[email protected]>
Needed after: nextcloud/server#37726 Signed-off-by: Louis Chemineau <[email protected]> Signed-off-by: nextcloud-command <[email protected]>
Hello! Are you familiar with our documentation process already? Changes that affect administrators should be documented here: While I understand sometimes it's handy to merge fast to get your changes in, it would be great if next time you could try to add the documentation before merging. You can find more information on that here: If all your documentation efforts are done, please remove the label 'documentation' and check the checkbox in your opening note and I'll stop bugging you :) Many thanks in advance and thanks again for your work! |
After discussion:
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: