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

docs: update orm.md #3833

Merged
merged 5 commits into from
May 7, 2024
Merged

docs: update orm.md #3833

merged 5 commits into from
May 7, 2024

Conversation

flyingcrp
Copy link
Contributor

Add Different path support for DataSource

Checklist
  • npm test passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)
Description of change

Add Different path support for DataSource
@@ -1681,3 +1681,25 @@ export default {
- 1、检查 `config.default.ts` 中的 `entities` 配置是否正确
- 2、检查 `configuration.ts` 文件,确认是否引入 orm

### DataSource 的不同路径支持
```
Copy link
Member

Choose a reason for hiding this comment

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

image
这个上面有写吧。。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

对,我看到这里不完善所以顺手给过来。实际可以改到这里,表示支持多种方式的实体加载。我的本意是这个

Copy link
Member

Choose a reason for hiding this comment

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

虽然支持那么多种。。。但是如果现在只写 ts 后缀是会报错的,很多人看很多种就随便写一种或者干脆不看,所以之前就写了一种。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

确实会存在你说的情况,我又做了描述修改,把后缀匹配都加上去了。

Copy link
Member

Choose a reason for hiding this comment

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

英文的也顺便改了吧-

Copy link
Contributor Author

@flyingcrp flyingcrp May 7, 2024

Choose a reason for hiding this comment

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

😆 好了

Update orm.md
修改路径和文件名的匹配方式
@czy88840616 czy88840616 added the pr: documentation This PR works on the website or other text documents in the repo. label May 7, 2024
@czy88840616 czy88840616 changed the title Update orm.md docs: update orm.md May 7, 2024
Update orm.md
Update orm.md
@czy88840616 czy88840616 merged commit 8bc2f2b into midwayjs:main May 7, 2024
2 checks passed
@flyingcrp flyingcrp deleted the patch-2 branch May 7, 2024 10:43
czy88840616 pushed a commit that referenced this pull request May 10, 2024
* Update orm.md

Add Different path support for DataSource

* Update orm.md

Update orm.md

* Update orm.md

修改路径和文件名的匹配方式

* Update orm.md

Update orm.md

* Update orm.md

Update orm.md

(cherry picked from commit 8bc2f2b)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: documentation This PR works on the website or other text documents in the repo.
Development

Successfully merging this pull request may close these issues.

None yet

2 participants