-
Notifications
You must be signed in to change notification settings - Fork 17
feat: 6849 loading screen by data #8046
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
Merged
Merged
Changes from 3 commits
Commits
Show all changes
18 commits
Select commit
Hold shift + click to select a range
b62f498
feat: progress is now reported with byte-weighted tracking
alejandro-jimenez-dcl 7fcd792
fix: wip
alejandro-jimenez-dcl 20f96d2
fix: update progress computation and handle unknown-size
alejandro-jimenez-dcl 7628ed2
chore: code review
alejandro-jimenez-dcl 66fa0d8
feat: convert the head request into a HeadOp to be part of the webreq…
alejandro-jimenez-dcl 24002eb
Merge remote-tracking branch 'origin/dev' into feat/6849-loading-scre…
alejandro-jimenez-dcl dd5bb7d
refactor: simplify byte-weighted loading progress
alejandro-jimenez-dcl 83b57cf
Merge branch 'dev' into feat/6849-loading-screen-by-data
alejandro-jimenez-dcl 9e21fd7
refactor: skip ReadLoadingState for finished scene assets
alejandro-jimenez-dcl 65c5125
feat: gate byte-weighted loading progress behind feature flag
alejandro-jimenez-dcl 9f5b98f
fix: test compilation
alejandro-jimenez-dcl 681ad3b
chore: missing meta
alejandro-jimenez-dcl 5bb5cd8
refactor: simplify byte progress smoothing, add tracker tests
alejandro-jimenez-dcl b5bc906
fix: code convention
alejandro-jimenez-dcl 6daec5c
feat: add feature flag
alejandro-jimenez-dcl d522d5c
Merge branch 'dev' into feat/6849-loading-screen-by-data
alejandro-jimenez-dcl 5a26fef
Merge remote-tracking branch 'origin/dev' into feat/6849-loading-scre…
alejandro-jimenez-dcl f282fff
fix: handle HEAD timeout exceptions
alejandro-jimenez-dcl File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
14 changes: 14 additions & 0 deletions
14
Explorer/Assets/DCL/WebRequests/IStreamableLoadingProgress.cs
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,14 @@ | ||
| namespace DCL.WebRequests | ||
| { | ||
| public interface IStreamableLoadingProgress | ||
|
alejandro-jimenez-dcl marked this conversation as resolved.
Outdated
|
||
| { | ||
| float Progress { get; } | ||
| long ContentLength { get; } | ||
| } | ||
|
|
||
| public interface IStreamableLoadingProgressHandler : IStreamableLoadingProgress | ||
| { | ||
| void SetProgress (float progress); | ||
| void SetContentLength (long length); | ||
| } | ||
| } | ||
3 changes: 3 additions & 0 deletions
3
Explorer/Assets/DCL/WebRequests/IStreamableLoadingProgress.cs.meta
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or 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
This file contains hidden or 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
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.