Skip to content

Dashboard refactoring + Enable changes of state#22

Merged
joaomdmoura merged 3 commits intomasterfrom
dashboard-refactoring
Jan 11, 2018
Merged

Dashboard refactoring + Enable changes of state#22
joaomdmoura merged 3 commits intomasterfrom
dashboard-refactoring

Conversation

@joaomdmoura
Copy link
Owner

This is not exactly what I intended but it kind of relates to #20
But it tackles #16 for good 😄

This enable users to change states through the Machinery Dashboard
while still passing through the machinery internals, including
guard and callback functions.

@joaomdmoura joaomdmoura self-assigned this Jan 11, 2018
json_decoder: Poison,
length: 500_000_000


Copy link
Owner Author

Choose a reason for hiding this comment

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

There should be no more than 1 consecutive blank lines.

@joaomdmoura joaomdmoura force-pushed the dashboard-refactoring branch 2 times, most recently from 7fdb8f8 to 7dd4dd8 Compare January 11, 2018 16:03
Repository owner deleted a comment from coveralls Jan 11, 2018
Repository owner deleted a comment from coveralls Jan 11, 2018
This enable users to change states through the Machinery Dashboard
while still passing through the machinery internals, including
guard and callback functions.
@joaomdmoura joaomdmoura force-pushed the dashboard-refactoring branch from 7dd4dd8 to b043e48 Compare January 11, 2018 16:22
Repository owner deleted a comment from coveralls Jan 11, 2018
@coveralls
Copy link

coveralls commented Jan 11, 2018

Coverage Status

Coverage increased (+0.1%) to 98.78% when pulling f413692 on dashboard-refactoring into 8e0c9b3 on master.

Repository owner deleted a comment from coveralls Jan 11, 2018
@joaomdmoura joaomdmoura merged commit d8613d4 into master Jan 11, 2018
@joaomdmoura joaomdmoura deleted the dashboard-refactoring branch January 11, 2018 17:03
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.

2 participants