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(comments): Extend Vue before calling new on it #46643

Merged
merged 2 commits into from
Jul 25, 2024

Conversation

susnux
Copy link
Contributor

@susnux susnux commented Jul 19, 2024

Summary

#45848 introduced a regression as the Vue was not extended but the Vue component was directly tried to instantiate which does not work as a Vue component is basically an Object that does not has any constructor.

Checklist

Copy link
Contributor

@kesselb kesselb left a comment

Choose a reason for hiding this comment

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

Thanks, it does work again ;)

@susnux susnux force-pushed the fix/comments-vue-constructor branch from 77ce099 to 85ef5ae Compare July 19, 2024 17:06
@susnux
Copy link
Contributor Author

susnux commented Jul 19, 2024

/compile

@susnux susnux force-pushed the fix/comments-vue-constructor branch from b3ac628 to ea05554 Compare July 19, 2024 17:35
@susnux
Copy link
Contributor Author

susnux commented Jul 19, 2024

/compile

@artonge artonge force-pushed the fix/comments-vue-constructor branch 2 times, most recently from 894ef7b to 8330eda Compare July 24, 2024 14:23
@artonge artonge force-pushed the fix/comments-vue-constructor branch from 8330eda to ab5ac1f Compare July 24, 2024 14:23
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
Signed-off-by: Louis Chemineau <louis@chmn.me>
@susnux susnux force-pushed the fix/comments-vue-constructor branch from ab5ac1f to ece4aec Compare July 24, 2024 23:38
@susnux susnux enabled auto-merge July 24, 2024 23:38
@susnux susnux merged commit 37483aa into master Jul 25, 2024
109 checks passed
@susnux susnux deleted the fix/comments-vue-constructor branch July 25, 2024 00:02
@blizzz blizzz mentioned this pull request Jul 25, 2024
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.

[Bug]: File comments broken
4 participants