Skip to content

Commit

Permalink
Merge pull request SpaceyaTech#50 from SpaceyaTech/moredocs
Browse files Browse the repository at this point in the history
Update documentation and testing guidelines for TANSTACK-QUERY and TANSTACK-ROUTER
  • Loading branch information
tigawanna authored Dec 13, 2024
2 parents 31b7e9c + 4a9735a commit 4aa8754
Show file tree
Hide file tree
Showing 4 changed files with 147 additions and 0 deletions.
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -28,3 +28,4 @@ tsconfig.node.tsbuildinfo
/blob-report/
/playwright/.cache/
tsconfig.app.tsbuildinfo
codealike.json
32 changes: 32 additions & 0 deletions docs/TANSTACK-QUERY.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,32 @@
# tips

- prefer `useSuspenseQuery` over `useQuery`
- prefer defining `queryOptions` over inlinignthem into the query hooks

```tsx
import { queryOptions,useSuspenseQuery } from "@tanstack/react-query";
//users/queryOptions.ts
// this is re-usable and decltters your components
export const userQueryOptions = queryOptions({
queryKey: ["user"],
queryFn: () => {
...
}
})
// users.tsx
const query = useSuspenseQuery(userQueryOptions);
```
- wrap the data fetching components with `Suspense` component

```tsx
// users page
<div>
<header>Users page</headers>
<Suspense fallback={<div>Loading users</div>}>
<Users />
</Suspense>
</div>
```

run `p/mpm run page users` to scafold a page with these best practices

108 changes: 108 additions & 0 deletions docs/TANSTACK-ROUTER.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,108 @@
- anything under the src/routes folder is a route

ex

- `src/routes/index.tsx` ->` /`
- `src/routes/profile.tsx` or `src/routes/profile/index.tsx` -> `/profile/`

Dynamic routes

- `src/routes/users/$oneuser/index.tsx` -> `/users/$oneuser`

inside here you'll have access to the `oneuser` param

```tsx
// src/routes/users/$user/index.tsx
import { createFileRoute, useParams } from '@tanstack/react-router'

export const Route = createFileRoute('/users/$user')({
component:OneUserComponent,
})
functon OneUserComponent() {
const {user} = useParams({
from:"/users/$user"
})
return (
<div>
<h1>{user}</h1>
</div>
)
}
```

if you nest multiple dynamic routes then you can access them in the same way

```tsx
// src/routes/users/$user/friends/$friend/index.tsx
import { createFileRoute, useParams } from '@tanstack/react-router'

export const Route = createFileRoute('/users/$user/friends/$friend/')({
component: RouteComponent,
})

function RouteComponent() {
const { friend,user } = useParams({
from: "/users/$user/friends/$friend/",
});
return (
<div>
<h1>{user}</h1>
<h1>{friend}</h1>
</div>
);
}


```

you can also define search params ,validate them and auth guard

```tsx
import {
createFileRoute,
redirect,
useNavigate,
useSearch,
} from "@tanstack/react-router";
import { z } from "zod";
const searchparams = z.object({
// make it optional if it won't always be used
page: z.number().optional(),
});

export const Route = createFileRoute("/users/")({
component: RouteComponent,
// declare and validate your search params here
// this page should alwatys have /users?page=1
validateSearch: (search) => searchparams.parse(search),
// this is how you auth guard routes (only allow logged in users here )
async beforeLoad(ctx) {
const viewer = ctx.context?.viewer;
if (!viewer?.record) {
throw redirect({
to: "/auth",
// this search params will be used to redirect you back to this page once you log in
search: { returnTo: ctx.location.pathname },
});
}
},
});

function RouteComponent() {
const { page } = useSearch({
from: "/users/",
});
const navigate = useNavigate({
from: "/users",
});
return (
<div>
{page}
<button onClick={() => navigate({ search: { page: (page ?? 0) + 1 } })}>
next page
</button>
</div>
);
}

```
6 changes: 6 additions & 0 deletions docs/TESTING.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,11 @@
# Testing

>[!NOTE]
> Prefer e2e (playwright) fro testing the ui instead of unit tests. and uase the tooling like recording of tests and locators.
> use unit tests for units of logic and functions only


```sh
npm run test
```
Expand Down

0 comments on commit 4aa8754

Please sign in to comment.