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

Fixes necesarios del ABM de Afiliados #1

Open
Eourist opened this issue Jun 22, 2020 · 0 comments
Open

Fixes necesarios del ABM de Afiliados #1

Eourist opened this issue Jun 22, 2020 · 0 comments
Assignees
Labels
bug Something isn't working

Comments

@Eourist
Copy link
Owner

Eourist commented Jun 22, 2020

Dejo acá las cosas que hacen falta cambiar en la vista:

  1. Validar que los campos no sean null: Cuando presiono buscar pero no hay nada en el campo DNI, salta una NullPointerException porque intenta convertir "" a int. Lo mismo pasa al intentar hacer el alta o la eliminación, tanto con el campo DNI como con el nombre.
  2. Borrar el campo ID. Si la búsqueda es por DNI, no hace falta que el usuario pueda ver ni usar el ID del Afiliado.
  3. Podría agregarse un desplegable con todos los afiliados como método de búsqueda alternativa, para que no tengas que andar buscando los DNI siempre que querés modificar/eliminar un Afiliado.
  4. El botón guardar y modificar podrían ser el mismo, que en BD se haga el alta o la update según halla un Afiliado seleccionado o no. Para esto sería mejor que la selección de afiliado sea desde un ComboBox en vez del campo DNI, así sería mas fácil determinar cuando se trata de una edición o de un alta.
  5. Algún tipo de alerta o mensaje para que el usuario sepa que esta pasando. Si ocurre un error, que se muestre de alguna forma, sino no hay forma de saber si las cosas están funcionando o no.
@Eourist Eourist added the bug Something isn't working label Jun 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants