Create new searchbar and UsersView requests #128

wbuezas · 2025-03-20T14:01:05Z

wbuezas commented

2025-03-20 14:01:05 +00:00

No description provided.

wbuezas added 1 commit 2025-03-20 14:01:05 +00:00

gitea/hedera-web/pipeline/pr-beta This commit looks good Details

c24480099f Create new searchbar and UsersView requests

jsegarra requested changes 2025-03-21 14:14:23 +00:00

Dismissed

jsegarra left a comment

Me parece bien la propuesta de la nueva VnSearchbar, pero una pregunta: será retrocompatible con el resto de la app? Todas las busquedas que hacemos ya atacan a salix o queda alguna con sql?

Mi propuesta es dejar de momento todas las propiedades y si existe sqlQuery, llamar a jApi y sino a api

Me parece bien la propuesta de la nueva VnSearchbar, pero una pregunta: será retrocompatible con el resto de la app? Todas las busquedas que hacemos ya atacan a salix o queda alguna con sql? Mi propuesta es dejar de momento todas las propiedades y si existe sqlQuery, llamar a jApi y sino a api

wbuezas commented

2025-03-21 18:14:20 +00:00

Me parece bien la propuesta de la nueva VnSearchbar, pero una pregunta: será retrocompatible con el resto de la app? Todas las busquedas que hacemos ya atacan a salix o queda alguna con sql?

Mi propuesta es dejar de momento todas las propiedades y si existe sqlQuery, llamar a jApi y sino a api

Claro, por eso cree un nuevo componente, el anterior sigue existiendo y se sigue usando en las otras vistas.
Cuando todas las vistas utilicen este nuevo componente borramos el anterior.

> Me parece bien la propuesta de la nueva VnSearchbar, pero una pregunta: será retrocompatible con el resto de la app? Todas las busquedas que hacemos ya atacan a salix o queda alguna con sql? > > Mi propuesta es dejar de momento todas las propiedades y si existe sqlQuery, llamar a jApi y sino a api Claro, por eso cree un nuevo componente, el anterior sigue existiendo y se sigue usando en las otras vistas. Cuando todas las vistas utilicen este nuevo componente borramos el anterior.

wbuezas requested review from jsegarra 2025-03-23 20:22:34 +00:00

jsegarra commented

2025-03-23 22:30:41 +00:00

Me parece bien la propuesta de la nueva VnSearchbar, pero una pregunta: será retrocompatible con el resto de la app? Todas las busquedas que hacemos ya atacan a salix o queda alguna con sql?

Mi propuesta es dejar de momento todas las propiedades y si existe sqlQuery, llamar a jApi y sino a api

Claro, por eso cree un nuevo componente, el anterior sigue existiendo y se sigue usando en las otras vistas.
Cuando todas las vistas utilicen este nuevo componente borramos el anterior.

Excelente, lo tenemos controlado.
Comentario: en lilium cuando hacemos algo así le ponemos el sufijo de Beta

> > Me parece bien la propuesta de la nueva VnSearchbar, pero una pregunta: será retrocompatible con el resto de la app? Todas las busquedas que hacemos ya atacan a salix o queda alguna con sql? > > > > Mi propuesta es dejar de momento todas las propiedades y si existe sqlQuery, llamar a jApi y sino a api > > Claro, por eso cree un nuevo componente, el anterior sigue existiendo y se sigue usando en las otras vistas. > Cuando todas las vistas utilicen este nuevo componente borramos el anterior. Excelente, lo tenemos controlado. Comentario: en lilium cuando hacemos algo así le ponemos el sufijo de Beta

👍 1

jsegarra added 1 commit 2025-03-23 22:30:45 +00:00