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

Chore/upgrade angular v18 #573

Merged
merged 13 commits into from
Jul 31, 2024
Merged

Chore/upgrade angular v18 #573

merged 13 commits into from
Jul 31, 2024

Conversation

yangxiaolang
Copy link
Collaborator

@yangxiaolang yangxiaolang commented Jul 8, 2024

升级更改

  1. Angular 升级到 v18
  2. Storybook 升级到 ^8
  3. Angular CDK 升级到 v18
  4. Typescript 升级到 >= 5.4
  5. Eslint 升级到 v9

升级遇到的问题

  1. Typescript 无法升级到 >=5.2 的版本, 升级之后不能正常解析 extends
    相关问题: Error when using ts-node with tsonfig/bases and typescript 5.3.2 TypeStrong/ts-node#2088
    处理方法: 升级 ts-node 到 v10.9.2

  2. zonejs 升级到 v0.14.x 之后不再支持深度导入, 而 storybook 的一部分是使用的 zone.js/dist/zone 导入
    处理方法: 升级 storybook 到 ^7.5.0, 不升级到 v8 是因为对 stories 处理的变更较大. 需要大量修改 mdx

  3. aluada-fe 升级后及一些其他的新建 Angular 应用使用 aui 时 aui-table 不能正常渲染行

    这是先升级 aui 的原因, 但其实他是 cdk 的问题

    问题原因: alauda-fe 升级之后 aui-table 不能正常渲染可能是因为这个 fix(cdk/table): error if outlets are assigned too early angular/components#28551 解决的问题. 这个 PR 为 CdkTable 引入了 ContentInit 的生命周期
    而新建 Angular 应用则是因为安装的 @angular/cdk 较新, 使用的 AuiTable 的 ngAfterContentInit 没有调用 super.ngAfterContentinit 导致没有渲染行
    解决方式: aui 升级 @angular/cdkv18 , 并做升级更改, 之后 alauda-fe 也要升级 @angular/cdk

  4. eslint@9 变更很大,配置文件需要使用 flat config, 先暂时使用一个临时的配置文件

Copy link

changeset-bot bot commented Jul 8, 2024

🦋 Changeset detected

Latest commit: e408c02

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@alauda/ui Major

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@fengtianze
Copy link
Collaborator

其他依赖也检查一下有没有新版本。
mdx需要怎么改,很麻烦?

@yangxiaolang
Copy link
Collaborator Author

yangxiaolang commented Jul 9, 2024

@fengtianze

其他依赖也检查一下有没有新版本

好, 我把 fe 处理好就看下

mdx需要怎么改,很麻烦?

嗯, 升级 v8 之后现在的就都不能用了, 要转换成 csf , 然后所有的文案就都没有了,只有组件演示 ; 看样子是要变成 mdx1 + csf 的形式, 这个还是单独升吧

@fengtianze fengtianze merged commit a9fe415 into master Jul 31, 2024
8 checks passed
@fengtianze fengtianze deleted the chore/upgrade-angular-v18 branch July 31, 2024 06:43
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