14.5 Tanstack uloskirjautumissivulle | Antti/add-tanstack-library-logout-page #64
Conversation
…omething is happening
…now. This most likely will get changed
… antti/add-tanstack-library-logout-page
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
Tässä ei ollut oikein mitään kommentoitavaa. Pari muutosta jotka tein, ne ovat täsmälleen samoja kuin aiemmissa PR:issä Oliko näitä PR:iä liian paljon, olisiko ollut parempi hiukan yhdistellä vai oliko tämä ihan ok? 😅 |
samuliasmala
changed the base branch from
main
to
antti/add-tanstack-library-delete-and-edit-modals
| Lähetä | ||
| {isPending && ( | ||
| <span className="absolute ml-2 mt-1"> | ||
| <SvgSpinner | ||
| width={20} | ||
| height={20} | ||
| className="animate-spin" | ||
| /> | ||
| </span> | ||
| )} |
There was a problem hiding this comment.
Olisiko tähänkin hyvä laittaa pelkkä Spinneri, kun palautetta lisätään? Eikä Lähetäja sitten Spinneri.
Samanlainen toteutus löytyy DeleteModal.tsxstä ja EditModal.tsxstä
There was a problem hiding this comment.
Joo laittaisin tähänkin pelkän spinnerin.
Ei ollut liian paljon. On parempi kun PR:t on pieniä eli hyvä kun et yhdistellyt. |
Tämä pitäisi olla viimeinen PR liittyen Tanstackiin. Pilkoin ne kunnolla niin niistä ei kovin isoja tullut 😄 👍