Skip to content

Prevent Writing Invalid Content#582

Open
krakow10 wants to merge 2 commits intorojo-rbx:masterfrom
krakow10:content-non-nil
Open

Prevent Writing Invalid Content#582
krakow10 wants to merge 2 commits intorojo-rbx:masterfrom
krakow10:content-non-nil

Conversation

@krakow10
Copy link
Copy Markdown
Contributor

@krakow10 krakow10 commented Dec 5, 2025

The Roblox documentation for Content states that Content::Object always contains a non-nil ref. ContentType::Object(Ref::none()) probably gets deserialized as ContentType::None by Roblox, but we should stop generating invalid files.

Todo:

  • Change deserialize to convert ContentType::Object(Ref::none()) to ContentType::None
  • Serialize ContentType::Object(Ref::none()) as ContentType::None

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.

1 participant