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

Nombre por defecto de espectros que sea real #67

Open
facundoq opened this issue Jul 12, 2022 · 5 comments
Open

Nombre por defecto de espectros que sea real #67

facundoq opened this issue Jul 12, 2022 · 5 comments

Comments

@facundoq
Copy link
Member

Está implementado como que si el nombre es "" entonces se muestra nuevo#indice. Cambiar eso por directamente ponerle el nombre Nuevo#1 y listo.

Entiendo que lo usan para validar. En lugar de eso, en la validación, fijarse si el nombre es de formato "Nuevo#{num}" y si es así declarar ese campo como inválido.

@joaquinmir
Copy link
Collaborator

El tema es que hacerlo asi genera unos problemas con la actualizacion de los componentes, es mas por eso que por la validacion.

@facundoq
Copy link
Member Author

facundoq commented Jul 13, 2022 via email

@joaquinmir
Copy link
Collaborator

No es un problema con la logica de la app, digo internamente no hay problema con iniciar los espectros con Nuevo#x, el tema es que con los componentes se complica la cosa. Por ejemplo si tenes Nuevo#1, Nuevo#2 y Nuevo#3. Si borro Nuevo#2 con esa logica me quedaria Nuevo#1, Nuevo#3. Y para hacer que quede bien necesitaria mas logica que lo haria menos eficiente. Ademas quedaria en el input del form Nuevo#1 por ejemplo y no se si es lo mejor, de ultima lo vemos mañana.

@facundoq
Copy link
Member Author

facundoq commented Jul 14, 2022 via email

@joaquinmir
Copy link
Collaborator

Ya hice el cambio, despues checkeenlo porque como comentaba se pueden dar casos en los que se repitan los nombres.

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

2 participants