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

make proposers and withdrawers the same format (for assembly_id < 19)… #31 #32

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

Conversation

hunkim
Copy link
Contributor

@hunkim hunkim commented Nov 27, 2015

제가 #31 에 올려둔 이슈를 해결했습니다. 이제 19대 이상의 자료에서는 withdrawers도 proposers와 같은 형식으로 자료가 나옵니다.

제가 19대에 자료에 시험해보니 잘 됩니다.

"withdrawers": [
    {
      "name_kr": "\uac15\uae30\uc724",
      "party": "\uc0c8\ub204\ub9ac\ub2f9",
      "name_cn": "\u59dc\u8d77\u6f64"
    },
    {
      "name_kr": "\uae40\uc131\ucc2c",
      "party": "\uc0c8\ub204\ub9ac\ub2f9",
      "name_cn": "\u91d1\u76db\u8d0a"
    },

@e9t
Copy link
Member

e9t commented Nov 28, 2015

말씀하신대로 형식을 통일하니 훨씬 좋아보이네요. 감사합니다 :)
다만 이 코드는 지금 pokr.kr에 production용으로 사용되고 있는 코드와 완전히 동일한 것이라, API가 변경되면 깨지는 부분이 있을까 염려됩니다. 보내주신 pull request는 일단 열어두고, 차후에 업데이트/ 점검시 같이 적용할 수 있도록 하겠습니다.

@hunkim
Copy link
Contributor Author

hunkim commented Dec 2, 2015

제가 branch를 만들지 않았더니 추가로 수정한것도 여기 pull reuqest에 붙어 버리네요. 간단하게 safe.py를 추가 해두었습니다. 3번 http_get과 parsing을 3번 해봅니다.

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