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

Webfiles in old data model - load optimization and support webfiles browser refresh load #662

Merged
merged 4 commits into from Aug 8, 2023

Conversation

tyaginidhi
Copy link
Contributor

@tyaginidhi tyaginidhi commented Aug 7, 2023

This PR addresses the following scenario:

  1. For a website, only load annotations (aka CSS/media file etc - related to the portal data currently being viewed). For old data model - we did not have annotation entity distribution per portal. This information is now being proactively fetched from adx_webfiles entity.
  2. This PR also introduces the media file optimization we are doing for new data model - where we load all text files like css/txt on first load, while media files like jpeg/mp4 are loaded on file explorer file's click.
  3. This also addresses the gap where now on browser refresh webfiles currently open in browser can be directly fetched from Dataverse - without waiting for the entire portal data to be loaded first.

@tyaginidhi tyaginidhi marked this pull request as ready for review August 7, 2023 05:02
@tyaginidhi tyaginidhi requested a review from a team as a code owner August 7, 2023 05:02
@tyaginidhi tyaginidhi enabled auto-merge (squash) August 8, 2023 03:56
@tyaginidhi tyaginidhi merged commit be18db4 into main Aug 8, 2023
6 checks passed
@tyaginidhi tyaginidhi deleted the users/nityagi/UXOptimizationsV2 branch August 8, 2023 03:58
@tyaginidhi tyaginidhi mentioned this pull request Aug 11, 2023
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.

None yet

3 participants