-
Notifications
You must be signed in to change notification settings - Fork 199
docs: add a map with flying lines demo #2714
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
Merged
xiaoluoHe
merged 14 commits into
VisActor:develop
from
FunctionEurus:demo/map-with-flying-lines
May 28, 2024
Merged
docs: add a map with flying lines demo #2714
xiaoluoHe
merged 14 commits into
VisActor:develop
from
FunctionEurus:demo/map-with-flying-lines
May 28, 2024
Conversation
This file contains hidden or 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
xiaoluoHe
reviewed
May 23, 2024
…/VChart into demo/map-with-flying-lines
抱歉,由于我没能在pr页面里找到这条评论,因此就使用邮件直接回复了~
已经将地图数据修改为散点地图demo中所使用的全球地图了。同时发现一个小bug,缩放或移动时折线图的位置会保持固定,因此我在spec里先关闭了roam,正在寻找解决方法。
…--------------原始邮件--------------
发件人:"Feifei He ***@***.***>;
发送时间:2024年5月23日(星期四) 晚上7:48
收件人:"VisActor/VChart" ***@***.***>;
抄送:"FUNction ***@***.***>;"Mention ***@***.***>;
主题:Re: [VisActor/VChart] docs: add a map with flying lines demo (PR #2714)
-----------------------------------
@FunctionEurus 看了下 geojson.cn 这个站点文档,没有说明地图数据获得了审图号,所以有一定的使用风险(地图合规问题还是比较敏感的)
所以,方便的话,最好换成其他国家的地图~感谢感谢🙏
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
抱歉,之前的评论,我编辑时不小心删除了。 |
好的!不过review评论我只找到不要出现中文这个,已经解决了 |
@FunctionEurus 向上翻一下 pr conversation |
xiaoluoHe
approved these changes
May 27, 2024
@xiaoluoHe 感谢您的截图,截图中的问题均已在新的commit中修正。不过还是很抱歉,我还是只能找到不要出现中文的review comment,而且也没有收到github的邮件推送,不知道是不是权限问题之类导致的 |
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.
[中文版模板 / Chinese template]
🤔 This is a ...
🔗 Related issue link
implement #2660
🔗 Related PR link
🐞 Bugserver case id
💡 Background and solution
📝 Changelog
☑️ Self-Check before Merge
🚀 Summary
copilot:summary
🔍 Walkthrough
copilot:walkthrough