[GitHub] couchdb-fauxton issue #864: Update replication to work with scheduler api

2017-03-08 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/864 > Something has changed broken alignment of the values in the replication form: This is actually a firefox issue. I've managed to make a work around. > The

[GitHub] couchdb-fauxton issue #865: Permanent Notification Banner Plugin

2017-03-06 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/865 Awesome. This has been merged. You can close the ticket. --- 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

[GitHub] couchdb-fauxton issue #864: Update replication to work with scheduler api

2017-03-06 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/864 @justin-mcdavid-ibm I've made some of the changes. Others of them I have some comments: > I'm sorry for the confusion in the mockups, but I think we should keep the &

[GitHub] couchdb-fauxton issue #865: Permanent Notification Banner Plugin

2017-03-06 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/865 This looks good. There doesn't seem to be a way to clear the notification. Could you fix the linting issues. --- If your project is set up for it, you can reply to this email

[GitHub] couchdb-fauxton issue #859: fix replication source/target url

2017-03-02 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/859 @nolanlawson could you close this since we have fixed it in pouchdb-server --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] couchdb-fauxton pull request #864: Update replication to work with scheduler...

2017-03-02 Thread garrensmith
GitHub user garrensmith opened a pull request: https://github.com/apache/couchdb-fauxton/pull/864 Update replication to work with scheduler api This is to support the new scheduler api that will land in CouchDB soon You can merge this pull request into a Git repository by running

[GitHub] couchdb-nano issue #6: Adding replication using the "_replicator" database

2017-03-02 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-nano/pull/6 Thanks @carlosduclos I've merged this to master. Could you close this PR. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] couchdb-fauxton pull request #802: COUCHDB-3224: Fixed database name encodin...

2017-02-27 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/802#discussion_r103267380 --- Diff: app/addons/documents/doc-editor/components.react.jsx --- @@ -227,7 +227,7 @@ var AttachmentsPanelButton = React.createClass

[GitHub] couchdb-fauxton issue #802: COUCHDB-3224: Fixed view displaying for database...

2017-02-27 Thread garrensmith
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] couchdb-fauxton issue #863: fix font routes

2017-02-23 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/863 Travis is taking really long. This is green over here https://travis-ci.org/garrensmith/couchdb-fauxton/builds/204752027 --- If your project is set up for it, you can reply

[GitHub] couchdb-fauxton pull request #863: fix font routes

2017-02-23 Thread garrensmith
GitHub user garrensmith opened a pull request: https://github.com/apache/couchdb-fauxton/pull/863 fix font routes You can merge this pull request into a Git repository by running: $ git pull https://github.com/garrensmith/couchdb-fauxton fix-style-paths Alternatively you can

[GitHub] couchdb-fauxton issue #659: [WIP] Clipboard.js

2017-02-22 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/659 @benkeen we have implemented clipboard.js, this PR helped. Could you close this PR --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] couchdb-fauxton issue #669: Replication page update

2017-02-22 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/669 @benkeen this worked was merged in with the other PR. Could you close this PR. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] couchdb-fauxton issue #824: Replication: hotfix url parsing for invalid urls

2017-02-22 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/824 I made some changes here and its passing here https://travis-ci.org/robertkowalski/couchdb-fauxton/builds/204485099 Apache travis is slow at the moment. So I'm going to merge

[GitHub] couchdb-fauxton issue #862: handle username and passwords correctly with bas...

2017-02-22 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/862 merged --- 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 enabled and wishes so

[GitHub] couchdb-fauxton pull request #862: handle username and passwords correctly w...

2017-02-22 Thread garrensmith
Github user garrensmith closed the pull request at: https://github.com/apache/couchdb-fauxton/pull/862 --- 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 enabled and wishes so

[GitHub] couchdb-nano issue #5: Ensure 'qs' is real whenever qs will be manipulated.

2017-02-22 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-nano/pull/5 Thanks. I've merged this in --- 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 enabled

[GitHub] couchdb-fauxton pull request #862: handle username and passwords correctly w...

2017-02-22 Thread garrensmith
GitHub user garrensmith opened a pull request: https://github.com/apache/couchdb-fauxton/pull/862 handle username and passwords correctly with base64 encoding You can merge this pull request into a Git repository by running: $ git pull https://github.com/garrensmith/couchdb

[GitHub] couchdb-fauxton issue #860: replace url construction with FauxtonAPI lookup

2017-02-22 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/860 @millayr could you close this PR. --- 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

[GitHub] couchdb-nano issue #5: Ensure 'qs' is real whenever qs will be manipulated.

2017-02-21 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-nano/pull/5 @glynnbird could you review this and then I 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 project does

[GitHub] couchdb-fauxton pull request #861: fix deployment to work with styles.css

2017-02-21 Thread garrensmith
GitHub user garrensmith opened a pull request: https://github.com/apache/couchdb-fauxton/pull/861 fix deployment to work with styles.css You can merge this pull request into a Git repository by running: $ git pull https://github.com/garrensmith/couchdb-fauxton fix-release

[GitHub] couchdb-fauxton issue #860: replace url construction with FauxtonAPI lookup

2017-02-21 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/860 +1 --- 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 enabled and wishes so

[GitHub] couchdb-fauxton issue #858: Replication from local

2017-02-20 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/858 Oh no. That's frustrating.  All misspelling thanks to my iPhone. On Mon, Feb 20, 2017 at 8:24 PM +0200, "Nolan Lawson" <notificati...@gith

[GitHub] couchdb-fauxton issue #858: Replication from local

2017-02-20 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/858 Thanks merged. --- 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 enabled

[GitHub] couchdb-fauxton pull request #858: Replication from local

2017-02-20 Thread garrensmith
Github user garrensmith closed the pull request at: https://github.com/apache/couchdb-fauxton/pull/858 --- 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 enabled and wishes so

[GitHub] couchdb-fauxton issue #854: don't encode username/pass unless necessary

2017-02-20 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/854 One thing, we need full urls because CouchDB 2.0 requires it. --- 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] couchdb-fauxton issue #854: don't encode username/pass unless necessary

2017-02-20 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/854 @nolanlawson I've added a test and continued this work here https://github.com/apache/couchdb-fauxton/pull/858 If the tests all pass I'll merge it all in. --- If your project

[GitHub] couchdb-fauxton pull request #858: Replication from local

2017-02-20 Thread garrensmith
GitHub user garrensmith opened a pull request: https://github.com/apache/couchdb-fauxton/pull/858 Replication from local You can merge this pull request into a Git repository by running: $ git pull https://github.com/garrensmith/couchdb-fauxton replication-from-local

[GitHub] couchdb-fauxton issue #853: code-split "brace" into second bundle

2017-02-20 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/853 This is really interesting. I want to hold off merging it and do a bit of research on this. It also might be worth upgrading to webpack 2. --- If your project is set up for it, you can

[GitHub] couchdb-fauxton issue #849: Currently reducers seem to be handled quite stra...

2017-02-20 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/849 Thanks @timmak I've merged this in. Can you close this PR. For future reference don't merge from master into your working branch. Rather rebase master into your PR branch

[GitHub] couchdb-fauxton issue #852: trim moment locale using IgnorePlugin

2017-02-20 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/852 Thanks. I've merged it in. --- 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] couchdb-fauxton issue #852: trim moment locale using IgnorePlugin

2017-02-20 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/852 I've documented your ideas here https://issues.apache.org/jira/browse/COUCHDB-3303 --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] couchdb-fauxton issue #851: use modular pouchdb

2017-02-20 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/851 +1. Thanks @nolanlawson I've merged it in. --- 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

[GitHub] couchdb-fauxton issue #849: Currently reducers seem to be handled quite stra...

2017-02-19 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/849 +1 this is really cool. Could you rebase from master, I think we had an issue with the docker image. The tests should pass after that and then I can merge. --- If your project is set

[GitHub] couchdb-fauxton issue #856: use travis_retry in travis script

2017-02-19 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/856 +1 --- 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 enabled and wishes so

[GitHub] couchdb-fauxton issue #856: use travis_retry in travis script

2017-02-19 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/856 I don't have admin access. Because of the way it links to Apache. Committers don't get admin access. We working on changing that soon All misspelling thanks to my iPhone

[GitHub] couchdb-fauxton pull request #855: Update CONTRIBUTING with quickstart instr...

2017-02-19 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/855#discussion_r101925931 --- Diff: CONTRIBUTING.md --- @@ -8,6 +8,45 @@ with the framework you'd need to complete the task. The [Readme file](https

[GitHub] couchdb-fauxton pull request #849: Currently reducers seem to be handled qui...

2017-02-15 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/849#discussion_r101277979 --- Diff: app/core/base.js --- @@ -157,6 +157,10 @@ FauxtonAPI.setSession = function (newSession) { return FauxtonAPI.session.fetchUser

[GitHub] couchdb-fauxton issue #848: Fix build & config addon

2017-02-15 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/848 @skorobkov could you close this PR for me. I can't --- 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

[GitHub] couchdb-fauxton issue #852: trim moment locale using IgnorePlugin

2017-02-15 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/852 This is awesome. Managing the bundle size is definitely a neglected side of Fauxton. The builds are taking long to run today. But soon as they have built I will merge. --- If your

[GitHub] couchdb-fauxton pull request #849: Currently reducers seem to be handled qui...

2017-02-15 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/849#discussion_r101262280 --- Diff: app/core/base.js --- @@ -157,6 +157,10 @@ FauxtonAPI.setSession = function (newSession) { return FauxtonAPI.session.fetchUser

[GitHub] couchdb-fauxton pull request #849: Currently reducers seem to be handled qui...

2017-02-15 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/849#discussion_r101262321 --- Diff: app/core/base.js --- @@ -157,6 +157,10 @@ FauxtonAPI.setSession = function (newSession) { return FauxtonAPI.session.fetchUser

[GitHub] couchdb-fauxton pull request #833: remove backbone from auth

2017-02-10 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/833#discussion_r100530991 --- Diff: app/addons/auth/components/LoginForm.jsx --- @@ -0,0 +1,101 @@ +import React from "react"; +import ReactDOM from

[GitHub] couchdb-fauxton pull request #833: remove backbone from auth

2017-02-10 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/833#discussion_r100529877 --- Diff: app/addons/auth/base.js --- @@ -61,50 +71,16 @@ Auth.initialize = function () { className: 'logout

[GitHub] couchdb-fauxton pull request #833: remove backbone from auth

2017-02-10 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/833#discussion_r100531254 --- Diff: app/addons/auth/stores/ChangePassword.js --- @@ -0,0 +1,57 @@ +import FauxtonAPI from "../../../core/api";

[GitHub] couchdb-fauxton pull request #833: remove backbone from auth

2017-02-10 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/833#discussion_r100530478 --- Diff: app/addons/auth/components/LoginForm.jsx --- @@ -0,0 +1,101 @@ +import React from "react"; +import ReactDOM from

[GitHub] couchdb-fauxton pull request #833: remove backbone from auth

2017-02-10 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/833#discussion_r100529683 --- Diff: app/addons/auth/actiontypes.js --- @@ -2,24 +2,24 @@ // use this file except in compliance with the License. You may obtain

[GitHub] couchdb-fauxton pull request #833: remove backbone from auth

2017-02-10 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/833#discussion_r100529913 --- Diff: app/addons/auth/base.js --- @@ -61,50 +71,16 @@ Auth.initialize = function () { className: 'logout

[GitHub] couchdb-fauxton pull request #833: remove backbone from auth

2017-02-10 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/833#discussion_r100531532 --- Diff: app/addons/verifyinstall/resources.js --- @@ -11,152 +11,159 @@ // the License. import app from "../.

[GitHub] couchdb-fauxton pull request #833: remove backbone from auth

2017-02-10 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/833#discussion_r100528884 --- Diff: .babelrc --- @@ -3,5 +3,5 @@ "react", "es2015" ], - "plugins&qu

[GitHub] couchdb-fauxton pull request #833: remove backbone from auth

2017-02-10 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/833#discussion_r100529377 --- Diff: app/addons/auth/actions.js --- @@ -10,148 +10,162 @@ // License for the specific language governing permissions and limitations

[GitHub] couchdb-fauxton pull request #833: remove backbone from auth

2017-02-10 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/833#discussion_r100529625 --- Diff: app/addons/auth/actions.js --- @@ -10,148 +10,162 @@ // License for the specific language governing permissions and limitations

[GitHub] couchdb-fauxton pull request #833: remove backbone from auth

2017-02-10 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/833#discussion_r100529500 --- Diff: app/addons/auth/actions.js --- @@ -10,148 +10,162 @@ // License for the specific language governing permissions and limitations

[GitHub] couchdb-fauxton pull request #833: remove backbone from auth

2017-02-10 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/833#discussion_r100529832 --- Diff: app/addons/auth/base.js --- @@ -44,7 +54,7 @@ Auth.initialize = function () { } else if (session.isLoggedIn

[GitHub] couchdb-fauxton pull request #833: remove backbone from auth

2017-02-10 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/833#discussion_r100528767 --- Diff: .eslintrc --- @@ -31,9 +31,10 @@ "react/jsx-uses-vars": 2, "react/jsx-uses-react": 2 },

[GitHub] couchdb-fauxton issue #847: lodash updates switching from _.contains to _.in...

2017-02-10 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/847 +1. I've merged. Can you close this PR --- 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

[GitHub] couchdb-fauxton issue #846: more library updates

2017-02-07 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/846 merged --- 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 enabled and wishes so

[GitHub] couchdb-fauxton pull request #846: more library updates

2017-02-07 Thread garrensmith
Github user garrensmith closed the pull request at: https://github.com/apache/couchdb-fauxton/pull/846 --- 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 enabled and wishes so

[GitHub] couchdb-fauxton issue #833: remove backbone from auth

2017-02-07 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/833 Sure will do. I’m traveling and then at couchdb summit. So It will take me a few days to get to it. On 07 Feb 2017, 4:32 PM +0200, Tim Pinington <notificati...@github.

[GitHub] couchdb-fauxton pull request #846: more library updates

2017-02-07 Thread garrensmith
GitHub user garrensmith opened a pull request: https://github.com/apache/couchdb-fauxton/pull/846 more library updates You can merge this pull request into a Git repository by running: $ git pull https://github.com/garrensmith/couchdb-fauxton more-lib-updates Alternatively

[GitHub] couchdb-fauxton pull request #845: Lib upgrades

2017-02-07 Thread garrensmith
GitHub user garrensmith opened a pull request: https://github.com/apache/couchdb-fauxton/pull/845 Lib upgrades Upgrading some of our dependencies You can merge this pull request into a Git repository by running: $ git pull https://github.com/garrensmith/couchdb-fauxton lib

[GitHub] couchdb-fauxton pull request #844: Make db names more random

2017-02-06 Thread garrensmith
GitHub user garrensmith opened a pull request: https://github.com/apache/couchdb-fauxton/pull/844 Make db names more random You can merge this pull request into a Git repository by running: $ git pull https://github.com/garrensmith/couchdb-fauxton improve-encoding-test

[GitHub] couchdb-fauxton issue #802: COUCHDB-3224: Fixed view displaying for database...

2017-02-06 Thread garrensmith
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] couchdb-fauxton pull request #843: fix encoding in views

2017-02-06 Thread garrensmith
Github user garrensmith closed the pull request at: https://github.com/apache/couchdb-fauxton/pull/843 --- 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 enabled and wishes so

[GitHub] couchdb-fauxton issue #843: fix encoding in views

2017-02-06 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/843 Merged --- 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 enabled and wishes so

[GitHub] couchdb-fauxton pull request #843: fix encoding in views

2017-02-02 Thread garrensmith
GitHub user garrensmith opened a pull request: https://github.com/apache/couchdb-fauxton/pull/843 fix encoding in views You can merge this pull request into a Git repository by running: $ git pull https://github.com/garrensmith/couchdb-fauxton fix-more-encoding Alternatively

[GitHub] couchdb-fauxton pull request #842: use the fetch mock library

2017-02-02 Thread garrensmith
GitHub user garrensmith opened a pull request: https://github.com/apache/couchdb-fauxton/pull/842 use the fetch mock library You can merge this pull request into a Git repository by running: $ git pull https://github.com/garrensmith/couchdb-fauxton change-mock Alternatively

[GitHub] couchdb-fauxton pull request #841: Convert copy dependencies to use fs-extra

2017-01-25 Thread garrensmith
GitHub user garrensmith opened a pull request: https://github.com/apache/couchdb-fauxton/pull/841 Convert copy dependencies to use fs-extra You can merge this pull request into a Git repository by running: $ git pull https://github.com/garrensmith/couchdb-fauxton cp-verbose

[GitHub] couchdb-fauxton pull request #839: some fixes for jest to work with react-bo...

2017-01-24 Thread garrensmith
GitHub user garrensmith opened a pull request: https://github.com/apache/couchdb-fauxton/pull/839 some fixes for jest to work with react-bootstrap You can merge this pull request into a Git repository by running: $ git pull https://github.com/garrensmith/couchdb-fauxton jest

[GitHub] couchdb-fauxton pull request #838: make the query options in the header opti...

2017-01-24 Thread garrensmith
Github user garrensmith closed the pull request at: https://github.com/apache/couchdb-fauxton/pull/838 --- 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 enabled and wishes so

[GitHub] couchdb-fauxton issue #838: make the query options in the header optional

2017-01-24 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/838 merged --- 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 enabled and wishes so

[GitHub] couchdb-fauxton issue #837: Add an ETA field for active tasks tab

2017-01-20 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/837 Awesome. Let me know if I can help in anyway On 20 Jan 2017, 6:30 PM +0200, Kevin J. Qiu <notificati...@github.com>, wrote: > @garrensmith Yeah that's a good idea

[GitHub] couchdb-fauxton issue #837: Add an ETA field for active tasks tab

2017-01-19 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/837 @rnewson brings up a really good point. Since active tasks is polling in the background. Instead of calculating the ETA on one request. Could you rather wait for say 3 - 5 requests

[GitHub] couchdb-fauxton issue #835: 81371 - Unwanted scrollbars on notification cent...

2017-01-19 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/835 +1 --- 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 enabled and wishes so

[GitHub] couchdb-fauxton issue #837: Add an ETA field for active tasks tab

2017-01-19 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/837 @kevinjqiu thanks for this PR. Its great that you are contributing. The only thing is how accurate do you think that ETA is? Do we want to show users an ETA where we cannot guarantee

[GitHub] couchdb-fauxton issue #836: 81574 - API Url test intermittently fails

2017-01-19 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/836 +1 --- 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 enabled and wishes so

[GitHub] couchdb-fauxton issue #833: remove backbone from auth

2017-01-18 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/833 No problem. Let me know if I can help. You can ping me on irc `garrensmith`. Also for fast testing you can run `npm run devtests` then edit `tests/dev.js` to point to the test files you

[GitHub] couchdb-fauxton pull request #833: remove backbone from auth

2017-01-17 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/833#discussion_r96427095 --- Diff: app/core/base.js --- @@ -9,9 +9,11 @@ // WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the // License

[GitHub] couchdb-fauxton pull request #833: remove backbone from auth

2017-01-17 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/833#discussion_r96427042 --- Diff: app/core/base.js --- @@ -159,4 +161,15 @@ FauxtonAPI.setSession = function (newSession) { FauxtonAPI.reducers

[GitHub] couchdb-fauxton pull request #833: remove backbone from auth

2017-01-17 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/833#discussion_r96420216 --- Diff: app/addons/config/routes.js --- @@ -16,10 +16,10 @@ import CORSActions from "../cors/actions"; import ClusterAc

[GitHub] couchdb-fauxton pull request #833: remove backbone from auth

2017-01-17 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/833#discussion_r96420591 --- Diff: app/core/auth.js --- @@ -11,50 +11,30 @@ // the License. import FauxtonAPI from "./base"; -import Bac

[GitHub] couchdb-fauxton pull request #833: remove backbone from auth

2017-01-17 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/833#discussion_r96420470 --- Diff: app/core/auth.js --- @@ -11,50 +11,30 @@ // the License. import FauxtonAPI from "./base"; -import Bac

[GitHub] couchdb-fauxton pull request #834: Small fixes

2017-01-17 Thread garrensmith
Github user garrensmith closed the pull request at: https://github.com/apache/couchdb-fauxton/pull/834 --- 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 enabled and wishes so

[GitHub] couchdb-fauxton issue #834: Small fixes

2017-01-17 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/834 Merged --- 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 enabled and wishes so

[GitHub] couchdb-nano issue #4: Multiple fixes

2017-01-17 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-nano/pull/4 Thanks @glynnbird, this is great. I've squashed and merged --- 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

[GitHub] couchdb-fauxton pull request #834: Small fixes

2017-01-17 Thread garrensmith
GitHub user garrensmith opened a pull request: https://github.com/apache/couchdb-fauxton/pull/834 Small fixes This fixes replication edit to show all the correct info. Also fixes the styling for the styled select in the edit view and clone view You can merge this pull request

[GitHub] couchdb-fauxton issue #833: remove backbone from auth

2017-01-16 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/833 @timmak this looks great. I'll try and give a proper review later today or latest tomorrow. Could you try and fix the tests and make sure they pass. --- If your project is set up

[GitHub] couchdb-fauxton pull request #832: simplify editDocumentsFromView test

2017-01-12 Thread garrensmith
Github user garrensmith closed the pull request at: https://github.com/apache/couchdb-fauxton/pull/832 --- 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 enabled and wishes so

[GitHub] couchdb-fauxton pull request #832: simplify editDocumentsFromView test

2017-01-12 Thread garrensmith
GitHub user garrensmith reopened a pull request: https://github.com/apache/couchdb-fauxton/pull/832 simplify editDocumentsFromView test You can merge this pull request into a Git repository by running: $ git pull https://github.com/garrensmith/couchdb-fauxton simplify-edit

[GitHub] couchdb-fauxton pull request #832: simplify editDocumentsFromView test

2017-01-12 Thread garrensmith
GitHub user garrensmith opened a pull request: https://github.com/apache/couchdb-fauxton/pull/832 simplify editDocumentsFromView test You can merge this pull request into a Git repository by running: $ git pull https://github.com/garrensmith/couchdb-fauxton simplify-edit

[GitHub] couchdb-nano issue #3: add reference to server in doc scope

2017-01-12 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-nano/pull/3 @dominicbarnes could you close this PR now. --- 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

[GitHub] couchdb-nano issue #3: add reference to server in doc scope

2017-01-12 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-nano/pull/3 Thanks. I've merged it in. --- 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 enabled

[GitHub] couchdb-fauxton pull request #831: clean up app.js

2017-01-11 Thread garrensmith
GitHub user garrensmith reopened a pull request: https://github.com/apache/couchdb-fauxton/pull/831 clean up app.js webpack exposes jquery so this isn't needed You can merge this pull request into a Git repository by running: $ git pull https://github.com/garrensmith/couchdb

[GitHub] couchdb-fauxton pull request #831: clean up app.js

2017-01-11 Thread garrensmith
Github user garrensmith closed the pull request at: https://github.com/apache/couchdb-fauxton/pull/831 --- 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 enabled and wishes so

[GitHub] couchdb-fauxton issue #828: 76043 - tweaking hover color for attachment link...

2017-01-11 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/828 +1 --- 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 enabled and wishes so

[GitHub] couchdb-fauxton pull request #830: remove all unused variables and imports

2017-01-10 Thread garrensmith
GitHub user garrensmith opened a pull request: https://github.com/apache/couchdb-fauxton/pull/830 remove all unused variables and imports Enable the eslint `no-unused-vars`. This just makes our code a little neater. You can merge this pull request into a Git repository by running

[GitHub] couchdb-fauxton pull request #829: Make clipboard copy tests more robust

2017-01-09 Thread garrensmith
Github user garrensmith closed the pull request at: https://github.com/apache/couchdb-fauxton/pull/829 --- 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 enabled and wishes so

[GitHub] couchdb-fauxton issue #827: Allow hyphen in key name when matching edit line

2017-01-09 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/827 Thanks @Flamenco this is a great find and a good fix. The changes have been merged into master --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] couchdb-fauxton pull request #829: Make clipboard copy tests more robust

2017-01-09 Thread garrensmith
GitHub user garrensmith opened a pull request: https://github.com/apache/couchdb-fauxton/pull/829 Make clipboard copy tests more robust You can merge this pull request into a Git repository by running: $ git pull https://github.com/garrensmith/couchdb-fauxton make-copytest

[GitHub] couchdb-fauxton issue #823: remove tray and use only traycontents

2016-12-15 Thread garrensmith
Github user garrensmith commented on the issue: https://github.com/apache/couchdb-fauxton/pull/823 merged --- 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 enabled and wishes so

  1   2   3   4   5   >