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.

Sign up for GitHub

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

Jump to bottom

@alilc/lowcode-designer/lib/document/node/node.d.ts 丢失 #2677

Closed
eightHundreds opened this issue Nov 21, 2023 · 6 comments
Closed

@alilc/lowcode-designer/lib/document/node/node.d.ts 丢失 #2677

eightHundreds opened this issue Nov 21, 2023 · 6 comments
Labels
help wanted Extra attention is needed / 老铁，赚 PR 的机会来了

Comments

@eightHundreds
Copy link
Contributor

https://www.jsdelivr.com/package/npm/@alilc/lowcode-designer?tab=files&path=lib%2Fdocument%2Fnode

有node.js,没有node.d.ts
image
@liujuping
Copy link
Collaborator

这个对你的项目有什么影响吗？

@eightHundreds
Copy link
Contributor Author

开发体验问题.影响不大.

只是个人理解这个问题修起来比较简单.

@liujuping
Copy link
Collaborator

什么情况下，会使用到这里的类型定义，可以提供一个示例吗？我需要确定一下。

@eightHundreds
Copy link
Contributor Author

我在替换designer内的simulatorView。会用到一些内部的方法，我绕过了shell层，直接访问IDocument。
// 虽然你们不推荐绕过shell,但这是目前唯一的手段

@liujuping
Copy link
Collaborator

明白，这块大概率是出现了循环引用，可以帮忙找一下提一下 PR。

@JackLian JackLian added the help wanted Extra attention is needed / 老铁，赚 PR 的机会来了 label Nov 27, 2023
@github-actions GitHub Actions
Copy link

Hello @eightHundreds. We totally like your proposal/feedback, PR wanted。

你好 @eightHundreds，我们完全同意你的提议/反馈，欢迎 PR。

@eightHundreds eightHundreds mentioned this issue Jan 15, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted Extra attention is needed / 老铁，赚 PR 的机会来了
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@JackLian @liujuping @eightHundreds