okfn-brasil / querido-diario

📰 Diários oficiais brasileiros acessíveis a todos | 📰 Brazilian government gazettes, accessible to everyone.
https://queridodiario.ok.org.br/
MIT License
1.11k stars 411 forks source link

Novo municipio Maracacume #1310

Closed antonishuamani closed 3 weeks ago

antonishuamani commented 3 weeks ago

AO ABRIR uma Pull Request de um novo raspador (spider), marque com um X cada um dos items da checklist abaixo. Caso algum item não seja marcado, JUSTIFIQUE o motivo.

Layout do site publicador de diários oficiais

Marque apenas um dos itens a seguir:

Código da(s) spider(s)

Testes

log_ma_maracacume.txt ma_maracacume.csv

Verificações

Descrição

<Descreva o seu Pull Request informando a issue (caso exista) que está sendo solucionada ou uma descrição do código apresentado>

trevineju commented 3 weeks ago

[ X] Uma coleta-teste completa foi feita. Os arquivos de .log e .csv deste teste estão anexados na PR.

oi, @antonishuamani. Você marcou que executou e anexou os arquivos, porém não o fez. Pode anexar os arquivos?

antonishuamani commented 3 weeks ago

[ X] Uma coleta-teste completa foi feita. Os arquivos de .log e .csv deste teste estão anexados na PR.

oi, @antonishuamani. Você marcou que executou e anexou os arquivos, porém não o fez. Pode anexar os arquivos?

acabei de anexar os archivos que voce solicitou

trevineju commented 3 weeks ago

acabei de anexar os archivos que voce solicitou

boa, @antonishuamani! Obrigada! Olhei os logs e a coleta parece estar boa e certa.

trevineju commented 3 weeks ago

Vou remover o outro raspador que adicionou à PR. Não teria problema contribuir com dois (ficaria faltando só adicionar os arquivos de teste), mas a questão é que o documento do diário oficial - escolhe qualquer um daqui https://altoparnaiba.ma.gov.br/diario/diario pra ver - é um diário agregado de vários municípios e não um documento exclusivo de Alto Paraíba. E a arquitetura do QD demanda outro fluxo para adicionar um diário agregado, coisa que ainda não temos para a Federação do Maranhão.

antonishuamani commented 3 weeks ago

Vou remover o outro raspador que adicionou à PR. Não teria problema contribuir com dois (ficaria faltando só adicionar os arquivos de teste), mas a questão é que o documento do diário oficial - escolhe qualquer um daqui https://altoparnaiba.ma.gov.br/diario/diario pra ver - é um diário agregado de vários municípios e não um documento exclusivo de Alto Paraíba. E a arquitetura do QD demanda outro fluxo para adicionar um diário agregado, coisa que ainda não temos para a Federação do Maranhão.

O outro raspador (Parnaíba) foi um descuido; na verdade, eu não tinha a intenção de adicioná-lo ao PR. Estava apenas testando um novo código, então não tenho problema em que ele seja removido.

trevineju commented 3 weeks ago

O outro raspador (Parnaíba) foi um descuido; na verdade, eu não tinha a intenção de adicioná-lo ao PR. Estava apenas testando um novo código, então não tenho problema em que ele seja removido.

Sem problemas. Se você não usar a sua main para fazer contribuições, esse tipo de descuido tende a não acontecer. Fica a dica pras próximas ;)

antonishuamani commented 3 weeks ago

muito obrigado pelos conselhos. likes