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

Document createShallowWithTheme #641

Merged
merged 3 commits into from
Jun 25, 2018

Conversation

mrniket
Copy link
Contributor

@mrniket mrniket commented Jun 25, 2018

image

This change is Reviewable

@mrniket mrniket changed the base branch from master to UI_new_design June 25, 2018 10:10
@OlafSzmidt
Copy link
Contributor

Review status: 0 of 4 files reviewed, 1 unresolved discussion (waiting on @mrniket)


game_frontend/src/redux/store.js, line 5 at r1 (raw file):

import { createEpicMiddleware } from 'redux-observable'
import api from './api'
import { composeWithDevTools } from 'redux-devtools-extension/logOnlyInProduction'

How does this work? logOnlyInProduction?


Comments from Reviewable

@mrniket
Copy link
Contributor Author

mrniket commented Jun 25, 2018

Review status: 0 of 4 files reviewed, 1 unresolved discussion (waiting on @mrniket)


game_frontend/src/redux/store.js, line 5 at r1 (raw file):

Previously, OlafSzmidt (Olaf Szmidt) wrote…

How does this work? logOnlyInProduction?

We can use redux-devtools-extension in production, but the logOnly version (a subset of the devtools package). This means the performance of the webapp isn't affected too much in production.

Here is more information on the topic: https://medium.com/@zalmoxis/using-redux-devtools-in-production-4c5b56c5600f


Comments from Reviewable

@CelineBoudier
Copy link
Contributor

Reviewed 4 of 4 files at r1.
Review status: all files reviewed, 1 unresolved discussion (waiting on @OlafSzmidt)


Comments from Reviewable

@CelineBoudier
Copy link
Contributor

Review status: all files reviewed, 2 unresolved discussions (waiting on @OlafSzmidt and @mrniket)


game_frontend/package.json, line 40 at r1 (raw file):

    "redux": "^3.7.2",
    "redux-observable": "^0.18.0",
    "redux-devtools-extension": "^2.13.2",

sounds like a dev dependency? Am I missing the difference between devdependencies and dependencies?


Comments from Reviewable

@mrniket
Copy link
Contributor Author

mrniket commented Jun 25, 2018

Review status: all files reviewed, 2 unresolved discussions (waiting on @OlafSzmidt and @mrniket)


game_frontend/package.json, line 40 at r1 (raw file):

Previously, CelineBoudier (Celine Boudier) wrote…

sounds like a dev dependency? Am I missing the difference between devdependencies and dependencies?

I thought so too (which it was originally there). Then the deploy build phase failed on travis since that wasn't installed. So I looked into it and it has its uses in production. Here is the article for more information: https://medium.com/@zalmoxis/using-redux-devtools-in-production-4c5b56c5600f


Comments from Reviewable

@OlafSzmidt
Copy link
Contributor

:lgtm:


Reviewed 1 of 4 files at r1.
Review status: all files reviewed, 1 unresolved discussion (waiting on @CelineBoudier)


Comments from Reviewable

@CelineBoudier
Copy link
Contributor

:lgtm:


Review status: :shipit: complete! all files reviewed, all discussions resolved


Comments from Reviewable

@CelineBoudier CelineBoudier merged commit 61e080a into UI_new_design Jun 25, 2018
@CelineBoudier CelineBoudier deleted the doc_createShallowWithTheme branch June 25, 2018 13:44
OlafSzmidt pushed a commit that referenced this pull request Jul 4, 2018
* New UI: Move the UI contents into a new branch until fully functional (#547)

* setup code responses for react

* Add Editor Redux feature

* code review changes

* link editor code with django

right now the game id is fixed to 1

* test get code button

* Add GamePage snapshot test

* add post code epic

* Sends post request successfully

* add a way to get csrf token via ajax

* update npm packages

* add post request for code

* simplify passing of epics

* move api calls into an api folder

* simplify post api call

* simplify syntax

* Fix tests and add some more

* New UI: Code editor frontend (#548)

* setup code responses for react

* Add Editor Redux feature

* code review changes

* link editor code with django

right now the game id is fixed to 1

* Merge branch 'master' into django_editor_link

* test get code button

* Add GamePage snapshot test

* add post code epic

* Sends post request successfully

* add a way to get csrf token via ajax

* update npm packages

* add post request for code

* simplify passing of epics

* move api calls into an api folder

* simplify post api call

* simplify syntax

* Fix tests and add some more

* Add front-end component for AceEditor

* Modified theme of code editor

* Merge branch 'UI_new_design' into code_editor_frontend

* Fixed failing test and updated snapshot for code editor

* Merge branch 'code_editor_frontend' of https://github.com/ocadotechnology/aimmo into code_editor_frontend

* Editor state management (#555)

* Get intial state of code on editor load

* Added more types to handle changing code

* Wrote a test case for changeCodeEpic

* fix deps

* Inject testscheduler into changeCodeEpic

* State in post request should be evaluated at the time of the action

* fix eslint in GamePage tests

* code review changes

* Add Material Design to AI:MMO with a Basic Colour Palette (#618)

* Add material-ui core

* Add mui theme to MUIThemeProvider and styled components ThemeProvider

* Use theme colors for existing ui

I didn't do the Unity one because it's going to not have a background

* delete unused file

* add a withTheme testHelper

* Use withTheme testHelper

* Update snapshots

* Use absolute imports for testhelpers

* Merge branch 'master' into UI_new_design

* Fix build

* Add typography (#620)

* Add typography

* withTheme is a default export

* Fix tests so that they are testing the correct code

* Merge branch 'master'

Conflicts:
	game_frontend/package-lock.json
	game_frontend/package.json
	game_frontend/yarn.lock
	players/views.py

* JS: Update dependencies and lock files

* Add more snapshot tests for containers

* Merge with UI_new_design

* Document createShallowWithTheme (#641)

* Add documentation for `createShallowWithTheme` test helper

* Remove feature path

* Fix redux-dev-tools deployment bug

* [New UI] Unity Build in React (#616)

* Add react-unity-webgl package as a dependency

* Introduce new unity module to the component

* Update the README

* Move GameView to be a container

* Add some extra functionality local to the file for now

* Add utilities and a API with JSON settings. Resolves #612

* More functionality to connect two systems

* More Redux changes for success & connection

* Refactor method name of the view to not include 'view'

* Add initial state & send information across to Unity WebGL

* Add socketIO dependency to generated HTML

* Call Unity functions as redux actions now

* Remove irrelevant comments now

* Serialise boolean SSL flag for Unity

* More changes to how communication is conducted

* Handle errors

* Add lunch json for VS code to git ignore

* Emit success event in the epic

* Remove middlewares

* New feature introduced for Game

* Remove old game reducer and move to its own. Resolves #619

Same as below

* Separate epics for each event refactoring

* Remove older epic

* Update .gitignore launch json path
Ignore all .vscode files


Update .gitignore

* Remove a TODO comment

* Fix existing tests to suit new code

* Merge branch 'UI_new_design' into add_unity_to_react

* Regenerate yarn.lock

* Revert the refactoring, can be done in a different PR

* Add epic tests for the happy path

* Remove unwanted comment and add EOF lines

* Change reducer name, forgot to change beforehand

* Add reducer test for all current cases

* More function tests

* Some testing changes as requested in PR by @mrniket

* Change imports to one liner using aliases

* Remove redundant alias for default import

* Refactoring of setGameURL successful, rest not done

* Refactoring of setGameURL fixed

* Finish refactoring

* Merge remote-tracking branch 'origin/UI_new_design' into add_unity_to_react

* GameView minor esLint changes

* Merge remote-tracking branch 'origin/UI_new_design' into add_unity_to_react

* Fix indentation and other eslint issues in unity

* Certain eslint fixes

* More eslint

* Fix minor SSL bug

* Add extra developer settings to the parcel bundler

* Initial connectionParams naming changes


More naming changes


More naming changes again


connectionParams naming change done

* Name change in the view to parameters

* utilities called game_helpers now

* Add another shallow theme test to the layout

* Test passing, still unhappy with marble completion

* Use action creators instead of explicit objects

* Add css to Game Page testing

* Change the djangoBundler check environment before setting vars

* Update tests without theme to use shallow instead

* Export unity as a object containing all the calls instead

* Bring id of current avatar view to be called game_id again

* Add naming change to game_helpers as requested

* Fix tests to be correct now

* Make aimmo compatible with Django 1.9 (#642)

* Make aimmo compatible with Django 1.9

* Using patch version of Django 1.9 (#643)

* Using patch version of Django 1.9

* Merge remote-tracking branch 'origin/master' into add_unity_to_react

* Simply the API for api.js, put error in payload of fail actions

* Add test for emit unity failure

* Merge branch 'master' into UI_new_design
OlafSzmidt pushed a commit that referenced this pull request Jul 4, 2018
* New UI: Move the UI contents into a new branch until fully functional (#547)

* setup code responses for react

* Add Editor Redux feature

* code review changes

* link editor code with django

right now the game id is fixed to 1

* test get code button

* Add GamePage snapshot test

* add post code epic

* Sends post request successfully

* add a way to get csrf token via ajax

* update npm packages

* add post request for code

* simplify passing of epics

* move api calls into an api folder

* simplify post api call

* simplify syntax

* Fix tests and add some more

* New UI: Code editor frontend (#548)

* setup code responses for react

* Add Editor Redux feature

* code review changes

* link editor code with django

right now the game id is fixed to 1

* Merge branch 'master' into django_editor_link

* test get code button

* Add GamePage snapshot test

* add post code epic

* Sends post request successfully

* add a way to get csrf token via ajax

* update npm packages

* add post request for code

* simplify passing of epics

* move api calls into an api folder

* simplify post api call

* simplify syntax

* Fix tests and add some more

* Add front-end component for AceEditor

* Modified theme of code editor

* Merge branch 'UI_new_design' into code_editor_frontend

* Fixed failing test and updated snapshot for code editor

* Merge branch 'code_editor_frontend' of https://github.com/ocadotechnology/aimmo into code_editor_frontend

* Editor state management (#555)

* Get intial state of code on editor load

* Added more types to handle changing code

* Wrote a test case for changeCodeEpic

* fix deps

* Inject testscheduler into changeCodeEpic

* State in post request should be evaluated at the time of the action

* fix eslint in GamePage tests

* code review changes

* Add Material Design to AI:MMO with a Basic Colour Palette (#618)

* Add material-ui core

* Add mui theme to MUIThemeProvider and styled components ThemeProvider

* Use theme colors for existing ui

I didn't do the Unity one because it's going to not have a background

* delete unused file

* add a withTheme testHelper

* Use withTheme testHelper

* Update snapshots

* Use absolute imports for testhelpers

* Merge branch 'master' into UI_new_design

* Fix build

* Add typography (#620)

* Add typography

* withTheme is a default export

* Fix tests so that they are testing the correct code

* Merge branch 'master'

Conflicts:
	game_frontend/package-lock.json
	game_frontend/package.json
	game_frontend/yarn.lock
	players/views.py

* JS: Update dependencies and lock files

* Add more snapshot tests for containers

* Merge with UI_new_design

* Document createShallowWithTheme (#641)

* Add documentation for `createShallowWithTheme` test helper

* Remove feature path

* Fix redux-dev-tools deployment bug

* [New UI] Unity Build in React (#616)

* Add react-unity-webgl package as a dependency

* Introduce new unity module to the component

* Update the README

* Move GameView to be a container

* Add some extra functionality local to the file for now

* Add utilities and a API with JSON settings. Resolves #612

* More functionality to connect two systems

* More Redux changes for success & connection

* Refactor method name of the view to not include 'view'

* Add initial state & send information across to Unity WebGL

* Add socketIO dependency to generated HTML

* Call Unity functions as redux actions now

* Remove irrelevant comments now

* Serialise boolean SSL flag for Unity

* More changes to how communication is conducted

* Handle errors

* Add lunch json for VS code to git ignore

* Emit success event in the epic

* Remove middlewares

* New feature introduced for Game

* Remove old game reducer and move to its own. Resolves #619

Same as below

* Separate epics for each event refactoring

* Remove older epic

* Update .gitignore launch json path
Ignore all .vscode files


Update .gitignore

* Remove a TODO comment

* Fix existing tests to suit new code

* Merge branch 'UI_new_design' into add_unity_to_react

* Regenerate yarn.lock

* Revert the refactoring, can be done in a different PR

* Add epic tests for the happy path

* Remove unwanted comment and add EOF lines

* Change reducer name, forgot to change beforehand

* Add reducer test for all current cases

* More function tests

* Some testing changes as requested in PR by @mrniket

* Change imports to one liner using aliases

* Remove redundant alias for default import

* Refactoring of setGameURL successful, rest not done

* Refactoring of setGameURL fixed

* Finish refactoring

* Merge remote-tracking branch 'origin/UI_new_design' into add_unity_to_react

* GameView minor esLint changes

* Merge remote-tracking branch 'origin/UI_new_design' into add_unity_to_react

* Fix indentation and other eslint issues in unity

* Certain eslint fixes

* More eslint

* Fix minor SSL bug

* Add extra developer settings to the parcel bundler

* Initial connectionParams naming changes


More naming changes


More naming changes again


connectionParams naming change done

* Name change in the view to parameters

* utilities called game_helpers now

* Add another shallow theme test to the layout

* Test passing, still unhappy with marble completion

* Use action creators instead of explicit objects

* Add css to Game Page testing

* Change the djangoBundler check environment before setting vars

* Update tests without theme to use shallow instead

* Export unity as a object containing all the calls instead

* Bring id of current avatar view to be called game_id again

* Add naming change to game_helpers as requested

* Fix tests to be correct now

* Make aimmo compatible with Django 1.9 (#642)

* Make aimmo compatible with Django 1.9

* Using patch version of Django 1.9 (#643)

* Using patch version of Django 1.9

* Merge remote-tracking branch 'origin/master' into add_unity_to_react

* Simply the API for api.js, put error in payload of fail actions

* Add test for emit unity failure

* Merge branch 'master' into UI_new_design

* Fix linter error

* Use the new game_ide ui for AI:MMO and delete the old one 🙌

* Merge branch 'master' into UI_new_design

* Change travis.yml to build new UI before running all_tests


Silly mistake
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants