Closed gruble closed 2 weeks ago
Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-226.westeurope.5.azurestaticapps.net
Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-226.westeurope.5.azurestaticapps.net
Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-226.westeurope.5.azurestaticapps.net
Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-226.westeurope.5.azurestaticapps.net
Er det mulig å skille oppsummeringen i egen tabell under. For å skape litt mer sepperasjon. Ser litt uryddig ut i bunn.
Den er veldig bra! Digg med saker og PRer i oversikten :) Jeg lurte på om vi kanskje burde også ha en seksjon i selve komponenten? Eller mener du vi ikke trenger det? Vi kunne lage en separat PR på det hvis det krever mye jobb å bare ha den som en ekstra, ikke noe vi må ha. Jeg bare syns at folk kanskje vil hoppe direkte til komponenter istedenfor å se på oversikten, og det å se statusen kunne ha vært nyttig der og.
@amish1188 Enig! I Figma refereres det direkte til komponent.
Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-226.westeurope.5.azurestaticapps.net
Den er veldig bra! Digg med saker og PRer i oversikten :) Jeg lurte på om vi kanskje burde også ha en seksjon i selve komponenten? Eller mener du vi ikke trenger det? Vi kunne lage en separat PR på det hvis det krever mye jobb å bare ha den som en ekstra, ikke noe vi må ha. Jeg bare syns at folk kanskje vil hoppe direkte til komponenter istedenfor å se på oversikten, og det å se statusen kunne ha vært nyttig der og.
Ja, helt enig, @amish1188. Jeg vurderte det, men lot det være i første versjon for å få det viktigste på plass først. Jeg tror kanskje vi burde cache data fra Github hvis vi skal ha det, så vi ikke maser på Github hver gang vi viser en side til en komponent. Hva tror du? Det går an å sette filter på komponentnavn (label) i kallet til Github, så det er ikke så mye data som vil hentes uansett. Men det kan bli mange kall om folk klikker mye rundt
Veldig kult! Bra jobba. Lurer bare på noen figma ikoner er veldig små og noen forsvinner
@knutnve ja, bra du så det. Sleit litt med å få flexen til å funke som jeg ville, men trikset det til med å legge ikonet i egen tabell-kolonne
Er det mulig å skille oppsummeringen i egen tabell under. For å skape litt mer sepperasjon. Ser litt uryddig ut i bunn.
@knutnve enig. Har løst det nå ved å legge en "blank" rad mellom, så det ser ut som separate tabeller. Hvis de er i separate tabeller, blir ikke kolonnebreddene like
Den er veldig bra! Digg med saker og PRer i oversikten :) Jeg lurte på om vi kanskje burde også ha en seksjon i selve komponenten? Eller mener du vi ikke trenger det? Vi kunne lage en separat PR på det hvis det krever mye jobb å bare ha den som en ekstra, ikke noe vi må ha. Jeg bare syns at folk kanskje vil hoppe direkte til komponenter istedenfor å se på oversikten, og det å se statusen kunne ha vært nyttig der og.
Ja, helt enig, @amish1188. Jeg vurderte det, men lot det være i første versjon for å få det viktigste på plass først. Jeg tror kanskje vi burde cache data fra Github hvis vi skal ha det, så vi ikke maser på Github hver gang vi viser en side til en komponent. Hva tror du? Det går an å sette filter på komponentnavn (label) i kallet til Github, så det er ikke så mye data som vil hentes uansett. Men det kan bli mange kall om folk klikker mye rundt
Tenker du å cache det lokalt i nettleseren? Absolutt. Er alltid for å cache data der hvor det kan gjøres :). Kunne ha vært super enkelt hadde vi brukte tanstack query, men det kan vi snakke om en annen dag.
Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-226.westeurope.5.azurestaticapps.net
Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-226.westeurope.5.azurestaticapps.net
Vi viser nå alle åpne issues fra repoet vårt i komponentoversikten og på sida for hver komponent. Issues er i denne sammenheng både saker og pull requests.![image](https://github.com/user-attachments/assets/096e8878-f09d-43b2-bb97-a7db9c87991c)
En sak må merkes med komponentnavn for at vi skal vite at saken hører til en bestemt komponent.![image](https://github.com/user-attachments/assets/bd23c93d-70f5-43a5-8eee-315d87e400b8)
NB! Alle issues merka med komponentnavn blir vist. Det samme gjelder PR'er. PR'er som ikke er merka med komponentnavn, men er koblet til en komponent gjennom en sak, vises ikke. Årsaken er at endepunktet jeg bruker, gir ingen kobling mellom sak og PR. Dette kan vi sikkert få til på sikt, men det krever mer koding. Her er dokumentasjon på endepunktet jeg bruker: https://docs.github.com/en/rest/issues/issues?apiVersion=2022-11-28#list-repository-issues
Jeg har også forenklet koden for å summere antall komponenter og forsøkt å gjøre oversikten litt penere, bla.a. ved å bruke nve-tag i stedet for nve-badge.