-
Notifications
You must be signed in to change notification settings - Fork 140
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
Upload images through the markdown editor #261
Upload images through the markdown editor #261
Conversation
You can put the following code in a .sh file and execute it to create example app.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@y-chiasson Thanks for the PR! I like this feature and am interested in merging it. Can you find somewhere to document the feature, since there's no implementation requirements it can be brief. also, can you write a couple tests for it?
@jamesmk Thank you for having a look at this PR. I will commit some documentation and tests as soon as possible! |
How is the progress here? We could really need this feature! If there is something i could help with feel free to get in touch with we :) |
@christophebeling This feature is missing documentation and tests. If you are interested in contributing @y-chiasson might accept a PR into their |
@jamesmk what kind of tests did you think of? do you use any javascript testing framework yet? documentation dosen't look that tough since this feature is pretty easy to use. but testing all the new javascript might be a lot of work |
@christophebeling We don't have a dedicated JS testing frameworks, but you can test JS interactions via our acceptance testing framework, Capybara. I'm looking for a test emulating a successful interaction with the feature. We don't use the WYSIWYG internally, so we rely on tests to keep the less visible features stable. The more the merrier :) |
👍 for getting this in the next release |
What's the status of this? |
How can I use this? |
@johnnyshields I have requested some tests along with the PR. As soon as someone adds them, I'll be happy to merge. @abejosua You can point Fae to @y-chiasson's branch in your |
Can someone at FINE get this to the finish line? We've been using it in production for half a year without issue. |
#260