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

cea sign默认签到时会出现longtitude的bug #41

Closed
mdjhacker opened this issue Mar 19, 2022 · 6 comments
Closed

cea sign默认签到时会出现longtitude的bug #41

mdjhacker opened this issue Mar 19, 2022 · 6 comments
Labels
Type: Bug Type: Question

Comments

@mdjhacker
Copy link

image
@mdjhacker
Copy link
Author

在默认经纬度（使用时不填写），出现此类情况

@mdjhacker
Copy link
Author

修复方法，经过实际测试后为该字段没填写问题
image
image

@mdjhacker
Copy link
Author

此外一个小小的问题，就是说明文档中多了个]，与前文数据格式不太符合。容易造成困惑
image
image

@beetcb
Copy link
Contributor

beetcb commented Mar 19, 2022

在默认经纬度（使用时不填写），出现此类情况

似乎大多数学校签到信息都能找到 placeList，所以算是遇到了一个边缘情况，后期考虑加入错误处理

此外一个小小的问题，就是说明文档中多了个]，与前文数据格式不太符合。容易造成困惑

谢谢提出，已改 @mdjhacker

beetcb added a commit that referenced this issue Apr 14, 2022
@beetcb
fix: optimize template matching logic(#41,#36)
d6b7d16 
Include taskName during match, getting lon,lat from template instead
beetcb added a commit that referenced this issue Apr 14, 2022
@beetcb
fix: optimize template matching logic(#41,#46)
1889fc3 
Include taskName during match, getting lon,lat from template instead
@beetcb
Copy link
Contributor

beetcb commented Apr 14, 2022

新 patch 版本 cea@2.5.8 已修复 @mdjhacker

@beetcb beetcb closed this as completed Apr 14, 2022
@mdjhacker
Copy link
Author

@beetcb 谢谢您的贡献

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Type: Bug Type: Question
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mdjhacker @beetcb