Skip to content

fix: retain query string when resolveId#4702

Merged
antfu merged 1 commit intounocss:mainfrom
songpola:fix-retain-query-string
Jun 5, 2025
Merged

fix: retain query string when resolveId#4702
antfu merged 1 commit intounocss:mainfrom
songpola:fix-retain-query-string

Conversation

@songpola
Copy link
Contributor

@songpola songpola commented Jun 4, 2025

Close #4137

Related:
#2570
#2579
#2533

Apparently, the VIRTUAL_ENTRY_ALIAS regex has already captured the query string but it was not retained in the resolved ID result. This PR just add it back.

@songpola songpola requested review from antfu and zyyv as code owners June 4, 2025 02:44
@netlify
Copy link

netlify bot commented Jun 4, 2025

Deploy Preview for unocss ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit cf7d608
🔍 Latest deploy log https://app.netlify.com/projects/unocss/deploys/683fb2fea332780008c17c79
😎 Deploy Preview https://deploy-preview-4702--unocss.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Jun 5, 2025

Open in StackBlitz

commit: cf7d608

@antfu antfu added this pull request to the merge queue Jun 5, 2025
Merged via the queue into unocss:main with commit a2acc4c Jun 5, 2025
11 checks passed
@songpola songpola deleted the fix-retain-query-string branch June 5, 2025 11:01
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.

Mode that allows importing all styles per project, inlined

2 participants