search

renatopanda / IPTomar-MEI-IoT-ES

Projecto de Engenharia de Software
0 stars 0 forks source link

Caria #21

Closed caria closed 7 years ago

caria commented 7 years ago

Upload de Ficheiros. Falta ligação ao Projects. Necessita de correr update no Vagrant "sudo apt-get install imagemagick" para mostrar e importar thumbnail

renatopanda commented 7 years ago

É importante ir registando )p. ex. no readme.md) software extra que seja necessário instalar no servidor de produção, tal como o imagemagick.

nelsonmpg commented 7 years ago

Boas Paulo. Pelo que estive a ver no pull request existe muito locais nos ficheiros que alteras-te que deram conflito que não foram resolvidos pois aparece <<<<<<< HEAD ======= >>>>>>> origin/caria esto não é para estar no código, mas não basta ir aos ficheiros e apagar, é preciso verificar qual a parte do código em conflito que deve de ficar ou se tem que ficar as duas partes em conflito. É de todo conveniente que ao se efectuar um upload ele seja feito para locais diferentes de forma a separarmos imagens de documentos e se calhar por protejo de forma a facilmente o caminho para a imagem se construindo, também ser possível para o mesmo local efectuar o upload de mais do que uma imagem.

caria commented 7 years ago

Boa tarde, eu fiz o pull porque senão estou sempre a fazer tudo de novo porque as alterações dos colegas são constantes e eu não consigo inserir pelo menos o básico no projeto.

Vou tentar corrigir os conflitos.

O pull request até foi feito com ajuda do Vasco, já por ter receio de fazer alguma coisa mal feita.

Cumprimentos

No dia 27 de dezembro de 2016 às 20:40, nelsonmpg notifications@github.com escreveu:

Boas Paulo. Pelo que estive a ver no pull request existe muito locais nos ficheiros que alteras-te que deram conflito que não foram resolvidos pois aparece <<<<<<< HEAD ======= >>>>>>> origin/caria esto não é para estar no código, mas não basta ir aos ficheiros e apagar, é preciso verificar qual a parte do código em conflito que deve de ficar ou se tem que ficar as duas partes em conflito. É de todo conveniente que ao se efectuar um upload ele seja feito para locais diferentes de forma a separarmos imagens de documentos e se calhar por protejo de forma a facilmente o caminho para a imagem se construindo, também ser possível para o mesmo local efectuar o upload de mais do que uma imagem.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/nelsonmpg/IPTomar-MEI-IoT-ES/pull/21#issuecomment-269379763, or mute the thread https://github.com/notifications/unsubscribe-auth/ADtfLO2p12REAI1oDOKgl0y5lh5npZ5lks5rMXeogaJpZM4LWcDz .

renatopanda commented 7 years ago

@caria fizeste bem em abrir o pull request, a ideia é mesmo essa: 1) criar um branch a partir do projecto actualizado (master) 2) implementar uma funcionalidade 3) quando pronta e testada, abrir um pull request 4) corrigir eventuais problemas e fazer merge, ou seja, copiar isso para o master.

Claro que em equipa isto dá problemas, quanto mais tempo demoramos a fazer pull request, mais alterações vão ser feitas no master e mais desactualizado vai ficar o código por onde começamos em 1. Há duas opções: ou somos rápidos para evitar que haja tantas mudanças que o projecto rebente; ou se são funcionalidades mais demoradas, temos que ir tentando manter o nosso projecto actualizado, trazendo as alterações do master para o nosso branch de tempos a tempos.

Calhou-te a 2a opção. 😄 Dito isto, já devias ter aberto o pull request mais cedo. O projecto não pára e por isso é preciso este trabalho extra para uma integração contínua. Não tenhas medo de estragar, é assim que se aprende, foi assim que o @VascoJM , @PedroMatos1972 e @NearDeath fizeram ao início e eventualmente dá para voltar atrás.

Vou fazer uma revisão rápida do teu branch e anotar algumas coisas a corrigir. Vou esperar que outros também façam o mesmo e tu só tens que as corrigir fazendo novos commits para o teu branch. Assim que o branch seja considerado funcional será integrado no master. Como está actualmente provavelmente não funciona, pelo menos não correctamente. Quanto mais depressa te livrares deste trabalho, mais cedo podes partir para outro. 👍

caria commented 7 years ago

Boas, estive a concluir um trabalho, agora já me posso voltar a dedicar ao projeto.

O que foi feito foi apenas o básico, em seguida estava a pensar fazer um db migration à tabela de projeto para adicionar um campo tipo titledoc: string para poder associar fotos aos projetos.

Pelo que entendi pretendem que o upload de ficheiros seja feito em separado dos ficheiros tipo imagem?

Vou verificar os erros.

Obrigado

2016-12-28 17:14 GMT+00:00 renatopanda notifications@github.com:

Assigned #21 https://github.com/nelsonmpg/IPTomar-MEI-IoT-ES/pull/21 to @caria https://github.com/caria.

— 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/21#event-906700927, or mute the thread https://github.com/notifications/unsubscribe-auth/ADtfLCooVUCt434JwHmNVUBza08r2JsNks5rMpjngaJpZM4LWcDz .

MiguelFCoelho commented 7 years ago

Boas, eu estive a falar com o prof para tentar te ajudar, criei um branch novo e fui fazer testes. Consigo quando se adiciona uma pessoa, adicionar já com imagem. e no show tb já apareçe. Vou continuar a fazer para adicionar pdf no projeto, e pdf na presentation. e depois quando acabar, vemos todos para juntar o que fiz ao teu trabalho tb.

Espero que não te importes =S