Skip to content
This repository has been archived by the owner on May 28, 2024. It is now read-only.

Fix removed global function call to escapeHTML #185

Merged
merged 4 commits into from
Sep 15, 2020

Conversation

MorrisJobke
Copy link
Member

nextcloud/server#19357

  • share a video by public link
  • open link
  • expected: a video player is shown
  • actual: no video player is shown and an error about missing escapeHTML is printed

@skjnldsv @ChristophWurst Mind to compile this and add a commit with the new assets here, because I'm currently fixing my local npm environment. At least this should fix it. 🙈

nextcloud/server#19357

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
@skjnldsv

This comment has been minimized.

@MorrisJobke
Copy link
Member Author

@skjnldsv @ChristophWurst Mind to compile this and add a commit with the new assets here, because I'm currently fixing my local npm environment. At least this should fix it. 🙈

Managed to compile it now :)

Works locally.

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
@MorrisJobke MorrisJobke force-pushed the fix/noid/20-escape-html branch from ff97493 to 5d9d918 Compare September 15, 2020 08:15
@MorrisJobke
Copy link
Member Author

Managed to compile it now :)

Seems to build slightly different on my machine and here. Let's build it then here.

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
@MorrisJobke MorrisJobke force-pushed the fix/noid/20-escape-html branch from 93fd4d3 to 8129eef Compare September 15, 2020 09:42
@rullzer

This comment has been minimized.

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants