Skip to content

Conversation

@MichaelDeBoey
Copy link
Contributor

Follow-up of #2833 & #2914

Things I did extra:

  • Put all styles into the elements.ts file
  • Make DashboardComponent a FunctionComponent
  • Inline DashboardContent
  • Export Dashboard by a named export instead of a default export

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

lbogdan commented Feb 8, 2020

Build for latest commit 562c622 is at https://pr3486.build.csb.dev/s/new.

Copy link
Contributor

@SaraVieira SaraVieira left a comment

Choose a reason for hiding this comment

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

Thank you!

@SaraVieira SaraVieira merged commit a57f8e0 into codesandbox:master Feb 13, 2020
@MichaelDeBoey
Copy link
Contributor Author

With pleasure! 🙂

@MichaelDeBoey MichaelDeBoey deleted the overmind/Dashboard branch February 13, 2020 13:28
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