search

culturagovbr / mapasculturais_descontinuado

Mapas Culturais
16 stars 2 forks source link

Arrumar link do "Logo Instituição" - SaaS #66

Closed marcus2703 closed 7 years ago

marcus2703 commented 7 years ago

Link aponta para # relativo de onde está, gerando o erro abaixo se estiver em alguma das entidades do Mapas. erro_link_logo_instituicao

marylly commented 7 years ago

Correção disponível em http://master.map.as @thaisrigolon @GustavoVS @robertkapa

GustavoVS commented 7 years ago

@marylly quando coloquei no branch master basicamente quebrou.. creio que por outras coisas que estão no master.

Você pode colocar em algum outro branch mais 'limpo' ou criar um novo a partir do v2 (que é o que estamos usando) com as alterações?

marylly commented 7 years ago

@GustavoVS houve uma grande correção no core por causa dos arquivos da plataforma, acredito que agora será possível efetuar este teste.

GustavoVS commented 7 years ago

@marylly não conseguimos continuar os testes após colocar no master. Basicamente: Temas personalizados não carregam Temas customizados (SNIIC, Museus, Bibliotecas) carregam mas ficam exibindo url antes de carregar a página e ficam com a barra do menu estranha.

Gravei um pequeno vídeo para mostrar isso :D

https://www.youtube.com/watch?v=TSpE_RtPX1k&feature=youtu.be

marylly commented 7 years ago

@GustavoVS o erro que ocorre no Theme.php é porque é necessário executar a atualização de pacotes como mencionei na issue #69.

sudo apt-get install imagemagick libmagickcore-dev libmagickwand-dev php-imagick

thaisrigolon commented 7 years ago

Marylly, o Gustavo atualizou o branch RC do MinC com a execução dessa atualização de pacotes, mas não apareceram essas alterações para teste.

marylly commented 7 years ago

@thaisrigolon poderiam verificar essa issue, por gentileza?

marcus2703 commented 7 years ago

@marylly @thaisrigolon testei aqui a modificação. Ela corrigiu o erro reportado. Mas precisamos de um campo para definir a URL que será aberta ao clicar na imagem. Ela está abrindo uma nova janela do Mapas em questão (usando a url do subsite).

marylly commented 7 years ago

Campo disponível na aba Textos na seção Página Inicial. @thaisrigolon @marcus2703. Disponível na branch rc, sendo possível testar em http://rc.map.as

GustavoVS commented 7 years ago

@marylly está dando erro ao atualizar para o branch RC:

selecao_019

Antes do erro na página, também aparecem esses erros no deploy.sh (PHP Fatal error: Call to a member function getRevisionData() on a non-object in /srv/mapas/mapasculturais/src/protected/application/lib/MapasCulturais/Traits/EntityRevision.php on line 137):

selecao_020

thaisrigolon commented 7 years ago

Testamos no ambiente de teste e está ok. @marcus2703 , podemos fazer o deploy?

marcus2703 commented 7 years ago

Testei com sucesso a solução. Utilizei o servidor http://dev.saas.mapas.cultura.gov.br no teste de configuração. No teste utilizei a instância http://jaguarao.dev.saas.mapas.cultura.gov.br/ abriu com sucesso a URL configurada em uma nova janela.