feat/lazy load tree details #776
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
WilsonNet
force-pushed
the
feat/lazy-load-tree-details
branch
2 times, most recently
from
e6efa71 to
7aa8c9e
Compare
WilsonNet
changed the base branch from
main
to
refactor/api-response-snake-case
murilx
force-pushed
the
refactor/api-response-snake-case
branch
2 times, most recently
from
c7fca4b to
b9f33d5
Compare
WilsonNet
force-pushed
the
feat/lazy-load-tree-details
branch
from
88fe89e to
45792cf
Compare
murilx
force-pushed
the
refactor/api-response-snake-case
branch
from
b9f33d5 to
3f7ea5e
Compare
WilsonNet
force-pushed
the
feat/lazy-load-tree-details
branch
2 times, most recently
from
86cb9ea to
3f7c564
Compare
WilsonNet
changed the base branch from
refactor/api-response-snake-case
to
main
WilsonNet
force-pushed
the
feat/lazy-load-tree-details
branch
from
3f7c564 to
5b86a2f
Compare
WilsonNet
changed the title
murilx
reviewed
Jan 17, 2025
Now the order of the lazy load is Summary > Current Page Tab> Other Pages Tab - Fixed typos in variable names (e.g., "enviroment" to "environment") - Updated type definitions in backend and frontend files - Refactored API calls to use a more modular approach - Improved lazy loading of tree details in the frontend - Updated schema.yml to reflect changes in type definitions Closes #755
WilsonNet
force-pushed
the
feat/lazy-load-tree-details
branch
from
5b86a2f to
4ce802e
Compare
Francisco2002
approved these changes
Jan 17, 2025
| treeId, | ||
| filter = {}, | ||
| enabled = true, | ||
| variant, |
There was a problem hiding this comment.
nit: use full variant as default
There was a problem hiding this comment.
I tried but it is breaking the typescript somehow
murilx
approved these changes
Jan 17, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Lazy Load Tree Details so it uses the new split endpoints
How to test