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(theming): remove node-vibrant dependency and usage (was moved to backend) #50320

Merged
merged 2 commits into from
Jan 25, 2025

Conversation

susnux
Copy link
Contributor

@susnux susnux commented Jan 22, 2025

The color extraction is handled by the backend nowadays, the color this calculated was not used by the backend anymore.
Reduce dependencies + compiled size + security warning on that dependency.

Checklist

@susnux susnux added this to the Nextcloud 31 milestone Jan 22, 2025
@susnux susnux requested review from skjnldsv, a team, artonge and Pytal and removed request for a team January 22, 2025 16:54
@Altahrim Altahrim mentioned this pull request Jan 23, 2025
…backend)

The color extraction is handled by the backend nowadays, this was not
used by the backend anymore.
Reduce depdencies + compiled size + security warning on that dependency.

Signed-off-by: Ferdinand Thiessen <[email protected]>
@AndyScherzinger
Copy link
Member

/compile /

Signed-off-by: nextcloud-command <[email protected]>
@AndyScherzinger AndyScherzinger merged commit b119743 into master Jan 25, 2025
120 checks passed
@AndyScherzinger AndyScherzinger deleted the fix/node-vibrant branch January 25, 2025 17:09
@AndyScherzinger
Copy link
Member

/backport to stable31

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.

4 participants