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

fix: error while downloading camera reel #3087

Draft
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

lorenzo-ranciaffi
Copy link
Contributor

What does this PR change?

Remove the opening of the system folder where the file has been downloaded (there is no support for System.Diagnostic.Process API as stated here, at the very last point)

How to test the changes?

  1. Launch the explorer
  2. Open the camera reel gallery
  3. Download a reel and check that no error occurs
  4. Check that the file is present at folder ${user-home}/decentraland/reels

Our Code Review Standards

https://github.com/decentraland/unity-renderer/blob/master/docs/code-review-standards.md

@lorenzo-ranciaffi lorenzo-ranciaffi self-assigned this Jan 10, 2025
@lorenzo-ranciaffi lorenzo-ranciaffi added the force-build Used to trigger a build on draft PR label Jan 10, 2025
@lorenzo-ranciaffi lorenzo-ranciaffi added force-build Used to trigger a build on draft PR and removed force-build Used to trigger a build on draft PR labels Jan 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
force-build Used to trigger a build on draft PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants