Skip to content
This repository has been archived by the owner on Jul 11, 2024. It is now read-only.

Eager loading of relations whlie using filters(v3.4) #346

Open
wants to merge 53 commits into
base: master
Choose a base branch
from

Conversation

ilyaskarim
Copy link
Collaborator

@ilyaskarim ilyaskarim commented Feb 20, 2023

3.4.0

  • Added eager loading for list and view queries.
  • Improved lazy loading behavior in GraphQL to avoid unnecessary queries.
  • Added support for loading only requested fields in Sequelize.
  • Added recursion support for both list and view queries.
  • Added support for filtering associated data in recursive queries.
  • Removed moment.js and replaced with dayjs.
  • Scanned and verified fixes with SonarQube.
  • Removed lodash, installed its child packages to reduce app size.
  • Added startServer(), restartServer(), stopServer() methods
  • Changed list query to users instead of listUsers, EcommerceUsers becomes ecommerce_users.
  • Changed view query to singular user instead of viewUser, EcommerceUser becomes ecommerce_user.
  • Changed graphql list field list to rows.
  • Removed method applyRelationshipsFromStoreToGraphql, since we are moving to eager loading.
  • Added Module after module name in graphql types. For example, User is now UserModule.
  • Added tests to ensure graphql operations work perfectly.
  • Added option to output graphql type definitions in a file.

Todo

  • change list inside type ${module.name}List graphql type into rows
  • Cleanup function convertGraphqlRequestedFieldsIntoInclude
  • The view graphql query viewUser, viewGame should be just game, user
  • For list queries, listUsers, listGames, the query should be just users, games

@ilyaskarim ilyaskarim changed the title V3.4 Big update: V3.4 Eager loading of relations whlie using filters Feb 25, 2023
@ilyaskarim ilyaskarim changed the title Big update: V3.4 Eager loading of relations whlie using filters Eager loading of relations whlie using filters(v3.4) Feb 25, 2023
@ilyaskarim ilyaskarim marked this pull request as ready for review July 1, 2023 10:19
@ilyaskarim ilyaskarim requested a review from afaq-karim July 1, 2023 10:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants