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

Sub.filter function #808

Closed
aravantv opened this Issue Jan 15, 2017 · 3 comments

Comments

Projects
None yet
3 participants
@aravantv

aravantv commented Jan 15, 2017

Hi,

I would really like to see a Sub.filter function:

Sub.filter : (a -> Bool) -> Sub a -> Sub a

such that Sub.filter p s is a subscription which only sends updates if the value carried by the original subscription s passes the predicate p.

I would like to make a "sub-extras" package as indicated in the guidelines but the only solution I see requires changing the Native.Platform package and, as far as I understood, I should not be dealing with this in a separate library... right?

@process-bot

This comment has been minimized.

Show comment
Hide comment
@process-bot

process-bot Jan 15, 2017

Thanks for the issue! Make sure it satisfies this checklist. My human colleagues will appreciate it!

Here is what to expect next, and if anyone wants to comment, keep these things in mind.

process-bot commented Jan 15, 2017

Thanks for the issue! Make sure it satisfies this checklist. My human colleagues will appreciate it!

Here is what to expect next, and if anyone wants to comment, keep these things in mind.

@jvoigtlaender

This comment has been minimized.

Show comment
Hide comment
@jvoigtlaender

jvoigtlaender Jan 15, 2017

Contributor

There has been extensive discussion of exactly this request on the mailing list in the past (see links below).

The role of the issue tracker here is not for feature requests. If you want a new feature added, you will have to make a convincing argument in the discussion forum first. For this specific request, I and others have tried but been turned down.

What you should probably read:

And maybe also:

Contributor

jvoigtlaender commented Jan 15, 2017

There has been extensive discussion of exactly this request on the mailing list in the past (see links below).

The role of the issue tracker here is not for feature requests. If you want a new feature added, you will have to make a convincing argument in the discussion forum first. For this specific request, I and others have tried but been turned down.

What you should probably read:

And maybe also:

@aravantv

This comment has been minimized.

Show comment
Hide comment
@aravantv

aravantv Jan 15, 2017

Thanks! The conversation there reflects precisely my train of thoughts.

Sorry for using the issue tracker for feature request, there are so many options out there to communicate (Slack, Mailing list, Github, usw.) that I was not so sure what's the best way to do it. Now I know.

aravantv commented Jan 15, 2017

Thanks! The conversation there reflects precisely my train of thoughts.

Sorry for using the issue tracker for feature request, there are so many options out there to communicate (Slack, Mailing list, Github, usw.) that I was not so sure what's the best way to do it. Now I know.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment