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

Asset Streamer #126

Merged
merged 15 commits into from
Feb 24, 2025
Merged

Asset Streamer #126

merged 15 commits into from
Feb 24, 2025

Conversation

zpl-zak
Copy link
Member

@zpl-zak zpl-zak commented Feb 20, 2025

Initial implementation of client assets streaming.

Client-side scripting implementation is still pending. This feature only takes care of downloading promoted files from the server.

TODO:

  • Implement a simple download dialog in web app to display progress and potential errors
  • Implement client-side scripting and ensure Lua files are sent pre-compiled and bundled

Initial implementation of client assets streaming.

Client-side scripting implementation is still pending. This feature only takes care of downloading promoted files from the server.

TODO:
* Implement a simple download dialog in web app to display progress and potential errors
* Implement client-side scripting and ensure Lua files are sent pre-compiled and bundled
@zpl-zak zpl-zak linked an issue Feb 20, 2025 that may be closed by this pull request
3 tasks
@zpl-zak zpl-zak self-assigned this Feb 20, 2025
@zpl-zak
Copy link
Member Author

zpl-zak commented Feb 20, 2025

Unblocks MafiaHub/MafiaMP#117

@zpl-zak zpl-zak changed the title add: asset streamer Asset Streamer Feb 20, 2025
Avoid using the same channel we use for general sync.
* handle re-downloads for hot reloading
* provide a download finished/failed callback
* stop pending download if server asks us to download assets again or the client demands it (hot reloading purposes)
zpl-zak added a commit to MafiaHub/MafiaMP that referenced this pull request Feb 21, 2025
@zpl-zak zpl-zak requested a review from Segfaultd February 21, 2025 11:10
@Segfaultd Segfaultd merged commit d051a9e into develop Feb 24, 2025
1 check passed
@Segfaultd Segfaultd deleted the review/asset-streamer branch February 24, 2025 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Server->Client resource streaming
2 participants