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

Binding da propriedade name - Dynamic Forms - Lookup #105

Closed
isiqueira opened this issue Oct 3, 2019 · 5 comments
Closed

Binding da propriedade name - Dynamic Forms - Lookup #105

isiqueira opened this issue Oct 3, 2019 · 5 comments

Comments

@isiqueira
Copy link
Contributor

No componente Dynamic Form, quando adicionado um componente Lookup a propriedade name informada não é respeitada.

Verifiquei o arquivo po-dynamic-form-fields.component.html na linha 107, não está sendo feito o binding com a variável e sim atribuindo valor a propriedade name.
PS: há alguma documentação sobre como abrir uma PR?

@jhonyeduardo
Copy link
Contributor

Boa tarde @isiqueira

Analisamos e isto ocorre realmente,
Vamos adicionar este item em nosso backlog para uma correção futura.

Temos sim, veja o documento CONTRIBUTING

Qualquer dúvida estamos a disposição.

@isiqueira
Copy link
Contributor Author

Obrigado @jhonyeduardo. Parabéns aos envolvidos pelo ótimo projeto.

@isiqueira
Copy link
Contributor Author

@jhonyeduardo como era uma mudança simples, eu tomei a liberdade de mandar uma PR com a mudança, me desculpe caso não tenha seguido as regras.

@jhonyeduardo
Copy link
Contributor

@isiqueira Excelente iniciativa!

Vamos revisar a sua PR e qualquer coisa entramos em contato, mas creio que seja realmente o que voce fez,

Obrigado pela contribuição

@jhonyeduardo
Copy link
Contributor

Parabéns pela primeira contribuição aprovada!!

Esperamos que possa sempre contribuir conosco!

Abraço.

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

No branches or pull requests

3 participants