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

feat: 添加 Leaflet 项目作为地图打点,可以在web上预览 #899

Merged
merged 2 commits into from
Jul 19, 2022

Conversation

robothot
Copy link
Contributor

@robothot robothot commented Jul 19, 2022

效果信息如下 地图数据来源 openstreetmap

image

link

后续加上搜索的功能,但是 /data/generated/cities-for-visualization.json 这里面生成的数据信息好像没有小区名称

@robothot
Copy link
Contributor Author

@MarkShawn2020 有什么想法, 可以执行 npm install && npm run start 来进行预览

@MarkShawn2020
Copy link
Collaborator

  1. 我在分支上运行看一下
  2. cities-for-visualization 顾名思义,是城市级别的,所有没有小区,需要小区级别数据的请参见我最新更新的房天下爬取接口

@MarkShawn2020 MarkShawn2020 changed the base branch from main to dev July 19, 2022 09:50
@MarkShawn2020 MarkShawn2020 merged commit 2a9b30e into WeNeedHome:dev Jul 19, 2022
@MarkShawn2020
Copy link
Collaborator

我刚跑了一下,目前的问题是 marker 不显示,你看看是啥原因

image

@robothot
Copy link
Contributor Author

@MarkShawn2020 你的 npm 版本是多少?我这把跑了一下没问题。

要求

npm:  >=7.0.0
node: >=16.0.0

能发一下给我看看你的 npm 的信息吗?

@MarkShawn2020
Copy link
Collaborator

@MarkShawn2020 你的 npm 版本是多少?我这把跑了一下没问题。

要求

npm:  >=7.0.0
node: >=16.0.0

能发一下给我看看你的 npm 的信息吗?

npm: 8.12.1
node: v18.4.0

@robothot
Copy link
Contributor Author

@MarkShawn2020 我运行了一下, 没有问题, 你试试单独用 vscode 打开这个项目

然后访问 http://127.0.0.1:9080/marker-icon.png 这个地址,应该可以找到对应的 icon

@robothot
Copy link
Contributor Author

  1. git clone [email protected]:WeNeedHome/SummaryOfLoanSuspension.git
  2. git checkout dev
  3. cd SummaryOfLoanSuspension/development/frontend/leaflet
  4. 打开 git base
  5. 执行 npm install
  6. 执行 npm run start

我怀疑是在根目录执行的命令,导致没有找到对应的 public 目录

@MarkShawn2020
Copy link
Collaborator

@MarkShawn2020 我运行了一下, 没有问题, 你试试单独用 vscode 打开这个项目

然后访问 http://127.0.0.1:9080/marker-icon.png 这个地址,应该可以找到对应的 icon

知道图片地址就行,我来调试调试

@robothot
Copy link
Contributor Author

@MarkShawn2020 感觉弄一个 github pages 会好一点, 然后采用 github action 进行部署

@MarkShawn2020
Copy link
Collaborator

找到原因了,是因为前端申请的资源是 Imagemarker-icon-2x.png,为啥有个2x啊?

@MarkShawn2020
Copy link
Collaborator

image

找到了,retina显示屏(我是mac)需要2x的图片,加一张就好了

@robothot

This comment was marked as resolved.

@robothot
Copy link
Contributor Author

@MarkShawn2020 等等 我打个补丁. 我这个代码写的有点问题,没考虑到 retina显示屏

@MarkShawn2020
Copy link
Collaborator

ok,验证通过了,你继续做你想做的吧(可以看一下我们的前端todo),我先给你合并一下

@robothot
Copy link
Contributor Author

@MarkShawn2020 我需要一个小区级别的数据格式. 这个能帮忙准备一下吗?

@MarkShawn2020
Copy link
Collaborator

MarkShawn2020 commented Jul 19, 2022

这个需要借助我今早发布的 backend-py-sdk,用来爬开发商与小区数据,目前优先级不是很高,而且量大,脏处理比较多,你会的话可以试一下,我还要先做其他 features

@MarkShawn2020
Copy link
Collaborator

MarkShawn2020 commented Jul 19, 2022

你的project已合入主分支,另外,你的技术栈跟我接近,或许可以考虑与我之前的react项目融合,一起开发

@robothot
Copy link
Contributor Author

我看了一下你的项目,是基于 CreateReactApp 的后面调整webpack 比较麻烦, 而且打包的时候按需加载也比较麻烦, 如果加 React Router DOM 做路由还要自己配置, 所以用 fastpack 的脚手架会好点, 编译速度和打包速度,以及首页加载速度都比 Create React App 要快

@MarkShawn2020
Copy link
Collaborator

我看了一下你的项目,是基于 CreateReactApp 的后面调整webpack 比较麻烦, 而且打包的时候按需加载也比较麻烦, 如果加 React Router DOM 做路由还要自己配置, 所以用 fastpack 的脚手架会好点, 编译速度和打包速度,以及首页加载速度都比 Create React App 要快

可以,那就基于你的项目开发,看情况我们合作

@MarkShawn2020
Copy link
Collaborator

webpack的调整一般可以用eject,暂时还没那个需求,其实如果有建站需求的话,可以考虑用一些ssr框架,性能会更好

@robothot
Copy link
Contributor Author

webpack的调整一般可以用eject,暂时还没那个需求,其实如果有建站需求的话,可以考虑用一些ssr框架,性能会更好

eject 太麻烦, 没事 fastpack 我熟悉一点, 而且 ui 框架 fast-ui 一整套搭建起来很方便的.

主要是后面想做 pwa, ssr 很消耗服务器资源的, pwa 比 ssr 快很多,并且不怎么消耗服务器资源

@MarkShawn2020
Copy link
Collaborator

嗯好的~

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