[Gastosabertos-dev] ao

Segunda Março 2 11:56:21 UTC 2015

Concordo completamente que servir arquivos estáticos daquela maneira não era
bom. Eu inclusive deixei um TODO na própria função dizendo que aquilo estava
feio.
Mas no momento da implementação não estava achando outro jeito simples (não
sabia que a API tinha uma pasta static dentro da receita), então fui naquele
mesmo temporariamente.

Edgar, eu acho que você manja muito e gosto muito de você como pessoa, mas só 
peço para maneirar um pouco nos tons dos comentários. Aqui na lista ainda está
mais de boa, mas no issue está agressivo.
Estou me esforçando ao máximo para tentar implementar as coisas do seu jeito
(que concordo que geralmente é melhor), mas nem tudo que está na sua cabeça é
trivial e transparente. Trivial no sentido de só ter uma implementação óbvia,
e transparente de ser fácil saber o que você está pensando. Tenho perguntado o
máximo que posso para você justamente para evitar fazer as coisas de outra
forma, mas nem sempre você responde e também não dá para perguntar linha a
linha...

Arrumei o que você falou. Aparentemente o erro de COR persiste mesmo com o
link funcionando:
http://demo.gastosabertos.org/receita/static/total_by_year_by_code/2014.json

Abs

Quoting Edgar Zanella Alvarenga (2015-03-02 07:51:12)
> Andres, você nao debugou o problema. Se d erro num request pra API,
> o Firefox indica erro de CORS. Mas pra saber se foi CORS, melhor ver
> a resposta ao request da API. Fazendo isso vi que o código estava
> dando um exception na linha 281 dum endpoint que você adicionou, que
> nunca deveia ter sido adicionado:
> 
> https://github.com/okfn-brasil/gastos_abertos/issues/132
> 
> Nunca faça um endpoint para servir arquivos estáticos! A não ser
> que tenha uma lógica extremamente complexa pra buscar os arquivos 
> baseado
> no request. No nosso caso precisamos apenas de arquivos estáticos
> nomeados pelo ano, nada mais do que isso.
> 
> Coloque os arquivos em data/total_by_year_by_code/ em algum local do 
> static
> do gastos_abertos no momento, e a requisição no view deverá ser 
> alterada de
> acordo. Só estou falando pra colocar lá para ser gerado novamente a 
> cada novo deploy
> da api pelo fab. Esqueça o que comentei no issue pra colocar no repo 
> gastos_abertos_website.
> 
> Abs,
> Edgar
> 
> On 02/03/2015 07:38, Andres MRM wrote:
> > Você diz a que fica em settings/local_settings.py?
> > Não mexi nisso no servidor, mas imagino que esteja, já que o 
> > DataTables está
> > conseguindo fazer o request normal...
> >
> >
> > Quoting Edgar Zanella Alvarenga (2015-03-02 07:26:06)
> >> Você está utilizando a configuração correta pro API_URL?
> >>
> >> On 02/03/2015 06:39, Andres MRM wrote:
> >> > Estou tentando fazer o drilldown funcionar no servidor, mas estou
> >> > tendo aquele
> >> > erro de COR:
> >> >
> >> >       Cross-Origin Request Blocked: The Same Origin Policy 
> >> disallows
> >> > reading the
> >> >       remote resource at
> >> >
> >> >       
> >> http://demo.gastosabertos.org/api/v1/receita/totaldrilldown?year=2014.
> >> > This
> >> >       can be fixed by moving the resource to the same domain or 
> >> enabling
> >> > CORS.
> >> >
> >> > Pelo visto o request do DataTables para a API está funcionando sem
> >> > problema,
> >> > só o meu que não. =(
> >> > Alguma ideia do porque?
> >> >
> >> > _______________________________________________
> >> > Gastosabertos-dev mailing list
> >> > Gastosabertos-dev em lists.okfn.org
> >> > https://lists.okfn.org/mailman/listinfo/gastosabertos-dev
> >>
> >> _______________________________________________
> >> Gastosabertos-dev mailing list
> >> Gastosabertos-dev em lists.okfn.org
> >> https://lists.okfn.org/mailman/listinfo/gastosabertos-dev
> > _______________________________________________
> > Gastosabertos-dev mailing list
> > Gastosabertos-dev em lists.okfn.org
> > https://lists.okfn.org/mailman/listinfo/gastosabertos-dev
> 
> _______________________________________________
> Gastosabertos-dev mailing list
> Gastosabertos-dev em lists.okfn.org
> https://lists.okfn.org/mailman/listinfo/gastosabertos-dev