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

feat: add a new parameter to get_table_one_big_table_query #629

Merged
merged 1 commit into from
Jul 3, 2024

Conversation

jhonylucas74
Copy link
Collaborator

@jhonylucas74 jhonylucas74 commented Jul 3, 2024

Purpose

Improve the SQL queries generated in the backend.

Description

This task simplifies the generated query to use a single, consolidated SQL table. By default, the generated SQL includes translations from the table, but it is now possible to pass a parameter that will always simplify the SQL.

Checklist

  • I have reviewed the code changes.
  • I have tested the changes locally.
  • I have updated the documentation if needed.
  • I have added/modified tests to ensure the changes are valid.

Testing and Evidence

The GraphQL query to be used: Pass includeTableTranslation as false to simplify the generated SQL query.

image

This PDF represents the two versions of the query generated from the same table.
Query_diff.pdf

Next Steps

Implement this in the frontend.

@jhonylucas74 jhonylucas74 merged commit 00db381 into main Jul 3, 2024
4 checks passed
@jhonylucas74 jhonylucas74 deleted the feat/improve-get-one-big-table-query branch July 3, 2024 14:33
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.

Incluir versão da query sem tradução de tabelas no backend
3 participants