Skip to content

WebUI: remove "lodash" dep#6369

Merged
6543 merged 4 commits into
woodpecker-ci:mainfrom
6543-forks:rm-lodash
Apr 4, 2026
Merged

WebUI: remove "lodash" dep#6369
6543 merged 4 commits into
woodpecker-ci:mainfrom
6543-forks:rm-lodash

Conversation

@6543

@6543 6543 commented Apr 2, 2026

Copy link
Copy Markdown
Member

make our webui smaler by 0.1Mb ... and remove dep that is haunted by security issues and almost replaced by modern browser APIs.

https://developer.mozilla.org/de/docs/Web/API/Window/structuredClone#browser-kompatibilit%C3%A4t

@6543 6543 added ui frontend related dependencies labels Apr 2, 2026
@6543 6543 requested a review from qwerty287 April 2, 2026 15:02
@6543 6543 enabled auto-merge (squash) April 4, 2026 12:14
@6543 6543 merged commit de4f9c2 into woodpecker-ci:main Apr 4, 2026
6 of 7 checks passed
@woodpecker-bot woodpecker-bot mentioned this pull request Apr 4, 2026
1 task
@6543 6543 deleted the rm-lodash branch April 4, 2026 13:14
@6543 6543 added the security label Apr 4, 2026
@qwerty287

Copy link
Copy Markdown
Contributor

It seems this broke editing secrets and registries.

Clicking the edit button fails with DOMException: Proxy object could not be cloned. (for example in AdminRegistries.vue, line 100 for the admin registries)

@6543 6543 mentioned this pull request Apr 8, 2026
@6543

6543 commented Apr 8, 2026

Copy link
Copy Markdown
Member Author

-> #6409

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants