Pakettien päivittäminen | antti/update-packages #60

anttiasmala · 2024-12-25T14:26:47Z

No description provided.

…or now, will move it better location if needed later on

…ded. Add placeholder values for loading and error

… Finnish. Loading dot loop taken here: https://stackoverflow.com/a/40991531

…omponent due to no need in Home function

…used for development for now

…g things

…s serverless

…5 seconds. Make it 3 dots long and make it that it has same amount of time with each dot so it looks better

…d of variables imported from giftQuery itself

…eated useGetGifts

…able

…tData in /pages/index.tsx

…dateQueries

…ts query instead

… queryClient's invalidateQueries but it shows all the possible keys that can be invalidated

…t is inside Home component

…SingleQueryKey for now. This most likely will be changed in the future. Changes in EditModal.tsx and DeleteModal.tsx

…t seem to be updated anymore. Reference: mysticatea/eslint-plugin-node#360

…ind any breaking changes information regarding it so could not test

vercel · 2024-12-25T14:26:52Z

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

Name	Status	Preview	Comments	Updated (UTC)
lahjalista	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 15, 2025 2:12pm

…t. Everything worked perfectly fine with Prisma 6 but for updating packages --force had to be used

.eslintrc.json

anttiasmala · 2024-12-25T14:44:16Z

.eslintrc.json

+      "plugins": ["@tanstack/query"],
      "extends": [
        "plugin:@typescript-eslint/recommended",
        "plugin:@typescript-eslint/recommended-requiring-type-checking",
-        "plugin:import/typescript"
+        "plugin:import/typescript",
+        "plugin:@tanstack/eslint-plugin-query/recommended"


Liittyy tanstack-brancheihin, jonka pohjalta väsäsin tätä branchia

.eslintrc.json

…revent n/no-extraneous-import for now. The rule can be disabled if needed but let's go with this for now

…eprecated .eslintrc.json method and replace it with the upcoming Flat Config method

…s to do when updating packages

anttiasmala · 2025-01-15T14:08:17Z

INFO.md

 package.json

-PACKAGES NOT UPDATED, NOT WORKING AT THE MOMENT 19.09.2024
-
-Minor Backwards-compatible features
-❯ ◯ typescript 5.4.5 → 5.6.2
+Seuraavia paketteja ei ole päivitetty

 Major Potentially breaking API changes
-◯ @types/node 20.12.4 → 22.5.5
-◯ @typescript-eslint/eslint-plugin 7.5.0 → 8.6.0
-◯ eslint 8.57.0 → 9.10.0
+❯ ◯ @prisma/client 5.20.0 → 6.2.1
+◯ @types/node 20.12.4 → 22.10.6
+◯ prisma 5.20.0 → 6.2.1
+
+`@prisma/client`iä ja `prisma`a ei ole päivitetty, koska `lucia-auth`in peer-dependency ei tue vielä Prisma v6:sta. Kyseisestä asiasta on tehty PR https://github.com/lucia-auth/lucia/pull/1760 jos tuota ei ole korjattu helmikuuhun mennessä, korjataan itse tai keksitään jokin muu ratkaisu
+
+`@types/node`a ei ole päivitetty, koska pitänee seurata package.json:in node-versiota
+
+#########################################################################
+
+**Testaa näitä päivittämisen yhteydessä**
+
+- [] poista oma testaustietokanta ja lisää tietokanta uudelleen `npm run prisma db push`
+
+- [] kokeile ESLint ettei heitä virheitä
+
+- [] testaa sivu että normaalikäyttö toimii (rekistöröinti, kirjautuminen, lahjan (lisääminen, poistaminen, muokkaaminen), palaute)
+
+- [] API-endpointtien testaaminen


Vaihdoin tekstin tässä tiedostossa vastaamaan nykyistä

Lisäsin myös TODO-listan pakettien päivittämisen jälkeen. Helpompi muistaa

anttiasmala · 2025-01-15T14:10:03Z

api.rest

@@ -30,6 +30,7 @@ Content-Type: application/json


 # GIFT REQUESTS
+###


###-merkit lisätty, jottei @giftUUID ja @userUUID eivät riko niiden alla / yllä olevia requesteja

anttiasmala · 2025-01-15T14:19:13Z

pages/_app.tsx

-import '~/styles/globals.css';
+import '../styles/globals.css';


import '~/styles/globals.css'; vaihdettu import '../styles/globals.css';

Ilman vaihdosta seuraava virheilmoitus tulee: no-extraneous-import https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-extraneous-import.md

Tuo virhekoodi mun mielestä ihan hyvä, joten en itse sitä ehkä poistaisi. Yritin myös lisätä globals.css:n sallituksi, mutta ilmeisesti hyväksyy vain moduleita

Mennäänkö ../-tavalla vai mennäänkö turvallisemmalla ~/-tavalla ja lisätään vain seuraava rivi: // eslint-disable-next-line n/no-extraneous-import? 😄

anttiasmala · 2025-01-15T14:31:04Z

eslint.config.mjs

Isoin muutos tässä on se, että siirrytään .eslintrc.*stä eslint.config.(m)js:n (Flat Config) käyttöön

.eslintrc.*n tuki näyttäisi loppuvan ESLint 10:n kohdalla, joka tulee melko pian. Viittaus: https://eslint.org/blog/2023/10/flat-config-rollout-plans/#eslintrc-removed-in-eslint-v10.0.0

Käytin tämän tiedoston luomiseen ESLintin ohjetta https://eslint.org/docs/latest/use/configure/migration-guide. Tämä tiedosto tuli npx @eslint/migrate-config .eslintrc.json -komennolla. Jouduin tekemään jonkin verran muokkauksia, jotta sain siitä toimivan

Tässä kesti erittäin kauan tehdä. Tämmöinen "konffaus" ei ole yhtään tuttua ja en oikein tiennyt mistä aloittaa / mitä pitää tehdä 😄

Omissa testeissä homma näytti toimivan, mutta katotaan mikä on lopputulos 🤣

Säilytin kaikki samat säännöt mitä meillä olikin. Yhden pienen muokkauksen tein, mutta siitä toisessa kommentissa lisää

anttiasmala · 2025-01-15T14:35:58Z

eslint.config.mjs

+            // Handled by import/typescript
+            // n/no-missing-import is used here instead of node/no-missing-import because seems like eslint-plugin-node is not updated frequently
+            // node/no-missing-import: https://github.com/mysticatea/eslint-plugin-node | n/no-missing-import: https://github.com/eslint-community/eslint-plugin-n
+            'n/no-missing-import': 'off',


Tässä on ainoa muokkaus ruleseihin. Vaihdoin node/ -> n/. Tuota eslint-plugin-nodea ei näytetä päivittävän samalla tavalla. Sanottakoon se, että en tiedä onko tarpeellinen muutos

eslint-plugin-noden Github: https://github.com/mysticatea/eslint-plugin-node

eslint-plugin-n Github: https://github.com/eslint-community/eslint-plugin-n

Tässä vielä issue, jossa päädyttiin tekemään tuo uusi eslint-plugin-n mysticatea/eslint-plugin-node#300

anttiasmala · 2025-01-15T14:39:18Z

eslint.config.mjs

+    {
+      // .next folder added to ignores. This is otherwise being ESLinted
+      ignores: ['**/.next/'],
+    },


Tässä luultavasti on meikän konffaustaidoissa vika. Eli ilman ignorausta .next-kansio ajetan myös ESLintillä

Onko kyseinen kansio ajettu aikaisemmin ESLintillä?

anttiasmala · 2025-01-15T14:39:50Z

package.json

+    "@eslint/compat": "1.2.5",
+    "@eslint/eslintrc": "3.2.0",
+    "@eslint/js": "9.18.0",


Nämä kolme devDependeciä liittyi tähän: https://eslint.org/docs/latest/use/configure/migration-guide

Eivät liene tarpeellisia enää

anttiasmala added 30 commits November 1, 2024 11:35

Install @tanstack/eslint-plugin-query and @tanstack/react-query

d927a38

Change TODO.md files TODO section

bc8b7d9

Create queryClient and add QueryClientProvider. They're in _app.tsx f…

7015d15

…or now, will move it better location if needed later on

Create a component of GiftList. Make it to be used instead of hard co…

97fdb8e

…ded. Add placeholder values for loading and error

Add a small cool loading dot loop and change text fetching text to be…

e7f9739

… Finnish. Loading dot loop taken here: https://stackoverflow.com/a/40991531

Add an TODO task to TODO.md

13b9c74

Move useQuery to Home function. Move a lot of useStates to GiftList c…

16e086f

…omponent due to no need in Home function

Add file utils/utilFunctions.ts that includes util functions. Mostly …

8ac9971

…used for development for now

Added a comment and left a test sleep function that will be clarifyin…

046c930

…g things

ESLint and fixes

b45e952

Remove deleteExpiredSessionsLoop function, does not work with Vercel'…

bb0ad82

…s serverless

Add tanstack to eslintrc.json. Rules are recommended by Tanstack

626b5df

Remove unnecessary useEffect in '/pages/index.tsx'

cb430a8

Remove unnecessary console.logs in /pages/index.tsx

583cf68

ESLint changes. Make loading dot animation 2 seconds long instead of …

0f714cc

…5 seconds. Make it 3 dots long and make it that it has same amount of time with each dot so it looks better

Change queryKey and queryFn. giftQuery is now its own variable instea…

625f082

…d of variables imported from giftQuery itself

Change back the variable declarations from useQuery

37fc468

Move the useQuery to /utils/apiRequests.ts

48e9f37

Remove an unnecessary function. Change the useQuery to use freshly cr…

4d44f2b

…eated useGetGifts

Remove couple useStates and change them to use useGetGifts' data vari…

c9e7f7b

…able

Remove async. It broke React

5349549

Add invalidateQueries into /pages/index.tsx

9dafd1d

Add try/catch block into /utils/apiRequests.ts

0f3cc99

Change return false to return an empty array to fix an error with gif…

0bf8b11

…tData in /pages/index.tsx

Remove refreshGiftList function and change them to queryClient.invali…

148aeb0

…dateQueries

Remove parameters from GiftList component. Change it to use useGetGif…

05b2e18

…ts query instead

Create a type for queryKeys

6c0c903

Add a new function invalidateSingleQueryKey. It will be same as using…

9d11db1

… queryClient's invalidateQueries but it shows all the possible keys that can be invalidated

Remove a few parameters, and remove props from GiftList component tha…

5302753

…t is inside Home component

Remove refreshGiftList function and change them to use new invalidate…

90f08e2

…SingleQueryKey for now. This most likely will be changed in the future. Changes in EditModal.tsx and DeleteModal.tsx

anttiasmala added 7 commits December 23, 2024 12:02

Update more packages

ec2e718

Add eslint-disable-next-line lines to prevent ESLint running

7818a6d

Install a new library eslint-plugin-n. Old eslint-plugin-node does no…

c61dda6

…t seem to be updated anymore. Reference: mysticatea/eslint-plugin-node#360

Change node/no-missing-import to n/no-missing-import

9c8f0a1

Updated minor updates that had come during updating

22b9c6d

Update dotenv-cli package. It has major update. However I could not f…

44e146a

…ind any breaking changes information regarding it so could not test

Update TODO.md

b1e912f

Lower the prisma version back to 5.20.0 due to lucia-auth requiring i…

7d40537

…t. Everything worked perfectly fine with Prisma 6 but for updating packages --force had to be used

vercel bot deployed to Preview December 25, 2024 14:36 View deployment

anttiasmala commented Dec 25, 2024

View reviewed changes

.eslintrc.json Outdated Show resolved Hide resolved

anttiasmala commented Dec 25, 2024

View reviewed changes

.eslintrc.json Outdated Show resolved Hide resolved

anttiasmala added 5 commits January 9, 2025 16:36

Update more packages

e74251b

Remove eslint-disable-next-lines. Change ~/styles/ to ../styles/ to p…

d3140ef

…revent n/no-extraneous-import for now. The rule can be disabled if needed but let's go with this for now

Remove .eslintrc.json and add eslint.config.mjs. Remove the soon be d…

0eab8b1

…eprecated .eslintrc.json method and replace it with the upcoming Flat Config method

Update more packages

d5543f6

Update eslint-config-prettier. No breaking things found during testing

6a936d4

vercel bot deployed to Preview January 15, 2025 12:55 View deployment

Add more details of non updated packages and add a checklist for test…

fa92377

…s to do when updating packages

vercel bot deployed to Preview January 15, 2025 13:41 View deployment

anttiasmala commented Jan 15, 2025

View reviewed changes

Remove commented config inside eslint.config.mjs

773ed65

vercel bot deployed to Preview January 15, 2025 14:12 View deployment

anttiasmala commented Jan 15, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Pakettien päivittäminen | antti/update-packages #60

Pakettien päivittäminen | antti/update-packages #60

anttiasmala commented Dec 25, 2024

vercel bot commented Dec 25, 2024 •

edited

Loading

anttiasmala Dec 25, 2024

anttiasmala Jan 15, 2025

anttiasmala Jan 15, 2025

anttiasmala Jan 15, 2025

anttiasmala Jan 15, 2025 •

edited

Loading

anttiasmala Jan 15, 2025 •

edited

Loading

anttiasmala Jan 15, 2025

anttiasmala Jan 15, 2025 •

edited

Loading

		@@ -30,6 +30,7 @@ Content-Type: application/json


		# GIFT REQUESTS
		###

		import '~/styles/globals.css';
		import '../styles/globals.css';

Pakettien päivittäminen | antti/update-packages #60

Are you sure you want to change the base?

Pakettien päivittäminen | antti/update-packages #60

Conversation

anttiasmala commented Dec 25, 2024

vercel bot commented Dec 25, 2024 • edited Loading

anttiasmala Dec 25, 2024

Choose a reason for hiding this comment

anttiasmala Jan 15, 2025

Choose a reason for hiding this comment

anttiasmala Jan 15, 2025

Choose a reason for hiding this comment

anttiasmala Jan 15, 2025

Choose a reason for hiding this comment

anttiasmala Jan 15, 2025 • edited Loading

Choose a reason for hiding this comment

anttiasmala Jan 15, 2025 • edited Loading

Choose a reason for hiding this comment

anttiasmala Jan 15, 2025

Choose a reason for hiding this comment

anttiasmala Jan 15, 2025 • edited Loading

Choose a reason for hiding this comment

vercel bot commented Dec 25, 2024 •

edited

Loading

anttiasmala Jan 15, 2025 •

edited

Loading

anttiasmala Jan 15, 2025 •

edited

Loading

anttiasmala Jan 15, 2025 •

edited

Loading