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: defer catalog loading until dashboard is fully executed #5486

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

Conversation

pbenes
Copy link
Contributor

@pbenes pbenes commented Oct 17, 2024

  • replace full catalog loads on dash viewmode with targeted calls to only the displayforms needed
  • defer drilling and edit mode switching until full catalog is loaded
  • mark app with css class when catalog finishes loading (for e2e)

Important

Please, don't forget to run rush change for the commits that introduce new features 🙏


Refer to documentation to see how to run checks and tests in the pull request. This is the list of the most used commands:

extended test - backstop
extended test - tiger-cypress - integrated
extended test - tiger-cypress - isolated
extended test - tiger-cypress - record

@pbenes pbenes requested a review from kandl as a code owner October 17, 2024 09:09
- replace full catalog loads on dash viewmode with targeted calls to
  only the displayforms needed
- defer drilling and edit mode switching until full catalog is loaded
- mark app with css class when catalog finishes loading (for e2e)
@pbenes
Copy link
Contributor Author

pbenes commented Oct 17, 2024

extended test - tiger-cypress - record

@gooddata
Copy link

gooddata bot commented Oct 17, 2024

Job gooddata-ui-sdk-unit-tests at https://checklist.intgdc.com/job/gooddata-ui-sdk/job/gooddata-ui-sdk-unit-tests/22128/ finished with 'FAILURE' status.

@gooddata
Copy link

gooddata bot commented Oct 17, 2024

Job gooddata-ui-sdk-pull-request-dispatcher-pipeline at https://checklist.intgdc.com/job/gooddata-ui-sdk/job/gooddata-ui-sdk-pull-request-dispatcher-pipeline/2295/ finished with 'FAILURE' status.

@pbenes pbenes added the do not merge Do not merge this yet label Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge Do not merge this yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant