search

data-students / apunts-dades

https://apunts-dades.cat
12 stars 0 forks source link

34 feature canviar text box per un botó #53

Closed marcfranquesa closed 5 months ago

marcfranquesa commented 5 months ago

Ja que estava he ficat els dos butonets, he optat per color claret que ja està massa saturat del fosc.

image
tidb-cloud-branching[bot] commented 5 months ago
The latest updates on TiDB Serverless Branch. Learn more about branching GitHub integration↗︎ BranchName ClusterID Status Updated
34-feature-canviar-text-box-per-un-botó_53_3cc0527 10034588272270804309 ACTIVE 2024-03-11T16:06:16Z
vercel[bot] commented 5 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
apunts-dades ✅ Ready (Inspect) Visit Preview Mar 11, 2024 4:05pm
marcfranquesa commented 5 months ago

@PauMatas @puigde m'agradaria squashejar commits, crec que no puc activar-ho, algun admin ha de permetre-ho. Sino ja faré un squash manual abans de fer el merge.

PauMatas commented 5 months ago

@PauMatas @puigde m'agradaria squashejar commits, crec que no puc activar-ho, algun admin ha de permetre-ho. Sino ja faré un squash manual abans de fer el merge.

Si no vaig errat (rectifica'm @puigde) diria que només nosaltres dos podíem fer merge a master fins ara.

També et faltava cambiar la PR de draft a PR PR, ja t'ho he fet, i al ferho ha sortit la opció de fer squash and merge.

marcfranquesa commented 5 months ago

@PauMatas @puigde m'agradaria squashejar commits, crec que no puc activar-ho, algun admin ha de permetre-ho. Sino ja faré un squash manual abans de fer el merge.

Si no vaig errat (rectifica'm @puigde) diria que només nosaltres dos podíem fer merge a master fins ara

No se si parlem del mateix, hi ha una opció per "squashejar" commits, és a dir, quan es faci merge, que nms aparegui un sol commir (en comptes dels meus 3 + el merge). Però per fer-ho crec que heu d'activar la opció de poder fer squashos.

PauMatas commented 5 months ago

Nono, ja està activada segur @marcfranquesa pq amb la resta de prs ho hem fet

marcfranquesa commented 5 months ago

aaah val merci, que noob sorry

puigde commented 5 months ago

@PauMatas @marcfranquesa el package-lock.json s'ha creat perquè s'haurà instal·lat alguna cosa utilitzant npm en comptes de yarn. Intentem tenir-ho tot únicament amb yarn en futures ocasions així ens estalviem possibles problemes.