Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add full param options for monitors #199

Merged
merged 1 commit into from
Sep 16, 2019

Conversation

Seanstoppable
Copy link
Contributor

Add a more generic function that will let people mix and match
Have existing functions wrap this better function
Resolves the rest of state support in #164

@Seanstoppable
Copy link
Contributor Author

@ojongerius Hey, just wanted to follow up on this, now that we are through the holidays 😄

@chirauki
Copy link

chirauki commented Sep 3, 2019

Hi there, any progress on this one? Could really use the feature. I'm happy to open a new PR with a rebased version if @Seanstoppable is not available.

@Seanstoppable
Copy link
Contributor Author

I'm happy to re-pick up this work

Add a more generic function that will let people mix and match
Have existing functions wrap this better function
Resolves the rest of state support in zorkian#164
@Seanstoppable
Copy link
Contributor Author

@chirauki rebased

@Seanstoppable Seanstoppable mentioned this pull request Sep 11, 2019
@chirauki
Copy link

So cool! I guess someone will nned to take a look, approve and merge? @ojongerius maybe?

Copy link
Contributor

@platinummonkey platinummonkey left a comment

Choose a reason for hiding this comment

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

LGTM from monitors team, nice approach 👍

@bkabrda
Copy link
Collaborator

bkabrda commented Sep 16, 2019

This looks awesome, thanks a bunch for doing it. Everything looks fine, so I'm going to merge right now. 👍

@bkabrda bkabrda merged commit d2641d2 into zorkian:master Sep 16, 2019
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.

4 participants