Merge Request #23

Merged
softwarepublico/colab!23
Created by Gustavo Jaruga Cruz

Proxy data importation

Please merge https://beta.softwarepublico.gov.br/gitlab/softwarepublico/colab/merge_requests/18 first, as this assumes proxybase is already an app

Assignee: Sergio Oliveira
Milestone: None

Merged by Sergio Oliveira

Source branch has been removed
Commits (7)
2 participants
    9fe63c7bd60deeb55e409a1d7dd173f5?s=40&d=identicon
    Sergio Oliveira started a discussion on the outdated diff
    last updated by Sergio Oliveira
    colab/proxy/gitlab/data_api.py
      1 +from colab.proxy.gitlab.models import *
      2 +from colab.proxy.proxybase.proxy_data_api import ProxyDataAPI
      3 +from django.db.models.fields import DateTimeField
      4 +from dateutil.parser import parse
      5 +import urllib2
      6 +import json
      7 +from django.conf import settings
      8 +
      9 +class GitlabDataAPI(ProxyDataAPI):
      10 +
      11 +
      12 + def fetchProjects(self):
    1
    9fe63c7bd60deeb55e409a1d7dd173f5?s=40&d=identicon
    Sergio Oliveira started a discussion on the outdated diff
    last updated by Sergio Oliveira
    colab/proxy/proxybase/proxy_data_api.py
      1 +
      2 +class ProxyDataAPI():
    1
    • 9fe63c7bd60deeb55e409a1d7dd173f5?s=40&d=identicon
      Sergio Oliveira @seocam

      Em python 2 voce precisa extender object ProxyDataAPI(object). Se não fizer isso vai ter o comportamento das classes do Python

      Em python 3 não precisa mais extender object e nesse caso os parenteses nem são necessários.

      Choose File ...   File name...
      Cancel
    9fe63c7bd60deeb55e409a1d7dd173f5?s=40&d=identicon
    Sergio Oliveira started a discussion on the outdated diff
    last updated by Gustavo Jaruga Cruz
    colab/management/initconfig.py
    66 66 # PROXIED_APPS:
    67 67 # gitlab:
    68 68 # upstream: 'http://localhost:8090/gitlab/'
      69 +# auth_token: ''
    2
    • 9fe63c7bd60deeb55e409a1d7dd173f5?s=40&d=identicon
      Sergio Oliveira @seocam

      A url de request chama de private_token, a variavel que voces utilizam chama admin_token e nos settings auth_token. Porque nao usamos apenas um nome?

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

      Verdade... nem notei quando estava escrevendo

      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/jenkins/data_api.py
    ... ... @@ -0,0 +1,12 @@
  • 9fe63c7bd60deeb55e409a1d7dd173f5?s=40&d=identicon
    Sergio Oliveira @seocam

    Roda o flake8 nos arquivos modificados pra pegarmos os erros. Logo logo vamos integrar isso nos testes unitarios ;)

    Choose File ...   File name...
    Cancel