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

Refactor Trello Driver to match standards #7

Closed
nicoespeon opened this issue May 20, 2016 · 0 comments · Fixed by #46
Closed

Refactor Trello Driver to match standards #7

nicoespeon opened this issue May 20, 2016 · 0 comments · Fixed by #46
Assignees
Milestone

Comments

@nicoespeon
Copy link
Owner

I guess my current implementation of the Trello Driver is not standard, according to the docs:

The output of the driver function can either be a single Observable or a queryable collection of Observables. […]

In some cases it is necessary to output a queryable collection of Observables, instead of a single one. A queryable collection of Observables is essentially a JavaScript object with a function used to choose a particular Observable based on a parameter, e.g. get(param).

http://cycle.js.org/drivers.html

Let's think on that and refactor: advices and suggestions are welcome!

I might also extract the Driver into its own repo if it happen to be relevant.

@nicoespeon nicoespeon added this to the v1.0.0 milestone May 20, 2016
@nicoespeon nicoespeon self-assigned this Jul 20, 2016
nicoespeon added a commit that referenced this issue Jul 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant