Next release

.github/CODEOWNERS

@@ -0,0 +1,4 @@
+# See https://help.github.com/articles/about-codeowners/
+
+# A Conveyal employee is required to approve PR merges
+* @conveyal/employees

.github/issue_template.md

@@ -0,0 +1,21 @@
+_**NOTE:** This issue system is intended for reporting bugs and tracking progress in software development. Although this software is licensed with an open-source license, any issue opened here may not be responded to in a timely manner. [Conveyal](https://www.conveyal.com) is unable to provide technical support for custom deployments of this software unless your company has a support contract with us. Please remove this note when creating the issue._
+
+## Observed behavior (please include a screenshot if possible)
+
+Please explain what is being observed within the application here.
+
+## Expected behavior
+
+Please explain what should happen instead.
+
+## Steps to reproduce the problem
+
+Please be as specific as possible.
+
+## Any special notes on configuration used
+
+Please describe any applicable config files that were used
+
+## Version of datatools-ui and datatools-server if applicable (exact commit hash or branch name)
+
+This info can be found by clicking on the gear icon on the sidebar

.github/pull_request_template.md

@@ -0,0 +1,14 @@
+### Checklist
+
+- [ ] Appropriate branch selected _(all PRs must first be merged to `dev` before they can be merged to `master`)_
+- [ ] Any modified or new methods or classes have helpful JSDoc and code is thoroughly commented
+- [ ] The description lists all applicable issues this PR seeks to resolve
+- [ ] The description lists any configuration setting(s) that differ from the default settings
+- [ ] All tests and CI builds passing
+- [ ] The description lists all relevant PRs included in this release _(remove this if not merging to master)_
+- [ ] e2e tests are all passing _(remove this if not merging to master)_
+- [ ] Code coverage does not significantly worsen (ideally it improves) _(remove this if not merging to master)_
+
+### Description
+
+Please explain the changes you made here and, if not immediately obvious from the code, how they resolve any referenced issues (and PRs if merging to master). Be sure to include all issues being resolved and any special configuration settings that are need for the software to run properly with these changes.

.gitignore

@@ -3,8 +3,10 @@ node_modules/
 dist
 *.log
 coverage*
+e2e-test-results
 tmp/
 .tags
+*.pid
 
 # Optional npm cache directory
 .npm
@@ -25,3 +27,4 @@ env.yml
 env.yml-original
 .env
 !configurations/test/env.yml
+scripts/*client.json

.travis.yml

@@ -1,25 +1,49 @@
-language: node_js
+# although this is a node.js project, specify the use of Java in order for OTP
+# and datatools-server to run as expected on Java 8 specifically. Although not
+# confirmed in any way, another thought with using a Java-specific image is that
+# some things may be more optimized for Java thus making for more expedient and
+# reliable execution of Java code.
+dist: trusty # jdk 8 not available on xenial
+language: java
+java:
+  - oraclejdk8
+install: true
 notifications:
   email: false
-  slack: conveyal:WQxmWiu8PdmujwLw4ziW72Gc
-node_js:
-  - '8'
+services:
+  # needed for e2e tests to start datatools-server
+  - mongodb
+  - postgresql
+addons:
+  postgresql: 9.6
 cache:
-  yarn: true
+  directories:
+    - $HOME/.cache/yarn
+    - $HOME/.cache/pip
 before_install:
-  - npm i -g codecov
-  # Use updated python to avoid SSL insecure warnings:
-  # https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings
-  - python --version
-  - pyenv versions
-  - pyenv global 2.7.14
+  # install node 12
+  - nvm install 12
+  # install yarn
+  - curl -o- -L https://yarnpkg.com/install.sh | bash
   - pip install --user mkdocs
+  - source ./scripts/check-if-e2e-should-run-on-travis.sh
+  # create database for e2e tests
+  - if [ "$SHOULD_RUN_E2E" = "true" ]; then psql -U postgres -c 'CREATE DATABASE catalogue;'; fi
+  # add aws credentials for datatools-server
+  - if [ "$SHOULD_RUN_E2E" = "true" ]; then mkdir ~/.aws && printf '%s\n' '[default]' 'aws_access_key_id=${AWS_ACCESS_KEY_ID}' 'aws_secret_access_key=${AWS_SECRET_ACCESS_KEY}' 'region=us-east-1' > ~/.aws/config; else mkdir ~/.aws && printf '%s\n' '[default]' 'aws_access_key_id=foo' 'aws_secret_access_key=bar' 'region=us-east-1' > ~/.aws/config; fi
 script:
+  - yarn
   - yarn run lint
   - yarn run lint-messages
   - yarn run flow
   - yarn run cover-client
-  - codecov
+  # upload coverage results from unit tests and then delete coverage reports to
+  # avoid uploading the same coverage results twice
+  - bash <(curl -s https://codecov.io/bash) -c -F unit_tests
+  - if [ "$SHOULD_RUN_E2E" = "true" ]; then yarn run cover-end-to-end; fi
+  # upload coverage results from e2e tests and then delete coverage reports to
+  # avoid uploading the same coverage results twice
+  - if [ "$SHOULD_RUN_E2E" = "true" ]; then bash <(curl -s https://codecov.io/bash) -c -F end_to_end_tests; fi
   - yarn run build -- --minify
   - mkdocs build
 
@@ -28,5 +52,5 @@ sudo: false
 
 # Push results to codecov.io
 after_success:
-  - bash <(curl -s https://codecov.io/bash)
-  - yarn run semantic-release
+  # only deploy the release to github as the package is not needed on npm
+  - semantic-release --prepare false --publish @semantic-release/github --verify-conditions @semantic-release/github

README.md

@@ -11,3 +11,13 @@ This repository serves as the front end UI for the Data Manager application. It
 View the [latest release documentation](http://conveyal-data-tools.readthedocs.org/en/latest/) at ReadTheDocs for more info on deployment and development as well as a user guide.
 
 Note: `dev` branch docs can be found [here](http://conveyal-data-tools.readthedocs.org/en/dev/).
+
+## Shoutouts 🙏
+
+<img src="browserstack-logo-600x315.png" height="80" title="BrowserStack Logo" alt="BrowserStack Logo" />
+
+Big thanks to [BrowserStack](https://www.browserstack.com) for letting the maintainers use their service to debug browser issues.
+
+<img src="https://www.graphhopper.com/wp-content/uploads/2018/03/graphhopper-logo-small.png" height="25" alt="GraphHopper Logo" />
+
+Street snapping powered by the <a href="https://www.graphhopper.com/">GraphHopper API</a>.