-
-
Notifications
You must be signed in to change notification settings - Fork 667
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
fix: remove columns from the search api query #6996
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
@@ -86,10 +86,11 @@ export const ProjectFeatureToggles = ({ | |||
createdAt: tableState.createdAt, | |||
}; | |||
|
|||
const { columns: _, ...apiTableState } = tableState; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
omit columns and rename to _ to avoid name clash with the existing columns variable
@@ -119,7 +120,6 @@ export const ProjectFeatureToggles = ({ | |||
setFeatureArchiveState, | |||
setFeatureStaleDialogState, | |||
} = useRowActions(refetch, projectId); | |||
const [showExportDialog, setShowExportDialog] = useState(false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unused variable
About the changes
When selecting columns to display in the UI we want to persist them in the URL for sharing but we don't want to send them to the API and generate unnecessary SWR request keys.
Also removed unused project overview component and unused variable.
Important files
Discussion points