Skip to content

Commit 5ae6e01

Browse files
kbowermakbowerma
authored
Merge pull request #2 from topcoderinc/missionPlanner
Mission planner
2 parents 94182f9 + 52c501b commit 5ae6e01

File tree

64 files changed

+2783
-35
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

64 files changed

+2783
-35
lines changed

‎.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,3 +5,4 @@ node_modules
55
.idea
66
dist
77
coverage
8+
.tmp

‎README.md

Lines changed: 26 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
## DSP app
1+
# dsp-fronted
22

33
## Requirements
44
* node v6 (https://nodejs.org)
@@ -11,7 +11,7 @@
1111

1212

1313
## Configuration
14-
Configuration files are located under `config` dir.
14+
Configuration files are located under `config` dir.
1515
See Guild https://github.com/lorenwest/node-config/wiki/Configuration-Files
1616

1717
|Name|Description|
@@ -32,7 +32,27 @@ See Guild https://github.com/lorenwest/node-config/wiki/Configuration-Files
3232
|`dev`|Start app in the dev mode.|
3333
|`lint`|Lint all `.js` files.|
3434
|`lint:fix`|Lint and fix all `.js` files. [Read more on this](http://eslint.org/docs/user-guide/command-line-interface.html#fix).|
35-
36-
37-
## Video
38-
http://take.ms/WZkTO
35+
|`test`|Run tests using [mocha-webpack](https://github.com/webpack/mocha-loader) for all `*.spec.(js|jsx)` files in the `src` dir.|
36+
37+
## Google Map
38+
In this project module [react-google-maps](https://github.com/tomchentw/react-google-maps) is used to work with google maps. So it can be used for any new functionality.
39+
40+
# Challenges
41+
42+
## [30055900](https://www.topcoder.com/challenge-details/30055900)
43+
## DONE
44+
- All modules were rewritten almost from the scratch because the previous code was very buggy, hard to support and too far from the redux way which is used in the new project. This was the biggest job. Current code is much more robust and is 99% stateless.
45+
- For most important parts detailed unit tests are written.
46+
- Redrawing mission on the map was optimised, no unnecessary redrawing.
47+
- Readme file was cleaned and updated with information about tests and module used to implement google maps for future developers.
48+
49+
## ADDITIONALLY
50+
- These small things from `kbowerma` was added:
51+
- - I know this was not in the challenge req but another thing that would be nice is if the label for PARAM4 changed to “Heading” only if NAV_WAYPOINT is selected. and PARAMA1 label changed to “hold time” only if NAV_WAYPOINT is selected.
52+
- - IT should be, but home and take off should be pinned together with the first click, but then should be able to be dragged or updated with text separately
53+
- All modules integrated with current project styles.
54+
- Test environment was set up. It uses `Mocha`, `Chai` and `Enzyme`. Also, it supports `jsx`, `css-modules` and `webpack resolve aliases`. Even though it's implicitly the scope of the challenge, it was a tangible part.
55+
56+
## NOTES
57+
- As there is no Authorization implemented in the project. In the API I've hardcoded automatic registering and authorization of a dumb user to make requests to the server.
58+
- A lot of files in the repository had the `crlf` line endings. Though `eslint` and `.editorconfig` prescribe using `lf` line endings. So all files were converted to `lf` line endings to pass the linting process and follow configuration.

‎blueprints/component/files/src/components/__name__/__name__.jsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,12 @@ import styles from './<%= pascalEntityName %>.scss';
44

55
export const <%= pascalEntityName %> = () => (
66
<div styleName="<%= dashesEntityName %>">
7+
<%= pascalEntityName %>
78
</div>
89
);
910

1011
<%= pascalEntityName %>.propTypes = {
11-
foo: PropTypes.string.isRequired,
12+
// foo: PropTypes.string.isRequired,
1213
};
1314

1415
export default CSSModules(<%= pascalEntityName %>, styles);

‎blueprints/component/files/src/components/__name__/__name__.scss

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
11
.<%= dashesEntityName %> {
2+
background-color: transparent;
3+
24
:global {
35

46
}

‎blueprints/route/files/src/routes/__name__/components/__name__View.js

Lines changed: 0 additions & 14 deletions
This file was deleted.

‎blueprints/route/files/src/routes/__name__/components/__name__View.jsx

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
import React, {PropTypes} from 'react';
2+
import CSSModules from 'react-css-modules';
3+
import styles from './<%= pascalEntityName %>View.scss';
4+
5+
export const <%= pascalEntityName %>View = () => (
6+
<div styleName="<%= dashesEntityName %>-view">
7+
<%= pascalEntityName %>View
8+
</div>
9+
);
10+
11+
<%= pascalEntityName %>View.propTypes = {
12+
// foo: PropTypes.string.isRequired,
13+
};
14+
15+
export default CSSModules(<%= pascalEntityName %>View, styles);

‎blueprints/route/files/src/routes/__name__/components/__name__View.scss

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
1-
.<%= camelEntityName %>View {
1+
.<%= dashesEntityName %>-view {
2+
background-color: transparent;
3+
24
:global {
35

46
}

‎blueprints/route/files/src/routes/__name__/containers/__name__Container.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { asyncConnect } from 'redux-connect';
2-
import {actions} from '../modules/<%= pascalEntityName %>';
2+
import { actions } from '../modules/<%= pascalEntityName %>';
33

44
import <%= pascalEntityName %>View from '../components/<%= pascalEntityName %>View';
55

‎blueprints/route/files/src/routes/__name__/index.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ export default (store) => ({
66
require.ensure([], (require) => {
77
const <%= pascalEntityName %> = require('./containers/<%= pascalEntityName %>Container').default;
88
const reducer = require('./modules/<%= pascalEntityName %>').default;
9+
910
injectReducer(store, { key: '<%= camelEntityName %>', reducer });
1011
cb(null, <%= pascalEntityName %>);
1112
}, '<%= pascalEntityName %>');

‎blueprints/route/files/src/routes/__name__/modules/__name__.js

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ export const SAMPLE = '<%= pascalEntityName %>/SAMPLE';
1111

1212

1313
export const sample2 = () => async (dispatch, getState) => {
14-
14+
getState(); // to pass eslint from the begining
1515
};
1616

1717
export const actions = {
@@ -23,5 +23,8 @@ export const actions = {
2323
// Reducer
2424
// ------------------------------------
2525
export default handleActions({
26-
[SAMPLE]: (state, {payload}) => state,
26+
[SAMPLE]: (state, {payload}) => {
27+
payload; // to pass eslint from the begining
28+
return state;
29+
},
2730
}, {});

0 commit comments

Comments
 (0)
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy