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

Crawler fix #2

Merged
merged 6 commits into from Jun 22, 2015
Merged

Crawler fix #2

merged 6 commits into from Jun 22, 2015

Conversation

pedrofelipe
Copy link
Contributor

No description provided.

@pedrofelipe pedrofelipe changed the title Crawler fix [WIP] Crawler fix Jun 22, 2015
@pedrofelipe
Copy link
Contributor Author

O exemplo de código que estava no blog retornava um hash vazio porque acrescentaram um parâmetro de semana no request dos preços de combustível numa cidade. Eu adicionei um método find_week_code que obtém o identificador da semana atual e envia junto com esse request.

O retorno agora ficou assim: https://gist.github.com/PedroFelipe/1c7a118351d7bc18cb87

Pelo que vi aí, pode haver um problema agora no parseamento. Vou verificar depois 😊

@pedrofelipe pedrofelipe changed the title [WIP] Crawler fix Crawler fix Jun 22, 2015
@rogeriozambon
Copy link
Owner

Obrigado pelo pull request, Pedro.

Há alguns dias, percebi que parou de funcionar e, devido às atividades, não consegui aplicar a correção. Que bom que você enviou ela. Acredito que, além disso, o código mereça uma "refatoração". Mas, fique à vontade para contribuir.

rogeriozambon added a commit that referenced this pull request Jun 22, 2015
@rogeriozambon rogeriozambon merged commit 1171437 into rogeriozambon:master Jun 22, 2015
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.

None yet

2 participants