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: error message #1580

Merged
merged 1 commit into from
Jul 19, 2021
Merged

fix: error message #1580

merged 1 commit into from
Jul 19, 2021

Conversation

ttys026
Copy link
Collaborator

@ttys026 ttys026 commented Jul 19, 2021

Checklist
  • npm test passes
  • tests are included
  • documentation is changed or added
  • commit message follows commit guidelines
Description of change
  • add lifeCycles beforeImport and afterImport, which indicate the start and end of importEntry stage

@ttys026 ttys026 requested a review from kuitos July 19, 2021 04:14
src/error.ts Outdated
}
}

export const isQiankunError = (error: any): error is QiankunError => {
Copy link
Member

Choose a reason for hiding this comment

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

这个 export 出去是来干啥的?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

这个我去掉吧

@@ -78,6 +78,8 @@ toc: menu
type Lifecycle = (app: RegistrableApp) => Promise<any>;
```

- beforeImport - `Lifecycle | Array<Lifecycle>` - 可选
Copy link
Member

Choose a reason for hiding this comment

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

beforeImport 应该是没必要的,调用了 loadMicroApp 可以等价于触发了 beforeImport,这中间误差在 ms 以内

@ttys026 ttys026 changed the title feat: add import lifeCycles fix: error message Jul 19, 2021
@kuitos kuitos merged commit 7001c18 into master Jul 19, 2021
@kuitos kuitos deleted the feat/import-lifeCycles branch July 19, 2021 07:45
kuitos pushed a commit that referenced this pull request May 29, 2022
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