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

Add select columns #69

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

carloeusebi
Copy link

The ExceptionModel can be quite big, as it carries with himself information about the headers, queries and trace.
Since those fields aren't shown in the table there is no need to load all those heavy information, I modified the query used to retrieve the models by selecting only the fields we are displaying.

Before:
image

After:
image
This pull request to src/Resources/ExceptionResource.php includes a change to enhance the functionality of the form method by modifying the query used in the table method. The most important change is:

Copy link

what-the-diff bot commented Jan 19, 2025

PR Summary

  • Query Modification to ExceptionResource's Table Method
    The team has made some changes to how we fetch data from our system's database. Earlier, we used to pull all data available. But now, we've streamlined this process. We're focusing only on specific aspects such as ID, path, method, type, code, IP, and creation date. This change will save system resources as it reduces unnecessary data retrieval.

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.

1 participant