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: add dayjs for antdv5 #42

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

Jandiasnow
Copy link

No description provided.

@JackLian JackLian requested a review from eternalsky October 9, 2023 03:19
@eternalsky
Copy link
Collaborator

添加其他 package 可以通过插件的配置项来完成,没有必要全部内置。

@Jandiasnow
Copy link
Author

@eternalsky 使用 extraAssets 的方式,实际 dayjs 会在 antd 之后加载,看能怎么解决
image

extraAssets: [
{
packages: [
{
package: '@ant-design/icons',
version: '4.7.0',
urls: [
//g.alicdn.com/code/npm/@ali/ant-design-icons-cdn/4.5.0/index.umd.min.js
],
library: 'icons'
},
{
package: 'dayjs',
urls: [
'https://jsd.onmicrosoft.cn/npm/[email protected]/dayjs.min.js'
],
library: 'dayjs',
version: '1.9.8'
},
{
package: 'antd',
version: '5.8.6',
urls: [
https://jsd.onmicrosoft.cn/npm/[email protected]/dist/antd.min.js
],
library: 'antd'
}
],
}
]

@eternalsky
Copy link
Collaborator

可以使用 builtinAssets

@Jandiasnow
Copy link
Author

可以使用 builtinAssets

builtinAssets 也是加了的,本地调试的时候有问题,打包发布之后是正常的

@eternalsky
Copy link
Collaborator

可以使用 builtinAssets

builtinAssets 也是加了的,本地调试的时候有问题,打包发布之后是正常的

builtAssets 和 ExtraAssets 本身也是有顺序的,如果觉得不满足你的需求,可以在这方面提一些你的 feature request。默认内置不再继续增加。

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