Merge Request #18

Closed
softwarepublico/colab!18
Created by Gustavo Jaruga Cruz

Proxy base template

Extract template and diazo rules to proxybase app

  • We believe that someone should do a strong review to re-ensure diazo rules and html are working as they should.
Milestone: None

Closed by Gustavo Jaruga Cruz

Changes were not merged into target branch

Commits (4)
3 participants
  • 9fe63c7bd60deeb55e409a1d7dd173f5?s=40&d=identicon
    Sergio Oliveira @seocam

    @rodrigosiqueiramelo, @darksshades estamos perdendo o histórico de alterações porque os arquivos não foram movidos usando git mv.

    Vejam:

    https://beta.softwarepublico.gov.br/gitlab/softwarepublico/colab/commits/667c50f0605902b63acdfba9d0e4c95cf4155fe1/colab/proxy/proxybase/views.py

    https://beta.softwarepublico.gov.br/gitlab/softwarepublico/colab/commits/master/colab/proxy/utils/views.py

    Este tipo de histórico é bem importante e principalmente em projetos abertos.

    Choose File ...   File name...
    Cancel
  • 9fe63c7bd60deeb55e409a1d7dd173f5?s=40&d=identicon
    Sergio Oliveira started a discussion on the outdated diff
    last updated by Sergio Oliveira
    colab/proxy/gitlab/templates/proxy/gitlab.html
    1   -{% extends 'base.html' %}
    2   -{% load static from staticfiles %}
    1
    • 9fe63c7bd60deeb55e409a1d7dd173f5?s=40&d=identicon
      Sergio Oliveira @seocam

      Porque esta linha foi removida? Nós ainda estamos usando a tag {% static ... %} no final deste template. Os "loads" de outros templates não são herdados.

      Choose File ...   File name...
      Cancel
    9fe63c7bd60deeb55e409a1d7dd173f5?s=40&d=identicon
    Sergio Oliveira started a discussion on the outdated diff
    last updated by Sergio Oliveira
    colab/proxy/proxybase/__init__.py
    ... ... @@ -0,0 +1,3 @@
    9fe63c7bd60deeb55e409a1d7dd173f5?s=40&d=identicon
    Sergio Oliveira started a discussion on the outdated diff
    last updated by Sergio Oliveira
    colab/proxy/proxybase/apps.py
      1 +from django.apps import AppConfig
      2 +
      3 +class ColabProxiedAppConfig(AppConfig):
      4 + colab_proxied_app = True
      5 +
      6 +class ProxyBaseAppConfig(ColabProxiedAppConfig):
    1
    • 9fe63c7bd60deeb55e409a1d7dd173f5?s=40&d=identicon
      Sergio Oliveira @seocam

      Aqui mais uma vez. Se a app não vai ser incluída não precisamos ter uma classe de App pra ela.

      Choose File ...   File name...
      Cancel
  • 9fe63c7bd60deeb55e409a1d7dd173f5?s=40&d=identicon
    Sergio Oliveira @seocam

    Como o template está sendo estendido sem que a app esteja nos settings ou sem setar o TEMPLATE_DIRS?

    Choose File ...   File name...
    Cancel
  • 9fe63c7bd60deeb55e409a1d7dd173f5?s=40&d=identicon
    Sergio Oliveira @seocam

    Na verdade acabei de ver que na verdade não temos o proxybase.html mas estamos estendendo ele. Os templates deveriam estender o base.html, não deveriam?

    Choose File ...   File name...
    Cancel
  • 9fe63c7bd60deeb55e409a1d7dd173f5?s=40&d=identicon
    Sergio Oliveira started a discussion on the outdated diff
    last updated by Sergio Oliveira
    colab/proxy/gitlab/templates/proxy/gitlab.html
    1   -{% extends 'base.html' %}
    2   -{% load static from staticfiles %}
      1 +{% extends 'proxybase.html' %}
    1
  • C6b14af78e51fba6beb90142971240cc?s=40&d=identicon
    Gustavo Jaruga Cruz @darksshades

    Mudanças realizadas(forced update).... o historico do git eu não sei se é possivel... ele está com o historico, mas somente mostra quando é utilizado o comando 'git log --follow filename'... a interface do gitlab não usa o follow e por isso não mostra... pelo que eu pesquisei não há uma forma de forçar ele a gerar o histórico unico pelo log normal, posso estar enganado.

    Choose File ...   File name...
    Cancel
  • 9fe63c7bd60deeb55e409a1d7dd173f5?s=40&d=identicon
    Sergio Oliveira @seocam

    O css está quebrado na home. Os ícones também não estão sendo carregados:

    [11/Dec/2014 15:18:16] "GET /static/third-party/third-party/bootstrap/fonts/glyphicons-halflings-regular.woff HTTP/1.1" 404 1823
    [11/Dec/2014 15:18:16] "GET /static/third-party/third-party/bootstrap/fonts/glyphicons-halflings-regular.ttf HTTP/1.1" 404 1820
    

    Pra ver os erros eu só rodei colab-admin runserver 0.0.0.0:8000.

    Choose File ...   File name...
    Cancel
  • C6b14af78e51fba6beb90142971240cc?s=40&d=identicon
    Gustavo Jaruga Cruz @darksshades

    Corrigido(forced_update), ele estava subindo as pastas erradas como mudou o local do .css... consertado agora

    Choose File ...   File name...
    Cancel
  • 08d286e4ed2d9bc51fc10057fc32a3d4?s=40&d=identicon
    Carlos Coêlho @carlos

    @rodrigosiqueiramelo @darksshades o css está quebrado nas outras páginas, rodei com o comando colab-admin runserver 0.0.0.0:8000

    Choose File ...   File name...
    Cancel
  • C6b14af78e51fba6beb90142971240cc?s=40&d=identicon
    Gustavo Jaruga Cruz @darksshades

    @carlospecter corrigido... faltou adicionar a div do namespace colab-style em alguns templates... creio que agora foram todos

    Choose File ...   File name...
    Cancel
  • 08d286e4ed2d9bc51fc10057fc32a3d4?s=40&d=identicon
    Carlos Coêlho @carlos
  • 9fe63c7bd60deeb55e409a1d7dd173f5?s=40&d=identicon
    Sergio Oliveira @seocam

    r-

    @darksshades @18oliveira.charles: A home continua quebrada. As caixas de texto estão todas para a esquerda exatamente como no último screenshot que mandei.

    Choose File ...   File name...
    Cancel
  • 9fe63c7bd60deeb55e409a1d7dd173f5?s=40&d=identicon
    Sergio Oliveira @seocam

    Reassigned to @darksshades

    Choose File ...   File name...
    Cancel
  • 9fe63c7bd60deeb55e409a1d7dd173f5?s=40&d=identicon
    Sergio Oliveira @seocam

    Coisas importantes ainda estão quebradas. Veja o anexo.

    Além da margem entre os títulos de todas as páginas (incluindo a home) os calendários também estão quebrados.

    Choose File ...   File name...
    Cancel
  • C6b14af78e51fba6beb90142971240cc?s=40&d=identicon
    Gustavo Jaruga Cruz @darksshades (Edited )

    Mexi um pouco na forma de resolver os conflitos que espero ter ficado melhor, o datepicker acima também arrumou.

    Choose File ...   File name...
    Cancel
  • C6b14af78e51fba6beb90142971240cc?s=40&d=identicon
    Gustavo Jaruga Cruz @darksshades

    Status changed to closed

    Choose File ...   File name...
    Cancel