Added played matches for a player

[pechitook]

👏 No lo probé a nivel UI pero el código va genial. Gracias @demianalonso!

[demianscv]

@p4bloch a nivel UI le va a volar el bocho a coco. Se lo va a volar tanto que se va a decidir a ayudarnos para que no le sangren los ojos cuando lo use.

[julioolvr]

Se lo va a volar tanto que se va a decidir a ayudarnos para que no le sangren los ojos cuando lo use.

No era joda 🙈 Jajaja

Igual posta que es super útil. Dos cosas rápido que se me ocurren para que sea más piola sin demasiado laburo:

• Que el cursor se ponga en modo link al pasar sobre las filas de la tabla (no me había dado cuenta que se podían clickear y no me daba cuenta cómo hacerlo andar).
• Que cada fila de la tabla nueva tenga un color diferente según si el partido lo ganó/empató/perdió el jugador seleccionado.

[demianalonso]

Le cambié lo del iconito para que se note que la fila es clickeable y los estilos para los partidos ganados/perdidos/empatados/no jugados.

Lo del layout lo quise hacer pero el GridList que tiene material-ui me parece que es para otra cosa. Como que no te deja toquetear mucho como se muestra, por ejemplo, todos los tiles tienen el mismo tamaño. Lo cual queda aún más feo cuando la lista de partidos es igual de grande que la de resultados y tiene el cuarto de datos.

También estoy interesado en que @p4bloch se explaye sobre lo de graphql que no conozco mucho y simplemente lo hice funcionar :D

[pechitook]

En cuanto al diseño, podemos fixear ese div al alto de la tabla y hacerlo overflow-y: auto?

No tocaste nada de GraphQL! Dashboard es quien hace una query juntando los fragments que tienen Table y MaxPoints, y les pasa la data por props cuando las tiene. WE'RE GOOD

[demianalonso]

Justamente @p4bloch la pregunta de Julio es si no hacía falta declarar los datos que usa. Aunque capaz simplemente al ser un componente dependiente del Table, entonces si su padre los declara, estamos bien. Eso es lo que queremos asegurarnos.

Por otro lado, no lo hice fixed porque no quise jejeje Estaba pensando en que quizás cuando no hay un jugador seleccionado, ahí se pueden mostrar TODOS los partidos posibles. Aunque capaz no tiene sentido y hay que pensar un poco algo de como debería verse. O secuestrar a coco un rato.

[julioolvr]

Con lo del diseño estoy de acuerdo en calentarnos el mínimo posible hasta que verdaderamente lo pensemos y ahí vemos qué armamos.

Lo de GraphQL es lo que dice @demianalonso, si bien funciona no sé qué es lo correcto, si que PlayerMatches tenga esa dependencia implícita en los datos que pide el padre o que declare su propio fragment, más allá de que la query final no traiga ningún dato nuevo.

[pechitook]

☝️ tiene sentido y es correcto lo que dice Julito, la idea sería que PlayerMatches defina un fragment y lo use su padre para finalmente pasárselo a Dashboard. Habría que investigar un poco a ver cómo se hace eso, si es mucho bardo podemos mergearlo así y ningún gatito saldrá lastimado

[pechitook]

@julioolvr @demianalonso qué dicen?

[julioolvr]

Merge