Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

Added code to prevent default scene files being moved/deleted #10358

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

Conversation

barankyle
Copy link
Member

Summary

Since new scene creation runs off of default-project/public/scenes/default.gltf, if this file has been moved, scene creation will break. The backend now disallows moving or deleting this scene and its related files.

Resolves IR-2566.

Subtasks Checklist

Breaking Changes

References

closes #insert number here

QA Steps

Copy link
Member

@HexaField HexaField left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of specific paths, I think we should just disallow any operations to the default project completely, if config.syncFileSystem or whatever it is is false.

@barankyle
Copy link
Member Author

Instead of specific paths, I think we should just disallow any operations to the default project completely, if config.syncFileSystem or whatever it is is false.

That's fine with me too. Good idea about tying it to syncFileSystem.

Since new scene creation runs off of default-project/public/scenes/default.gltf,
if this file has been moved, scene creation will break. The backend now disallows
moving, patching, or deleting anything in default-project when in a production
environment, which is determined via !config.fsProjectSyncEnabled.

Resolves IR-2566.
@barankyle barankyle requested a review from HexaField June 12, 2024 01:05
@barankyle barankyle marked this pull request as ready for review June 12, 2024 01:05
@HexaField
Copy link
Member

needs to be reimplemented around new CMS

@DanielBelmes
Copy link
Contributor

@HexaField did you mean to close this one?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants