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

Adiciona spider de TO Sampaio. #1029

Merged
merged 1 commit into from
Jan 22, 2024
Merged

Adiciona spider de TO Sampaio. #1029

merged 1 commit into from
Jan 22, 2024

Conversation

vbmendes
Copy link
Contributor

@vbmendes vbmendes commented Nov 5, 2023 •
edited
Loading

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

Implementa o spider para a cidade de Sampaio/TO.
Issue: #1022

@trevineju trevineju linked an issue Nov 9, 2023 that may be closed by this pull request
[Novo spider]: Sampaio-TO #1022
Closed
1 task
ogecece
ogecece requested changes Nov 29, 2023
View reviewed changes
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.

Tá 10/10 @vbmendes ! Só pequenos comentários:

@vbmendes vbmendes requested a review from ogecece November 29, 2023 15:57
@vbmendes
Copy link
Contributor Author

Tá 10/10 @vbmendes ! Só pequenos comentários:

Obrigado pela revisão @ogecece. Fiz as alterações sugeridas. Poderia revisar novamente, por favor?

@vbmendes @trevineju
Adiciona spider de TO Sampaio.
ce02828 
Corrige is_extra_edition de to_sampaio.

Update data_collection/gazette/spiders/to/to_sampaio.py

Signed-off-by: Vinícius Mendes <[email protected]>

Update data_collection/gazette/spiders/to/to_sampaio.py

Co-authored-by: Giulio Carvalho <[email protected]>
Signed-off-by: Vinícius Mendes <[email protected]>

Remove callback para utilizar o padrão.

Update data_collection/gazette/spiders/to/to_sampaio.py

Signed-off-by: Juliana Trevine <[email protected]>
@trevineju
Copy link
Member

Adicionei um ajuste no start_date e fiz um squash no histórico para resumir os commits em um.

Executei a coleta completa do raspador e tudo certo.
to_sampaio.csv
to_sampaio.txt

trevineju
trevineju approved these changes Jan 22, 2024
View reviewed changes
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, @vbmendes!

@trevineju trevineju merged commit bb8f4c7 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
Reviewers

@trevineju trevineju trevineju approved these changes

@ogecece ogecece Awaiting requested review from ogecece

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Novo spider]: Sampaio-TO
3 participants
@vbmendes @trevineju @ogecece