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

[stable26] Fix npm audit #766

Merged
merged 6 commits into from
Jul 12, 2023
Merged

Conversation

nextcloud-command
Copy link
Contributor

Auto-generated fix of npm audit

@nextcloud-command nextcloud-command added 3. to review dependencies Pull requests that update a dependency file labels Jun 25, 2023
@nextcloud-command nextcloud-command force-pushed the automated/noid/stable26-fix-npm-audit branch from 1e394f3 to fe315ee Compare July 2, 2023 03:29
@nextcloud-command nextcloud-command force-pushed the automated/noid/stable26-fix-npm-audit branch from fe315ee to 32e5439 Compare July 9, 2023 03:54
@szaimen szaimen added this to the Nextcloud 26.0.4 milestone Jul 10, 2023
szaimen
szaimen previously approved these changes Jul 10, 2023
Signed-off-by: GitHub <noreply@github.com>
@nextcloud-command nextcloud-command force-pushed the automated/noid/stable26-fix-npm-audit branch from 32e5439 to cfdb2c5 Compare July 10, 2023 14:19
@szaimen szaimen requested a review from danxuliu July 10, 2023 14:20
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
Since PDF.js was bumped to 2.5.207 in 4705de4 the event listened to
is "DOMContentLoaded" rather than "webviewerloaded".

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
@danxuliu
Copy link
Member

/compile /

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
Copy link
Collaborator

@szaimen szaimen left a comment

Choose a reason for hiding this comment

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

LGTM but didnt test

Copy link
Member

@danxuliu danxuliu left a comment

Choose a reason for hiding this comment

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

(Smoke) Tested and works 👍

@danxuliu danxuliu merged commit cdc7dc8 into stable26 Jul 12, 2023
17 checks passed
@delete-merged-branch delete-merged-branch bot deleted the automated/noid/stable26-fix-npm-audit branch July 12, 2023 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants