uspdev / web-ldap-admin

https://gitlab.uspdigital.usp.br/uspdev/web-ldap-admin
3 stars 10 forks source link

Corrigir caminhos para hospedar sistema em pasta #91

Closed masakik closed 4 years ago

masakik commented 4 years ago

Se o sistema estiver hospedado em /usuarios os caminhos dos menus vão apontar para a raiz do servidor.

Nesse caso vi que precisa ajustar o config/laravel-usp-theme.php e no master.blade.php incluir a tag 'base'. Mas isso seria no laravel-usp-theme e não sei o que isso implicaria em outros sistemas que usam ele.

thiagogomesverissimo commented 4 years ago

Não impactaria não, pois o config/laravel-usp-theme.php tá no escopo da instância

masakik commented 4 years ago

mas o master.blade.php ele copia do src e é onde incluiria a tag base. De fato tem outras formas de adequar que seria colocando o APP_URL na frente. Não sei qual seria melhor.

thiagogomesverissimo commented 4 years ago

Fazendo em config/laravel-usp-theme.php precisa mudar master.blade.php ?

masakik commented 4 years ago

vou tentar mexendo somente no config e outros arquivos desse projeto

thiagogomesverissimo commented 4 years ago

Beleza, se precisa mexer no master.blade.php sem problemas também. Afinal, se você prefixar tudo com {base} não vai quebrar quem usar domínio ou subdiretório.

masakik commented 4 years ago

Eu subi uma branch issue_91 que tem as correções para alguns casos. Tem como @thiagogomesverissimo ou @alecostaweb testar por aí? Ciomo tem a issue #90 não consigo entrar na página /ldapusers para testar. Eu usei a função url() nas blades que deve corrigir os caminhos. Se estiver tudo certo eu posso aplicar em todos mas meio às cegas por não ter como testar totalmente.

thiagogomesverissimo commented 4 years ago

Vamos corrigir isso (urls) de vez no laravel-usp-theme?

thiagogomesverissimo commented 4 years ago

Você corrigiu essa issue no último PR. Podemos de qualquer modo discutir os urls fornecidas pelo laravel-usp-theme