Skip to content

Conversation

@MichaelDeBoey
Copy link
Contributor

Follow-up of #2655

Things I did extra:

  • Put all styles into the elements.ts file
  • Make SignOutNoticeModal a FunctionComponent

I think this component (SignOutNoticeModal) can be deleted tho, since it's not used anymore? 🤔

@MichaelDeBoey MichaelDeBoey added 🔨 Refactor 🧠 Overmind Indicates that this is related to the app's State Management labels Nov 22, 2019
@lbogdan lbogdan temporarily deployed to pr3062 November 22, 2019 12:29 Inactive
@lbogdan
Copy link
Contributor

lbogdan commented Nov 22, 2019

Build for latest commit 71807ed is at https://pr3062.build.csb.dev/s/new.

@MichaelDeBoey MichaelDeBoey force-pushed the overmind/SignOutNoticeModal branch from fbeca3c to 80a1b52 Compare November 30, 2019 20:12
@lbogdan lbogdan temporarily deployed to pr3062 November 30, 2019 23:15 Inactive
@MichaelDeBoey MichaelDeBoey force-pushed the overmind/SignOutNoticeModal branch from 80a1b52 to f6dc3a5 Compare December 3, 2019 20:01
@lbogdan lbogdan temporarily deployed to pr3062 December 3, 2019 22:10 Inactive
@MichaelDeBoey MichaelDeBoey force-pushed the overmind/SignOutNoticeModal branch 2 times, most recently from c73804b to 9fab940 Compare December 4, 2019 17:55
@lbogdan lbogdan temporarily deployed to pr3062 December 4, 2019 20:01 Inactive
@SaraVieira
Copy link
Contributor

Awesome! Thank you!

@SaraVieira SaraVieira merged commit 5626af3 into codesandbox:master Dec 8, 2019
@MichaelDeBoey
Copy link
Contributor Author

With pleasure! 🙂

@SaraVieira I'll create a new PR to delete SignOutNoticeModal, since it's not used anymore 🙂

@MichaelDeBoey MichaelDeBoey deleted the overmind/SignOutNoticeModal branch December 8, 2019 17:43
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