Skip to content
This repository has been archived by the owner on Aug 7, 2018. It is now read-only.

Partido duplicado#260 #305

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

Conversation

Thaiane
Copy link
Contributor

@Thaiane Thaiane commented Jun 23, 2015

Foi modificado o algoritmo que filtra os partidos, para que ele pudesse verificar se tinham partidos duplicados passando. A parte de verificar os parlamentares não foi realizado, porque a gente não encontrou nenhum parlamentar duplicado nos gráficos.

@diraol
Copy link
Member

diraol commented Jun 23, 2015

Pessoal, vocês misturaram commits da issue #260 com a issue da timeline neste Pull Request?

@leonardofl
Copy link
Member

humm... no gráfico eu já vi parlamentar replicado sim. Em algum momento o Ivan Valente aparece replicado, por exemplo.
E nesse caso, antes de um contorno na interface, seria importante descobrir se o problema não é do importador que está inserindo parlamentares duplicados (mais provável).

@Thaiane
Copy link
Contributor Author

Thaiane commented Jun 24, 2015

@leonardofl e @diraol, vamos verificar as duas questões junto a equipe e amanhã (quinta) respondemos aqui.

@brenddongontijo
Copy link
Contributor

Opa, realmente tem commits da timeline, provavelmente iniciamos uma branch à partir da mesma.
Daremos uma olhada nisso, nas considerações da issue #260 e daremos um feedback depois.

ecmeirelles and others added 2 commits June 30, 2015 14:36
Signed-off-by: Brenddon Gontijo <[email protected]>
Signed-off-by: Ana Carolina Lopes <[email protected]>
Signed-off-by: Elaine Cristina Meirelles <[email protected]>
Signed-off-by: Elaine Cristina Meirelles <[email protected]>
Signed-off-by: Brenddon Gontijo <[email protected]>
@paulohtfs paulohtfs force-pushed the partido_duplicado#260 branch from e00ab1d to f574b26 Compare June 30, 2015 17:38
@vannessasoares
Copy link
Contributor

Retiramos os commits referentes a timeline. Olhamos nos métodos de importadores e vimos q ele contem uma verificação de partidos e ñ deixa duplicar. Não sabemos se tem outro local que possa ter ocorrido a duplicação...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants