Ver projectos com parciais + dados dinâmicos + expor informação extra

[renatopanda]

Como tínhamos combinado, alterei o vosso projecto para que a vista de projectos seja reutilizada nos sítios desejados. Aproveitei o andamento para expor toda a informação, já que o que o @Pacn91 tinha dúvidas nisto. Acabei por adicionar uma série de coisas extra, inclusive seeds novas (agora demora uns minutos a fazer db:setup.

Já agora, a pasta public/system vai crescendo com o upload de ficheiros e não me parece boa ideia que esteja no repositório daí que tenha colocado no .gitignore e apagado.

Testem bem isto e analisem bem as mudanças no código antes de aceitar o merge (ou recusar!).

nota: existem uma série de bugs ou coisas mal posicionadas no interface (tipo a sidebar andar a dançar, coisas fora do sítio ou imagens em tamanho errado, listas de 30 disciplinas no curso - que podem ser escondidas com o dropdown e / ou mostrar só as que têm projectos, etc.) mas isso não é comigo. Só perdi tempo a mostrar lá as coisas, o resto é para os responsáveis por essa parte resolverem :)

[PedroMatos1972]

Estava a testar o "Review" como o professor @renatopanda pediu, más ocorre um erro, algém já teve e consegui resolver ! @NearDeath, @Pacn91, @nelsonmpg, @VascoJM

[PedroMatos1972]

Bom resolvi o problema no meu com os seguintes gem's no gemfile;

gem 'imagemagick-identify', '~> 0.0.1'

gem 'imagemagick-binaries', '~> 6.8', '>= 6.8.7.5.1'

assim o rails db:setup já carrega tudo sem erros. Abraços; @renatopanda @NearDeath @Pacn91 @nelsonmpg @VascoJM

[renatopanda]

Duas questões. Tinhas o ImageMagick instalado na vm e fizeste o bundle install antes? Dentro da vm devia ser só fazer:

sudo apt-get install ImageMagick -y
bundle install
rails db:setup

A malta da imagens é que pode falar disso @caria @NearDeath . Mas a mensagem de erro que mostras é mesmo da falta do ImageMagick instalado no Ubuntu. Testem se as gems são mesmo necessárias ou não.

[PedroMatos1972]

Professor @renatopanda , sim tinha feito antes bundle install.... no vagrant, se calhar é do Mac.

• Eu coloquei essas duas gem's no gemfile e fiz o bundle install e o bundle update e funcionou o rails db:setup, consegui levantar o rails s, e testar o projecto.

[renatopanda]

Óptimo! Uma nota, não deves fazer bundle update, basta o bundle install. De resto depois de almoço corrijo. Não mexi nesses formulários mas é possível que estejam errados depois de outras alterações. De qualquer maneira inserir escola curso e disciplina através do site não vai ser praticamente usado. Testa o que te lembrares e indica os erros que depois submeto os fixes. As mudanças significativas que fiz foram no homem, as seeds e nos show das coisas todas.

[renatopanda]

@PedroMatos1972 não pode ser do Mac porque estamos todos a correr o mesmo SO, Ubuntu numa vm, logo se correr nos outros tem que dar no teu e vice versa. O que acho que te faltou foi um comando para instalar binários do ImageMagick dentro da vm, tal como o @caria tinha indicado e está no readme. Mas como disse logo à tarde eu ainda experimento isso.

A ideia aqui é para fazerem revisão do trabalho e indicarem erros ou alterações, não é para corrigirem por vocês se não dá a confusão do costume de muitos a alterarem o mesmo 😄

[PedroMatos1972]

Boas Professor @renatopanda , tive a testar o resto, no insert project novo, só regista se tiver seleccionado a box do user. Quando seleccionei como destaque o novo projecto a ser inserido, penso que deveria aparecer nos primeiro, más não acontece... de resto parece-me tudo ok, a funcionar a 100 à hora.

[renatopanda]

Nas próximas horas vou testar e fazer algumas correcções então.

[renatopanda]

Corrigi diversos erros antigos graças à revisão (que também devia ter sido de código) do @PedroMatos1972. No fundo foi o que faltou fazer em alguns merges anteriores. Daí a importância das revisões, se não existem mais facilmente são inseridos erros no master.

Já agora, o formulário de inserir/editar projecto tem que ser revisto. Mudei os campos das datas para algo mais básico por agora, o que estava era só um text_field. O que vocês viam como datepicker é só algo que o chrome faz mas nos outros browsers não funciona - depois falamos disto. Para além disso o formulário está demasiado complexo, na aula falamos e avançamos nisto.

Testem as últimas alterações e depois é com vocês.

[renatopanda]

Em todos eles? Ou há projectos sem data em que acontece isso e nunca abri nenhum por acaso?

On Jan 13, 2017 11:37, "PedroMatos1972" notifications@github.com wrote:

@PedroMatos1972 commented on this pull request.

erro no project#show após o insert do projecto... "undefined method `to_formatted_s' for nil:NilClass"

In ProjectSubmit/app/views/projects/show.html.erb https://github.com/nelsonmpg/IPTomar-MEI-IoT-ES/pull/33#pullrequestreview-16552567 :

     </h1>

     <div class="row">
       <div class="col-xs-12 meta">
         <div class="col-xs-6">

• Publicado por <%= @project.user.email.split('@')[0] %> • <%= @project.date.to_formatted_s(:long) %>
• <% if (@project.finished?) %>
• Projecto terminado em <%= @project.date.to_formatted_s(:long) %>

erro.... (undefined method `to_formatted_s' for nil:NilClass)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/nelsonmpg/IPTomar-MEI-IoT-ES/pull/33#pullrequestreview-16552567, or mute the thread https://github.com/notifications/unsubscribe-auth/ABi0JG3H4-7f3qrGsp2KlXn4244P4eOnks5rR2H3gaJpZM4LgDsM .

[renatopanda]

@PedroMatos1972 vou sair. Esse bug poderia acontecer caso o projeto não tenha data mas como nas seeds todos têm nem percebo como aconteceu, de qualquer forma corrigi isso - espero eu. Actualizei o bootstrap para a alpha6, têm que fazer bundle install.

[PedroMatos1972]

@renatopanda boas professor, sim eu reparei, más pensei que não queria assim, porque podemos não ter data de apresentação prevista....! Más se calhar assim faz mais sentido....