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

Create new Questionnaire (integration) #140

Merged
merged 38 commits into from
Jun 28, 2017
Merged
Show file tree
Hide file tree
Changes from 34 commits
Commits
Show all changes
38 commits
Select commit Hold shift + click to select a range
fa05b7c
hook apollio client up to store
hamishtaplin Jun 14, 2017
bf20e6a
wire questionnaire meta up to api
hamishtaplin Jun 14, 2017
16bcc23
handle mutations
hamishtaplin Jun 14, 2017
476a2a0
mutation onSubmit of form
hamishtaplin Jun 15, 2017
10138e2
WIP
hamishtaplin Jun 15, 2017
f16bce5
added debounced updating
hamishtaplin Jun 15, 2017
21c4573
Add gql to breadcrumbs
hamishtaplin Jun 16, 2017
46627b2
remove questionnaire ID
hamishtaplin Jun 16, 2017
2dcd597
minor cleanup
hamishtaplin Jun 16, 2017
e79b677
update some tests
hamishtaplin Jun 16, 2017
598f335
tests
hamishtaplin Jun 20, 2017
7bd2e0b
checkbox tests
hamishtaplin Jun 20, 2017
37cec38
more tests
hamishtaplin Jun 20, 2017
d748df1
enhancer test
hamishtaplin Jun 21, 2017
f132836
remove enhancer tests
hamishtaplin Jun 21, 2017
86b2c2d
appease lint gods
hamishtaplin Jun 21, 2017
86d5700
Update tests
hamishtaplin Jun 21, 2017
41c77d8
correct case on import
hamishtaplin Jun 21, 2017
462e029
revert changes to Breadcrumb component
hamishtaplin Jun 21, 2017
a2e6dd0
address code review comments
hamishtaplin Jun 21, 2017
6af3679
undo a whoopsie
hamishtaplin Jun 21, 2017
d9e4ad3
undo another whoopsie
hamishtaplin Jun 21, 2017
0b956cb
update snapshot
hamishtaplin Jun 22, 2017
15f2b41
Review changes
hamishtaplin Jun 22, 2017
d9bec58
slight refactor
hamishtaplin Jun 26, 2017
68af4e4
more refactoring
hamishtaplin Jun 26, 2017
9efad9b
simplify routes
hamishtaplin Jun 26, 2017
11f04c9
proptypes
hamishtaplin Jun 26, 2017
7e59d02
fix existing tests
hamishtaplin Jun 26, 2017
78c63d8
tests
hamishtaplin Jun 27, 2017
bc59cf7
More tests
hamishtaplin Jun 27, 2017
3062c9d
update snapshot
hamishtaplin Jun 27, 2017
63dd3b1
disable smoke tests
hamishtaplin Jun 28, 2017
d183cf5
Address PR comments
hamishtaplin Jun 28, 2017
64163d8
move queries to external files
hamishtaplin Jun 28, 2017
e1905af
moar PR comments
hamishtaplin Jun 28, 2017
6fc3591
add Travis config
hamishtaplin Jun 28, 2017
b1350d5
displayName fix
hamishtaplin Jun 28, 2017
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .env.development
Original file line number Diff line number Diff line change
@@ -1 +1,2 @@
REACT_APP_BASE_NAME=""
REACT_APP_API_URL="http://localhost:4000/graphiql"
1 change: 1 addition & 0 deletions .env.production
Original file line number Diff line number Diff line change
@@ -1 +1,2 @@
REACT_APP_BASE_NAME="/eq-author"
REACT_APP_API_URL=""
Copy link
Contributor

Choose a reason for hiding this comment

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

Are we able to override the REACT_APP_API_URL value with an environment variable. Or do we need to default it to process.env.REACT_APP_API_URL here?

We need to be able to customise this for successful deployment to CloudFoundry.

2 changes: 0 additions & 2 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,6 @@ before_script:
script:
- yarn lint -- --max-warnings=0
- yarn test -- --coverage
- yarn build
- ./scripts/run_smoketests.sh

env:
- secure: "x5tYroOoRdHt8xDpfIoH7Xy0YYg+gApnG2Cii6+mIUjeVJQJyl9gKepKxr+K22IMi+b9WxGJxHvjay9sYxaDFgLyVu/6XCpx74dmU0rnE6nKlYxza6cI5KDYWYS2PzMziKuRzCOb8Hp50gtLzq0LKxtLNB3ajn54r6vx4/jgSQ1GzuMoc419pOnbc56EPWWsk8JguLNRCEx/wIaed33X9Nig5j/ADCnXrIJ65wv1ErCgLSQGZqG+W4SlCbDfW9uzF9whzYMMB3HN/hzlSomwRMVVW2PmjP6n+BGN/RjVAdz0MI3lTsraYLn7C8h5p9xOGja98fTXq7nSFQY9nPjseQi6Q1e+MKpJmVaUkEVtosP8Lpkk/Fduqc5noXvITmxhPodrYZjBPQPFXXKN3OqLHebmZVs71DVfSQcYjcYi/pIUnZlW2h2VSYrlSEHri53evfKvwzTuH6+9xWJkJZYET62agDy2oZb4q7hAZoEXHWV7DxmvD8bHET0Be1cs3Og5RNmwVLjP7vR7KmOtN9Uc6ZA6LbKuJDsscmrA9kmyQ/tWui2qGfXoyrvGF2H0qO2sg9e0vf1fHc6ynKZ/30pRl43Kfm0sXGQmuiNIuQeB92nvUjf7+XYJ036jW2eVz3VvsBbMTTAzectX2tvMruMj74eBQEVuO6I2Kl9mAYA5cd0="
Expand Down
118 changes: 0 additions & 118 deletions data/2_0001.json

This file was deleted.

4 changes: 4 additions & 0 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,7 @@
"prettier": "^1.3.1",
"promise": "7.1.1",
"react-dev-utils": "^3.0.0",
"react-router-test-context": "^0.1.0",
"react-test-renderer": "^15.5.4",
"recursive-readdir": "^2.2.1",
"strip-ansi": "3.0.1",
Expand All @@ -87,11 +88,13 @@
"dependencies": {
"connected-react-router": "^4.2.1",
"enzyme": "^2.8.2",
"global": "^4.3.2",
Copy link
Contributor

Choose a reason for hiding this comment

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

What's this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

An accident!

"lodash": "^4.17.4",
"normalize.css": "^7.0.0",
"normalizr": "^3.2.3",
"react": "^15.5.4",
"react-addons-test-utils": "^15.5.1",
"react-apollo": "^1.4.2",
"react-dom": "^15.5.4",
"react-error-overlay": "^1.0.7",
"react-hot-loader": "next",
Expand All @@ -103,6 +106,7 @@
"redux": "^3.6.0",
"redux-devtools-extension": "^2.13.2",
"redux-thunk": "^2.2.0",
"serve": "^6.0.0",
"styled-components": "^2.0.0",
"tinycolor2": "^1.4.1"
},
Expand Down
69 changes: 0 additions & 69 deletions src/actions/questionnaire/index.js

This file was deleted.

112 changes: 0 additions & 112 deletions src/actions/questionnaire/index.test.js

This file was deleted.

Loading