Origin/restruturacao django

[viniarck]

Segue a solicitacao para realizar o pull request. Este commit que eu fiz foi para fechar a issue#5.

Na pratica eu fiz muito mais que a issue#5, aproveitei para estruturar melhor o codigo e comecei algumas views por exemplo a do carroceiro que esta relacionado a issue#1.

Na sequencia eu gostaria de conversar com voces sobre a issue#1, porque por exemplo, quando eu estava fazendo o modelo eu fiquei em duvida se o "phone" poderia ser UNIQUE, mas eu vi que na nossa database.sql um mesmo catador tinha dois enderecos, entao eu imagino que ele poderia ter o endereco da casa e o "atual do GPS (isto eh se no futuro existir o GPS para o carrinho)". Com base nisso (considerando as restricoes e o que poderia ou nao poderia, por exemplo nome null etc..

), eu fiz uma classe inteira para executar os testes de integracao (ja que nao temos a APIClient/consumer eu utilizer o requests do python para simular um client) e todos estes passaram (ou seja a view que eu fiz esta checkando todos as exceptions e etc e esta funcionando 100%):

(venv)arcanjo@ubuntu:~/PycharmProjects/pimpapp-api$ py.test integration_tests/tests.py
=========================================================================================== test session starts ===========================================================================================
platform linux -- Python 3.4.3+, pytest-2.8.2, py-1.4.30, pluggy-0.3.1
rootdir: /home/arcanjo/PycharmProjects/pimpapp-api/integration_tests, inifile:
collected 10 items

integration_tests/tests.py ..........

======================================================================================== 10 passed in 0.74 seconds ========================================================================================
(venv)arcanjo@ubuntu:~/PycharmProjects/pimpapp-api$

Segue em anexo algumas screenshots tambem, para voces ver que o JSON ja esta sendo recuperado

[Vido]

@viniciusarcanjo,

Sensacional. Você foi muito além, parabéns pelo trabalho. Acredito que isso feche o issue #5.
Pelo que eu vi aqui, você já conhece bem o Django. Você já chegou atropelando.
Você fez documentação, teste... Muito bom.

Mas eu vi que você adotou o numero do telefone como ID. Me parece uma boa opção. Porque até onde eu sei, os catadores da nossa base têm celular. Por outro lado, essa informação é meio volátil. Porque as pessoas perdem os celulares, trocam o número ou são roubadas com bastante frequência.

Existem também neste código questões que estamos discutindo no issue #1.
Vamos trocar uns email para discutir, assim o @fernandobalm também fica por dentro.

[fernandobalm]

Prezados,

Como eu imaginava, não estou conseguindo tempo para colaborar no projeto antes de 30/11. Porém se desejarem trocar e-mails para discutir as questões propostas estou à disposição.

Tenho acompanhado pelos e-mails o grande progresso do Vinícius. Fico feliz que tudo esteja fluindo e ele esteja avançando tão bem.

Sobre a questão que vocês levantaram, se é que eu entendi pela explicação dada no e-mail, talvez realmente o celular como identificador único (ou chave primária de acesso) não seja interessante pela mutabilidade da informação, como mencionou o Lucas, além de possíveis mudanças que a ANATEL possa vir a fazer (como foi o caso dos 9s no início) e também porque talvez seja possível 2 celulares de diferentes DDDs terem o mesmo número (para fixos isso é possível), a menos que se inclua o DDD no ID. Acho que seria melhor um ID interno para identificar os carroceiros ou então alguma informação que eles tivessem e não fosse sujeita a mudança.

Qualquer questão basta falarem.

Abraços!

Fernando Barreto de Almeida
mailto: [email protected]
http://fernandobalm.xpg.com.br
Telefone: 55 - 11 - 33410224
São Paulo - SP - Brasil

From: Lucas Vido
Sent: Monday, November 2, 2015 10:50 PM
To: PimpAPP/pimpapp-api
Cc: Fernando Barreto de Almeida
Subject: Re: [pimpapp-api] Origin/restruturacao django (#7)

@viniciusarcanjo,

Sensacional. Você foi muito além, parabéns pelo trabalho. Acredito que isso feche o issue #5.
Pelo que eu vi aqui, você já conhece bem o Django. Você já chegou atropelando.
Você fez documentação, teste... Muito bom.

Mas eu vi que você adotou o numero do telefone como ID. Me parece uma boa opção. Porque até onde eu sei, os catadores da nossa base têm celular. Por outro lado, essa informação é meio volátil. Porque as pessoas perdem os celulares, trocam o número ou são roubadas com bastante frequência.

Existem também neste código questões que estamos discutindo no issue #1.
Vamos trocar uns email para discutir, assim o @fernandobalm também fica por dentro.

—
Reply to this email directly or view it on GitHub.

---

Este email foi escaneado pelo Avast antivírus.
https://www.avast.com/antivirus