Skip to content

Fix alipay x sjs import order - #6093

Merged
chouchouji merged 2 commits into
uni-app-vue3-devfrom
fix-alipay-x-sjs-import-order
Oct 29, 2026
Merged

chouchouji merged 2 commits into
uni-app-vue3-devfrom
fix-alipay-x-sjs-import-order

Conversation

@chouchouji

Copy link
Copy Markdown
Member

No description provided.

Copilot AI balanced review requested due to automatic review settings October 28, 2026 06:26

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔵 Needs a closer look

The Alipay X SJS placement fix relies on runtime behavior that static review and unit tests cannot confirm, so it needs a device or IDE check before approval.

No open issues 🎉

Review effort: Lite

@github-actions

Copy link
Copy Markdown

Size report

Path Size
packages/size-check/dist/size-check.es.js 58.53 KB (0%)
packages/size-check/dist/style.css 2.97 KB (0%)
packages/uni-app/dist/uni-app.es.js 2.41 KB (0%)
packages/uni-h5-vue/dist/vue.runtime.esm.js 76.48 KB (0%)
packages/uni-mp-vue/dist/vue.runtime.esm.js 44.15 KB (0%)
packages/uni-mp-alipay/dist/uni.api.esm.js 16.37 KB (0%)
packages/uni-mp-alipay/dist/uni.mp.esm.js 9.47 KB (0%)
packages/uni-mp-baidu/dist/uni.api.esm.js 12.16 KB (0%)
packages/uni-mp-baidu/dist/uni.mp.esm.js 10.37 KB (0%)
packages/uni-mp-qq/dist/uni.api.esm.js 11.49 KB (0%)
packages/uni-mp-qq/dist/uni.mp.esm.js 8.52 KB (0%)
packages/uni-mp-toutiao/dist/uni.api.esm.js 11.59 KB (0%)
packages/uni-mp-toutiao/dist/uni.mp.esm.js 10.05 KB (0%)
packages/uni-mp-weixin/dist/uni.api.esm.js 12.64 KB (0%)
packages/uni-mp-weixin/dist/uni.mp.esm.js 9.08 KB (0%)
packages/uni-quickapp-webview/dist/uni.api.esm.js 11.45 KB (0%)
packages/uni-quickapp-webview/dist/uni.mp.esm.js 8.81 KB (0%)

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

No unresolved issues were found in the review evidence, and the described changes are limited to reordering Alipay X template import declarations along with the two matching test expectations.

No open issues 🎉

Review effort: Lite

@chouchouji
chouchouji force-pushed the fix-alipay-x-sjs-import-order branch from fea1fb1 to c6265f3 Compare October 29, 2026 03:13
@chouchouji
chouchouji merged commit b01e3be into uni-app-vue3-dev Oct 29, 2026
3 of 4 checks passed
@chouchouji
chouchouji deleted the fix-alipay-x-sjs-import-order branch October 29, 2026 06:20
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.

2 participants