Github user sergey-safarov commented on the issue:
https://github.com/apache/couchdb-fauxton/pull/802
Pull request is updated.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user garrensmith commented on the issue:
https://github.com/apache/couchdb-fauxton/pull/802
Thanks for testing, would you be able to make a fix for the bug you found?
All misspelling thanks to my iPhone.
On Mon, Feb 27, 2017 at 5:31 PM +0200,
Github user sergey-safarov commented on the issue:
https://github.com/apache/couchdb-fauxton/pull/802
@garrensmith thank you for update
I tested current fauxton master all fine except one "Cancel" button.
To reproduce required start editing of view or create new view and press
Github user garrensmith commented on the issue:
https://github.com/apache/couchdb-fauxton/pull/802
@sergey-safarov thanks for trying to help us with this. We have decided to
rather solve this with `encodeURIComponent`. I've just merged a PR that should
fix the view section.
Github user robertkowalski commented on the issue:
https://github.com/apache/couchdb-fauxton/pull/802
@sergey-safarov the PR looks good, it will be superseeded by
https://github.com/apache/couchdb-fauxton/pull/807 which contains more fixes
and changes our integration tests to default
Github user robertkowalski commented on the issue:
https://github.com/apache/couchdb-fauxton/pull/802
hey, thank you for your contribution -- i was on vacation, i'll check next
week!
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user garrensmith commented on the issue:
https://github.com/apache/couchdb-fauxton/pull/802
Ok. @robertkowalski needs to review this before we can merge.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user sergey-safarov commented on the issue:
https://github.com/apache/couchdb-fauxton/pull/802
About test cases. I can make hotfix but test case creation is to complex
task for me.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user sergey-safarov commented on the issue:
https://github.com/apache/couchdb-fauxton/pull/802
Hello @garrensmith error is fixed
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user garrensmith commented on the issue:
https://github.com/apache/couchdb-fauxton/pull/802
Thanks @sergey-safarov this is great. The tests are failing. Could you see
why. And could you maybe add a test or two to prove this improves and fixes the
`/` issues
---
If your
10 matches
Mail list logo