Skip to content

Conversation

@xrgzs
Copy link
Member

@xrgzs xrgzs commented Jun 12, 2025

  • Updated @solidjs/router to version 0.9.1, which is the last version without changes to Routes, Link, etc.
  • Remove the 'solid-router' git submodule
  • Fixed the issue that 'pathname' in 'useRouter' did not have url decode, and resolved the request error
  • Updated some of the underlying dependencies related to solid
  • Changed the icon for 'remove_empty_directory' to 'CgFolderRemove', the original 'HiOutlineFolderRemove' is deprecated
  • Replace '@motionone/solid' with 'solid-motionone', which is deprecated

@elysia-best
Copy link
Member

If this pr works, #4 is no longer needed.

@xrgzs
Copy link
Member Author

xrgzs commented Jun 12, 2025

This is the same modification of AlistGo/alist-web/pull/278. I've been testing in my fork for over a month and found no serious issues.

Copy link
Member

@elysia-best elysia-best left a comment

Choose a reason for hiding this comment

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

LGTM, it works after changing to new salt.

@cxw620 cxw620 linked an issue Jun 12, 2025 that may be closed by this pull request
Copy link
Member

@jyxjjj jyxjjj left a comment

Choose a reason for hiding this comment

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

因为暂时不需要改盐 我认为可以直接合并 - 同意。

@elysia-best
Copy link
Member

CC. @cxw620 can you squash merge this? And close #4 and #2

@cxw620 cxw620 merged commit 9a8df5e into OpenListTeam:main Jun 13, 2025
@cxw620
Copy link
Member

cxw620 commented Jun 13, 2025

CC. @cxw620 can you squash merge this? And close #4 and #2

Done

@xrgzs xrgzs deleted the remove-submodule branch June 13, 2025 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

前端依赖的submodule也应该fork过来

4 participants