Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: vue examples #607

Merged
merged 6 commits into from
Oct 17, 2023
Merged

docs: vue examples #607

merged 6 commits into from
Oct 17, 2023

Conversation

kouts
Copy link
Contributor

@kouts kouts commented Oct 15, 2023

Thanks for this great library and for making the Tanstack framework-agnostic!
This PR adds the following examples for Vue:

The padding examples are still pending, but I believe this is a good start.

PS: I tried running the docs locally but couldn't figure it out, it would be nice if we can have some instructions on how to do that.


const sorting = ref<SortingState>([])

const getSortingHandler = (e: Event, fn: any) => {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I couldn't figure out the type for this function here.

Comment on lines +92 to +96
const setSorting = (sortingUpdater: any) => {
const newSortVal = sortingUpdater(sorting.value)

sorting.value = newSortVal
}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here, I couldn't figure out the type for the sortingUpdater function.

@kouts kouts changed the title docs:vue examples docs: vue examples Oct 15, 2023
@@ -26,12 +26,12 @@ function ReactTableVirtualized() {
},
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file only contains formatting changes due to prettier being applied.

@@ -11,7 +11,7 @@
"linkAll": "lerna exec 'npm link' --parallel",
"unlinkAll": "lerna exec 'npm unlink' --parallel",
"dev": "npm run clean && npm run watch",
"prettier": "prettier \"packages/*/{src/**,examples/**/src/**}.{md,js,jsx,ts,tsx,json}\" --write",
"prettier": "prettier \"{packages/*/src/**,examples/*/*/src/**}.{md,js,jsx,ts,tsx,vue,json}\" --write",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added the .vue extension and fixed the prettier command glob so that it takes into account the examples as well.

@piecyk piecyk merged commit acaff41 into TanStack:beta Oct 17, 2023
2 checks passed
@piecyk
Copy link
Collaborator

piecyk commented Oct 17, 2023

Thanks @kouts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants