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

change url string fixes #306 #313

Merged
merged 6 commits into from
Aug 4, 2018
Merged

change url string fixes #306 #313

merged 6 commits into from
Aug 4, 2018

Conversation

tech4GT
Copy link
Member

@tech4GT tech4GT commented Jul 26, 2018

@jywarren New format is with {} as you suggested and it is also compatible with old ()
Signed-off-by: tech4GT varun.gupta1798@gmail.com

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
Signed-off-by: tech4GT <varun.gupta1798@gmail.com>
@tech4GT tech4GT closed this Jul 26, 2018
@tech4GT tech4GT reopened this Jul 26, 2018
@ghost ghost added in-progress and removed in-progress labels Jul 26, 2018
@tech4GT tech4GT closed this Jul 26, 2018
@tech4GT tech4GT reopened this Jul 26, 2018
@ghost ghost added in-progress and removed in-progress labels Jul 26, 2018
@tech4GT
Copy link
Member Author

tech4GT commented Aug 2, 2018

@jywarren ready to merge

Signed-off-by: tech4GT <varun.gupta1798@gmail.com>
Signed-off-by: tech4GT <varun.gupta1798@gmail.com>
@tech4GT
Copy link
Member Author

tech4GT commented Aug 4, 2018

@jywarren rebased and ready to merge!! Now working on overlay and colorbar!!

@jywarren
Copy link
Member

jywarren commented Aug 4, 2018

This is great. Could you put a note about backwards compatibility and a couple examples of string syntax in a new section called, maybe ## String syntax, maybe next to or within this section?

https://github.com/publiclab/image-sequencer/#saving-sequences

What do you think?

@tech4GT
Copy link
Member Author

tech4GT commented Aug 4, 2018

@jywarren Sure!! I am working on overlay, once I am done with that I'll put this in!! An hour maybe..

Signed-off-by: tech4GT <varun.gupta1798@gmail.com>
Signed-off-by: tech4GT <varun.gupta1798@gmail.com>
@tech4GT
Copy link
Member Author

tech4GT commented Aug 4, 2018

@jywarren Done! I think this is ready to be merged in!! Yay!!

Copy link
Member

@jywarren jywarren left a comment

Choose a reason for hiding this comment

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

This looks great. I may add a little to the readme later. Thanks!

@jywarren jywarren merged commit 43e6253 into publiclab:master Aug 4, 2018
@ghost ghost removed the in-progress label Aug 4, 2018
@jywarren jywarren mentioned this pull request Jan 4, 2019
4 tasks
This pull request was closed.
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.

2 participants