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 sequencer.importString() #277

Closed
jywarren opened this issue Jun 3, 2018 · 3 comments
Closed

add sequencer.importString() #277

jywarren opened this issue Jun 3, 2018 · 3 comments

Comments

@jywarren
Copy link
Member

jywarren commented Jun 3, 2018

This is a small one but we can now use this code:

https://github.com/tech4GT/image-sequencer/blob/ac183205f0e30015d1c273359e2d6ee7cede7538/examples/lib/defaultHtmlSequencerUi.js#L17-L20

as a built-in sequencer function called importString(), right?

@jywarren
Copy link
Member Author

jywarren commented Jun 3, 2018

GitMate.io thinks possibly related issues are #250 (Add step.toString() and sequencer.toString() methods), #157 (Add a Contrast module), #138 (Add a Dithering module), #113 (Add a Brightness module), and #158 (Add a Saturation Module).

@tech4GT
Copy link
Member

tech4GT commented Jun 3, 2018

@jywarren Exactly!!

@jywarren jywarren added this to the Core work milestone Jun 3, 2018
@jywarren
Copy link
Member Author

jywarren commented Jun 3, 2018

Awesome. I also reorganized the milestone order a bit reflecting some of our new smaller issues and recent progress. Great work this week, Varun!

https://github.com/publiclab/image-sequencer/milestone/2

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

No branches or pull requests

2 participants