fix: 使用更靠谱的方法从json提取数据 #193
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
😎 |
|
这个提交流程好专业，学习了 |
|
我看了下大致理解你的思路， 理论上一律使用最新节点可能也符合90%的使用场景，所以我觉得也许可以如果 mapping 过程发现任何应该要有选择而 dom 上却没正确找到的，fallback 回使用这个方法 |
|
No no我的正确率是100% |
|
我晚上看看 |
|
如果是原来的代码，因为没有前端，export all是不能正常工作的 |
|
我之前有几个对话因此丢过数据，我相信这个困扰很多人都有，估计很多客户就直接流失了 |
pionxzh
approved these changes
Dec 24, 2023
|
👍 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
目前还未去掉头部的两个system信息
closes #190