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

Modify useQueries to a simpler format #113

Open
michalstruck opened this issue Feb 15, 2024 · 0 comments
Open

Modify useQueries to a simpler format #113

michalstruck opened this issue Feb 15, 2024 · 0 comments
Labels
app Issues related to the native app refactoring

Comments

@michalstruck
Copy link
Collaborator

michalstruck commented Feb 15, 2024

Lowest priority, but CAN actually cause issues
no reason to do

export const useListRecords = (inventoryId: number) => {
  const query = useQuery(["recordsList", inventoryId], () =>
    listRecords(inventoryId)
  );
  return query;
};

instead of

export const useListBarcodes = (inventoryId: number) =>
  useQuery(
    ["barcodeList", inventoryId],
    async () => await barcodeList(inventoryId)
  );
@michalstruck michalstruck added app Issues related to the native app refactoring labels Feb 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app Issues related to the native app refactoring
Projects
None yet
Development

No branches or pull requests

1 participant