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

Fix event name files:node:rename #3797

Closed
wants to merge 1 commit into from
Closed

Conversation

artonge
Copy link
Contributor

@artonge artonge commented Jun 24, 2024

From the files action, it does not have a `d`.
Or maybe it should and the file action is wrong?

Other reference: https://github.com/nextcloud/text/blob/master/src/views/RichWorkspace.vue#L105

Signed-off-by: Louis <louis@chmn.me>
@artonge artonge added bug Something isn't working 4. to release Ready to release, can be waiting for the tests to finish technical debt javascript Pull requests that update Javascript code labels Jun 24, 2024
@artonge artonge self-assigned this Jun 24, 2024
@artonge
Copy link
Contributor Author

artonge commented Jun 26, 2024

Response from @skjnldsv, this is two different events. The one without the d is to trigger the rename.

@artonge artonge closed this Jun 26, 2024
@solracsf solracsf deleted the artonge/fix/rename_event_name branch June 26, 2024 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to release, can be waiting for the tests to finish bug Something isn't working javascript Pull requests that update Javascript code technical debt
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant