Skip to content

Conversation

@MichaelDeBoey
Copy link
Contributor

Follow-up of #2826

Things I did extra:

  • Make ConfigurationFiles a FunctionComponent
  • Extract ExistingConfigurations, FileConfig & OtherConfigurations from ConfigurationFiles
  • Put all styles into the elements.ts file
  • Move overmind subscriptions as close as possible to the components itself instead of passing it through

@MichaelDeBoey MichaelDeBoey added 🔨 Refactor 🧠 Overmind Indicates that this is related to the app's State Management labels Feb 13, 2020
@SaraVieira
Copy link
Contributor

Hey!

This is also the sidebar things so please don't change files inpages/Sandbox/Editor/Workspace/ as they are being completely changed

@lbogdan
Copy link
Contributor

lbogdan commented Feb 13, 2020

Build for latest commit 6e60a81 is at https://pr3510.build.csb.dev/s/new.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🧠 Overmind Indicates that this is related to the app's State Management 🔨 Refactor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants