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

default string format doesn't work in Markdown #306

Closed
jywarren opened this issue Jul 18, 2018 · 10 comments
Closed

default string format doesn't work in Markdown #306

jywarren opened this issue Jul 18, 2018 · 10 comments
Assignees
Labels

Comments

@jywarren
Copy link
Member

unfortunately, it reads the () as part of the URL syntax, it seems!

https://publiclab.github.io/image-sequencer/examples/#steps=ndvi(filter:blue),colormap(colormap:fastie)

See https://publiclab.org/notes/MaggPi/07-18-2018/ndvi-micrsocopy#c20160

Maybe we should switch for {}?

@gitmate
Copy link

gitmate bot commented Jul 18, 2018

GitMate.io thinks the contributor most likely able to help you is @tech4GT.

1 similar comment
@gitmate
Copy link

gitmate bot commented Jul 18, 2018

GitMate.io thinks the contributor most likely able to help you is @tech4GT.

@gitmate gitmate bot added the bug label Jul 18, 2018
@tech4GT
Copy link
Member

tech4GT commented Jul 21, 2018

@jywarren can you please elaborate on this? I'm sorry I did not quite get you.

@jywarren
Copy link
Member Author

jywarren commented Jul 23, 2018 via email

@tech4GT
Copy link
Member

tech4GT commented Jul 24, 2018

Oh I get it!! We can change this no problem!! @jywarren should this be a fto issue?

@jywarren
Copy link
Member Author

jywarren commented Jul 24, 2018 via email

@jywarren
Copy link
Member Author

jywarren commented Jul 24, 2018 via email

@tech4GT
Copy link
Member

tech4GT commented Jul 24, 2018

@jywarren yes we can but I'm afraid then it won't be reasonable as an fto. I'll do it myself then😄

@jywarren
Copy link
Member Author

jywarren commented Jul 24, 2018 via email

tech4GT added a commit to tech4GT/image-sequencer that referenced this issue Jul 26, 2018
Signed-off-by: tech4GT <varun.gupta1798@gmail.com>
@ghost ghost assigned tech4GT Jul 26, 2018
@ghost ghost added the in-progress label Jul 26, 2018
@jywarren
Copy link
Member Author

jywarren commented Aug 4, 2018

Solved in #313 !

@jywarren jywarren closed this as completed Aug 4, 2018
@ghost ghost removed the in-progress label Aug 4, 2018
jywarren pushed a commit that referenced this issue Aug 4, 2018
* change url string fixes #306

Signed-off-by: tech4GT <varun.gupta1798@gmail.com>

* update

Signed-off-by: tech4GT <varun.gupta1798@gmail.com>

* update

Signed-off-by: tech4GT <varun.gupta1798@gmail.com>

* fix typo

Signed-off-by: tech4GT <varun.gupta1798@gmail.com>
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants