Connections view #130

wbuezas · 2025-03-25T14:24:21Z

wbuezas commented

2025-03-25 14:24:21 +00:00

PR relacionada de salix: verdnatura/salix#3597

PR relacionada de salix: https://gitea.verdnatura.es/verdnatura/salix/pulls/3597

wbuezas added 1 commit 2025-03-25 14:24:22 +00:00

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

b691bf2cf4 Connections view

wbuezas requested review from jsegarra 2025-03-25 14:25:19 +00:00

jsegarra requested changes 2025-03-25 22:49:35 +00:00

Dismissed

src/pages/Admin/ConnectionsView.vue Outdated

						
				@ -40,0 +72,4 @@

				            return;

				        }

				        const formattedConnections = data

jsegarra commented

2025-03-25 22:32:24 +00:00

Esto se podría simplificar
connections.value = data .map(({ visitUser = {}, ...rest }) => ({ ...rest, user: visitUser.user, stamp: visitUser.stamp, visitAgent: visitUser.visitAccess?.visitAgent })) .filter(({ user }) => user);

Esto se podría simplificar `connections.value = data .map(({ visitUser = {}, ...rest }) => ({ ...rest, user: visitUser.user, stamp: visitUser.stamp, visitAgent: visitUser.visitAccess?.visitAgent })) .filter(({ user }) => user); `

wbuezas commented

2025-03-26 08:32:16 +00:00

Aplicado.

Commit: d9445f72e3

Aplicado. Commit: https://gitea.verdnatura.es/verdnatura/hedera-web/commit/d9445f72e3353d6b7d072c9a8add59a9447311b1

jsegarra commented

2025-03-28 08:22:32 +00:00

En loopback se pueden crear scopes, de esta manera, no haria falta pasarle el filter.
Ahora ya está fusionada en salix, pero bueno, lo dejo como comentario a futuro

En loopback se pueden crear scopes, de esta manera, no haria falta pasarle el filter. Ahora ya está fusionada en salix, pero bueno, lo dejo como comentario a futuro

src/pages/Admin/ConnectionsView.vue Outdated

						
				@ -109,13 +162,14 @@ onBeforeUnmount(() => clearInterval(intervalId.value));

				                    >

				                    <span

				                        v-if="

jsegarra commented

2025-03-25 22:34:38 +00:00

Esto se podria simplificar, porque el uso x6 de connection.visitagent no mola
Seria una funcion en el v-if y otra funcion a la que le pases connection y te lo devuelva formateado
Otra opcion es
const renderAgentDetails = (connection) => { const agent = connection.visitAgent; return agent?.platform && agent?.browser && agent?.version ? h('span', ${agent.platform} - ${agent.browser} - ${agent.version}) : null; };

<component :is="renderAgentDetails(connection)" />

Esto se podria simplificar, porque el uso x6 de connection.visitagent no mola Seria una funcion en el v-if y otra funcion a la que le pases connection y te lo devuelva formateado Otra opcion es `const renderAgentDetails = (connection) => { const agent = connection.visitAgent; return agent?.platform && agent?.browser && agent?.version ? h('span', `${agent.platform} - ${agent.browser} - ${agent.version}`) : null; };` ` <component :is="renderAgentDetails(connection)" />`

wbuezas commented

2025-03-26 08:38:11 +00:00

Aplicado.

Commit: 365d4d31e1

Aplicado. Commit: https://gitea.verdnatura.es/verdnatura/hedera-web/commit/365d4d31e105d5ccb748c046ce01b3db53eddc45