Skip to content

Conversation

@Achalogy
Copy link
Member

Funciona 😎

2025-10-12.17-15-35.mp4

@adrianrrruiz
Copy link
Contributor

Esta bueno, peeero lo de que este haciendo requests innecesarias no me cuadra tanto. Estaba buscando y con el cliente de supabase uno se puede suscribir a cambios que pueda tener la tabla "results". Acá hay un ejemplo:

const [rows, setRows] = useState<any[]>([]);

  useEffect(() => {
    // Cargar datos iniciales
    const load = async () => {
      const { data } = await supabase.from("clientes").select("*");
      setRows(data || []);
    };
    load();

    // Escuchar cambios en la tabla
    const channel = supabase
      .channel("public:clientes")
      .on(
        "postgres_changes",
        { event: "*", schema: "public", table: "clientes" },
        (payload) => {
          console.log("Cambio detectado:", payload);
          load(); // recargar lista
        }
      )
      .subscribe();

    return () => {
      supabase.removeChannel(channel);
    };
  }, []);

Copy link
Contributor

@adrianrrruiz adrianrrruiz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

El cliente de supabase lo puedes encontrar en src/lib/supabase

@adrianrrruiz
Copy link
Contributor

@Samu-Kiss que dices del diseño?

@adrianrrruiz adrianrrruiz linked an issue Oct 13, 2025 that may be closed by this pull request
@Achalogy
Copy link
Member Author

está chistoso eso del subscribe, ya lo reviso

@Samu-Kiss
Copy link
Contributor

Samu-Kiss commented Oct 13, 2025

@Samu-Kiss que dices del diseño?

que pena la demora con esto, han sido semanas complicadas. anyways, para el diseño estaba pensando en algo así

image

los numeritos irian en bc-liguria y los otros textos en montserrat :)

@Achalogy
Copy link
Member Author

Estuve revisando lo del update, hay que montar al postgres unos triggers y hacer un monton de cosas, lo voy a revisar bien antes de hacer algo raro.

@Achalogy
Copy link
Member Author

@Samu-Kiss revisa el diseño en la web

@adrianrrruiz
Copy link
Contributor

image Toca que si el avatar no carga, cargue una imagen por defecto. Creo que así lo tenías antes pero se daño @Achalogy

@Achalogy
Copy link
Member Author

Achalogy commented Nov 4, 2025

solo falta el valor de la imagen en el .env

@adrianrrruiz
Copy link
Contributor

Vale, entonces toca que agregues en el .env.example cómo le llamaste a la variable. Lo otro para saber es si revisaste lo del suscribe con supabase para lo cambios en el ranking? o la otra es usar el servicio de realtime de supabase @Achalogy

Copy link
Member

@TalkySafe143 TalkySafe143 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Esta con supabase realtime, solo puede existir una suscripción activa.

@TalkySafe143 TalkySafe143 merged commit 1159b1e into develop Jan 7, 2026
2 checks passed
@TalkySafe143 TalkySafe143 deleted the feat/update-ranking-scoreboard branch January 7, 2026 06:59
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

Successfully merging this pull request may close these issues.

Arreglar diseño del ranking y actualizar el scoreboard

5 participants