Skip to content

Add support for Roblox's internal Stories and Storybooks#290

Draft
vocksel wants to merge 108 commits intomainfrom
roblox-internal
Draft

Add support for Roblox's internal Stories and Storybooks#290
vocksel wants to merge 108 commits intomainfrom
roblox-internal

Conversation

@vocksel
Copy link
Contributor

@vocksel vocksel commented Nov 21, 2024

Problem

Describe the problem you are trying to solve

Solution

Describe the solution you came up with

@vocksel vocksel changed the title Add support for Roblox's internal Developer Storybook Add support for Roblox's internal Stories and Storybooks Nov 21, 2024
vocksel added a commit that referenced this pull request Sep 16, 2025
# Problem

For #290 I need to get an accurate snapshot of the entire plugin in
plain files. Our rbxm gets built with all of the packages, but I can't
work with the rbxm in this case

# Solution

I setup the build system so that Rojo isn't doing any stitching
together. The final build is output to `build/plugin` including a copy
of all of the packages that are required
vocksel added a commit that referenced this pull request Sep 17, 2025
# Problem

I observed some huge chunks of microprofiler time spent from
GetSetting/SetSetting calls. Presumably each one is making a disk
read/write. And while not in this PR, a change in #290 introduces
frequent read/writes that we need to accommodate

# Solution

Plugin settings are now read from disk only once. In-memory edits are
made from there.

A write is performed on each change, but if needed this can be revised
to batch updates on a timer
vocksel added a commit that referenced this pull request Sep 17, 2025
This is a future-looking change.

For #290 I need a function that can setup the plugin UI for an arbitrary
widget and button. This is something I anticipate exposing off of
FlipbookCore (#338) once I get around to that
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