Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Interface REST-like que retorna o status do associado (pesquisa por, CPF, e-mail) #4

Closed
petry opened this issue Jul 5, 2012 · 21 comments
Assignees
Labels

Comments

@petry
Copy link
Contributor

petry commented Jul 5, 2012

No description provided.

@valdergallo
Copy link
Contributor

Tem alguma idéia para esse ticket ? Autenticação com Basic-http ou OAuth ? Usando piston mesmo ?

@osantana
Copy link
Contributor

Vi que o Tastypie implementa vários tipos de autenticação... Dá uma olhada nele e, IMHO, implementa o que der menos trabalho.. afinal essa api servirá apenas para consumo interno dos outros sites da APyB (ex. o sistema de inscrição da PythonBrasil saber quem é associado para dar desconto)

@osantana
Copy link
Contributor

Essa funcionalidade será necessária para o site da PythonBrasil[9]

Não precisa estar no primeiro release mas seria legal se alguém trabalhasse nela... Nem que seja algo bem simples sem autenticação que retorne a situação do associado: /status/?email=fulano@foo.com -> {'status': 'ativo'}

@mondaini
Copy link
Contributor

mondaini commented Feb 1, 2013

Dá pra fazer essa issue em paralelo com a issue #5 que o @elyezer está terminando?

@elyezer
Copy link
Contributor

elyezer commented Feb 1, 2013

Pelo que entendi uma view somente resolveria o problema. Como estou fazendo um management command, não vejo nada que possa atrapalhar o desenvolvimento em paralelo.

@mondaini
Copy link
Contributor

mondaini commented Feb 2, 2013

Ah, blz. Vou dar uma olhada aqui.

@petry
Copy link
Contributor Author

petry commented Feb 2, 2013

@fmondaini se vc pegar esta tarefa, assine ela pra vc, assim não corremos o risco de de outras pessaos fazerem o mesmo trabalho :)

@mondaini
Copy link
Contributor

mondaini commented Feb 2, 2013

Pode fazer o assign pra mim? Parece que nao tenho a permissão aqui

@ghost ghost assigned mondaini Feb 2, 2013
@elyezer
Copy link
Contributor

elyezer commented Feb 6, 2013

@fmondaini teve algum avanço com a issue?

@mondaini
Copy link
Contributor

mondaini commented Feb 6, 2013

Tive avanços sim, @elyezer. Durante a semana ainda eu faço o pull request.

@elyezer
Copy link
Contributor

elyezer commented Feb 6, 2013

Está certo @fmondaini, só perguntei para ver se poderia ajudar em algo.

@mondaini
Copy link
Contributor

Estou com umas dúvidas:

  • Existe mais alguma informação que queiram incluir no json além de 'status'?
  • No admin, não dá pra incluir uma transaction manualmente. Isso é proposital? No mais acho que terminei.

A view volta 'ativo', 'inativo' ou 'inválido' para a busca com os parametros:
['first_name', 'last_name', 'cpf', 'phone', 'organization', 'email']

Exemplo:
http://localhost:8000/members/status/?first_name=Filipe

Resultado:
{'status': 'ativo'}

Peço desculpas pela demora em enviar o pull request :/

@mondaini
Copy link
Contributor

Outra pergunta: é necessário que esteja logado para acessar este recurso?

mondaini added a commit to mondaini/associados that referenced this issue Feb 11, 2013
@cadu-leite
Copy link
Contributor

na minha opinião não.

Em 10 de fevereiro de 2013 22:40, Filipe Mondaini
notifications@github.comescreveu:

Outra pergunta: é necessário que esteja logado para acessar este recurso?


Reply to this email directly or view it on GitHubhttps://github.com//issues/4#issuecomment-13365001..

Carlos Leite

@cadu_leite

ZNC Sistemas
http://www.znc.com.br

+55 12 3905 24 05

http://people.python.org.br/ http://pythonpeople.znc.com.br

mondaini added a commit to mondaini/associados that referenced this issue Feb 13, 2013
@petry
Copy link
Contributor Author

petry commented Feb 15, 2013

Fiz o merge @fmondaini, no entanto, as mensagens deveria sem em inglês com internacionalização, além do mais está tendo um problema de encode

Captura de Tela 2013-02-15 a s 10 24 17

Se conseguir resolver isso, será otimo :)

@mondaini
Copy link
Contributor

Blz, vou fazer os ajustes lá. ;)

Filipe Mondaini
www.mondaini.me http://www.mondaini.me

2013/2/15 Marcos Daniel Petry notifications@github.com

Fiz o merge @fmondaini https://github.com/fmondaini, no entanto, as
mensagens deveria sem em inglês com internacionalização, além do mais está
tendo um problema de encode

[image: Captura de Tela 2013-02-15 a s 10 24 17]https://f.cloud.github.com/assets/382581/160570/a74faa20-776a-11e2-90b6-507344da2109.png

Se conseguir resolver isso, será otimo :)


Reply to this email directly or view it on GitHubhttps://github.com//issues/4#issuecomment-13604309.

@osantana
Copy link
Contributor

Eu acho que as mensagens deveriam ser sempre em inglês, sem i18n. As interfaces REST de uma app são consumidas por máquinas e não por humanos, portanto, a i18n, nesse caso, mais atrapalha do que ajuda.

@mondaini
Copy link
Contributor

@petry / @osantana Então, com internacionalização ou sem (mas em inglês)?

@petry, o problema do encode é no retorno do json (ascii X utf-8)?

[]'s

Filipe Mondaini
www.mondaini.me http://www.mondaini.me

2013/2/15 Osvaldo Santana Neto notifications@github.com

Eu acho que as mensagens deveriam ser sempre em inglês, sem i18n. As
interfaces REST de uma app são consumidas por máquinas e não por humanos,
portanto, a i18n, nesse caso, mais atrapalha do que ajuda.


Reply to this email directly or view it on GitHubhttps://github.com//issues/4#issuecomment-13606651.

@petry
Copy link
Contributor Author

petry commented Feb 15, 2013

@fmondaini, em inglês, e retorna tudo com unicode que deve resolver o problema do encode.

@mondaini
Copy link
Contributor

blz!

mondaini pushed a commit to mondaini/associados that referenced this issue Feb 15, 2013
@mondaini
Copy link
Contributor

Feito

Filipe Mondaini
www.mondaini.me http://www.mondaini.me

On Fri, Feb 15, 2013 at 12:55 PM, Marcos Daniel Petry <
notifications@github.com> wrote:

@fmondaini https://github.com/fmondaini, em inglês, e retorna tudo com
unicode que deve resolver o problema do encode.


Reply to this email directly or view it on GitHubhttps://github.com//issues/4#issuecomment-13609983.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants