Skip to content

Conversation

@Gobinath-Manokaran
Copy link

What kind of change does this PR introduce?

Refactor for #2621 @Saeris @christianalfoni

What is the current behavior?

They are in ts with cerebral

What is the new behavior?

They are changed to ts files with Overmind

What steps did you take to test this? This is required before we can merge, make sure to test the flow you've updated.

yarn test
yarn lint
yarn typecheck
yarn start, go to Dashboard, drag things no error

Checklist

  • [x ] Documentation
  • [ x] Testing
  • [ x] Ready to be merged
  • [ x] Added myself to contributors table

@lbogdan lbogdan temporarily deployed to pr2865 October 19, 2019 07:17 Inactive
@lbogdan
Copy link
Contributor

lbogdan commented Oct 19, 2019

Build for latest commit a15f75a is at https://pr2865.build.csb.dev/s/new.

@Gobinath-Manokaran
Copy link
Author

Build for latest commit 4c7742b is at https://pr2865.build.csb.dev/s/new.

Thanks.. Functionalities are working fine

@SaraVieira
Copy link
Contributor

Thank you so much! All seems to still work great!

@MichaelDeBoey MichaelDeBoey added 🔨 Refactor 🧠 Overmind Indicates that this is related to the app's State Management labels Dec 16, 2019
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.

4 participants