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

fix two problems #20

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

greatbody
Copy link

修复检查到的问题:
1、手机端使用的时候,选择省份后,城市选择依然为空
2、传递的json由data打头,导致数据无法识别。

请测试后合并

@saberma
Copy link
Owner

saberma commented Jul 18, 2015

这个是否跟 #19 是同一个问题?

@greatbody
Copy link
Author

一半。关于json会带有controller名字这个。是重复的。
关于手机端选择“省”后无法自动带出“市”,这个问题,是我这有,他那个pull request没有的。
因为当时项目需要时间紧迫,就新开了一个分支,做了一个ChinaCitys,和你的ChinaCity区别,保证程序可以正常执行。修改仓促,没有做test。只是确保了我的项目引用ChinaCitys这个gem包无故障。

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