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

Issue-#441: Adição do link para plenária na lista de votações filtradas #470

Merged

Conversation

felipe72
Copy link
Contributor

O link ficou como demonstrado na imagem a seguir:

screenshot from 2018-05-10 16-39-47

Co-authored-by: Felipe [email protected]
Co-authored-by: Gustavo Lopes [email protected]

@felipe72 felipe72 changed the title Adição do link para plenária na lista de votações filtradas Issue-#441: Adição do link para plenária na lista de votações filtradas May 10, 2018
@coveralls
Copy link

Pull Request Test Coverage Report for Build 955

  • 0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 82.327%

Totals Coverage Status
Change from base Build 954: 0.0%
Covered Lines: 3913
Relevant Lines: 4753

💛 - Coveralls

@diraol
Copy link
Member

diraol commented May 11, 2018

Gente, só para dar um feedback temporário, eu olhei o PR e me parece tudo certo, amanhã (sábado) eu rodo localmente para ver e, estando tudo ok, eu faço o merge!

@diraol
Copy link
Member

diraol commented May 12, 2018

Pessoal, não consegui testar o PR de vocês por conta das issues #471 e #472. =(

Outra coisa, por favor, quando for um PR de um único commit sempre coloquem o número da issue sendo resolvida na própria mensagem do commit (no título do PR também é bom).
Se for mais de um commit, ai pode ser na descrição do Pull Request mesmo.
De preferência prefixado por um dos "comandos" que fecham a issue caso ela esteja sendo completamente resolvida pelo PR ou pelo commit. (p.ex. Close, closes, fix, etc). Isso ajuda a evitar que uma issue fique aberta depois de ter sido resolvida ;)

@leonardofl
Copy link
Member

Verifiquei aqui na minha máquina e tá legal. Valeu!

@leonardofl leonardofl merged commit 39745ac into radar-parlamentar:master May 12, 2018
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.

4 participants