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

São João de Meriti/RJ #1011

Merged
merged 1 commit into from
Jan 22, 2024
Merged

São João de Meriti/RJ #1011

merged 1 commit into from
Jan 22, 2024

Conversation

almeidadm
Copy link
Contributor

@almeidadm almeidadm commented Oct 28, 2023

Checklist - Novo spider

  • Você executou uma extração completa do spider localmente e os dados retornados estavam corretos.
  • Você executou uma extração por período (start_date e end_date definidos) ao menos uma vez e os dados retornados estavam corretos.
  • Você verificou que não existe nenhum erro nos logs (log_count/ERROR igual a zero).
  • Você definiu o atributo de classe start_date no seu spider com a data do Diário Oficial mais antigo disponível na página da cidade.
  • Você garantiu que todos os campos que poderiam ser extraídos foram extraídos de acordo com a documentação.

Descrição

Construção da spider pra São João de Meriti/RJ através do portal https://transparencia.meriti.rj.gov.br/.
Consulta mês a mês através do subdomínio /diario_oficial_get.php e desconsidera os itens com código de anexo igual a 1.
Download delay de 1.5 segundos para evitar o recebimento de error de servidor 500.
Estrutura de código similar ao encontrado para a cidade de Belford Roxo/RJ.

Resolve #961

@trevineju trevineju added the hacktoberfest-accepted Pull Requests aprovados na Hacktoberfest label Oct 28, 2023
Copy link
Member

@ogecece ogecece left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@almeidadm ótimo PR! Só dá pra dar uma melhorada nas requisições iniciais e corrigir umas bobagens:

@almeidadm
Copy link
Contributor Author

Atualizações feitas!

Copy link
Member

@trevineju trevineju left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Show, @almeidadm. Você poderia só juntar todos os 8 commits em um só?
Mesclo a PR assim que fizer esse ajuste

atualização: tinha que fazer ajuste em outro lugar e aí acabei já dando o squash aqui também

@trevineju
Copy link
Member

Arquivos da coleta completa de teste que executei hoje:
rj_meriti.csv
rj_meriti.txt

rj_sao_joao_de_meriti

pr_guarapauva

removing wrong spider

alterando conforme solicitações do PR

fix date filter in line 22

adding geq and leq to if statement
Copy link
Member

@trevineju trevineju left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Obrigada pela contribuição!! \o/

@trevineju trevineju merged commit 6cff16c into okfn-brasil:main Jan 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted Pull Requests aprovados na Hacktoberfest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

São João de Meriti - RJ
4 participants