Git Product home page Git Product logo

Comments (7)

cadu-leite avatar cadu-leite commented on August 15, 2024

fiz um pull request com uma view pra update dos dados.

sugestão ...
retirar aquele param do método save do form
e usar o user como named parameter
do contrário vai ter que rolar um teste na view.

def __init__(self, *args, **kargs):
    self.user = kargs.pop('user', None)
    self.commit = kargs.pop('commit', True)
    super(MembersForm, self).__init__(*args, **kargs)

caso o user não for passado como parametro, (caso o cara acesse a url de edição sem estar logado = user=anonymous) gera um erro de validação = sem user nem instancia o form.

from associados.

petry avatar petry commented on August 15, 2024

reabri esta issue, pois o comportamento ainda não está completo

from associados.

cadu-leite avatar cadu-leite commented on August 15, 2024

Olá @petry , diz o que ta faltando que eu revejo o código.

from associados.

petry avatar petry commented on August 15, 2024

Fala Cadu!

Então, já adicionei algumas coisas e modifiquei um pouco o seu código

Faltou proteger a rota por login, e atualizar os dados do usuário também, o form de membro tem somente os dados de associação, mas falta outros como nome, email, etc...

já criei um setup inicial de testes, seria interessante testar a edição em si, validados se os dados são editados de fato após o POST

from associados.

cadu-leite avatar cadu-leite commented on August 15, 2024

beleza,
é faltou ser um pouco mais pró-ativo.
tive problemas na instalação do requiremts de teste... eu grito se não conseguir.
não é preguiça de fazer testes! eu juro.

from associados.

petry avatar petry commented on August 15, 2024

vc está usando os comandos do makefile? o make deps deve instalar todas as dependencias, se tiver dando algum problema com o script, me manda o traceback que verifico aqui

from associados.

cadu-leite avatar cadu-leite commented on August 15, 2024

Em 10 de agosto de 2012 18:43, Marcos Daniel Petry <[email protected]

escreveu:

vc está usando os comandos do makefile? o make deps deve instalar todas
as dependencias, se tiver dando algum problema com o script, me manda o
traceback que verifico aqui

sim, rodei o make. te mando quando chegar em casa

Carlos Leite
ZNC Sistemas
+55 12 3905 24 05

from associados.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.