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

New infra structure #10

Merged
merged 3 commits into from
Oct 20, 2018
Merged

New infra structure #10

merged 3 commits into from
Oct 20, 2018

Conversation

ltyDion
Copy link
Contributor

@ltyDion ltyDion commented Oct 20, 2018

New infra structure for the front-end. Now we have two ports. One for development and one for final publish. Details about how to run please see the readme of front-end.

@Liby99
Copy link
Member

Liby99 commented Oct 20, 2018

Please go to package.json and change the FILL-IN-HERE placeholders.

Copy link
Member

@Liby99 Liby99 left a comment

Choose a reason for hiding this comment

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

I’m kind of being harsh on this code but yeah in general you did a good job. I just want to show you the code review process. After you addressed the changes push to this branch again so that this pull request could be updated

@@ -11,6 +11,7 @@ import { Input, Button, Card, Row, Col, Icon } from 'react-materialize';
import Header from './Header';
import LeasingCard from './LeasingCard';
import GoogleMapTest from './GoogleMapTest';
import '~/components/styles/search-page.scss';
Copy link
Member

Choose a reason for hiding this comment

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

If the style sheet is in the same directory, then use import ‘./styles/...’

@@ -0,0 +1,60 @@
/************** search page *************/
Copy link
Member

Choose a reason for hiding this comment

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

You can remove the comments in these files now

@@ -0,0 +1,9 @@
* {
font-family: "Microsoft JhengHei", sans-serif;
Copy link
Member

Choose a reason for hiding this comment

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

Make sure the indentation is 2 spaces

@Liby99 Liby99 merged commit df1f6b0 into front-end Oct 20, 2018
@Liby99
Copy link
Member

Liby99 commented Oct 20, 2018

Thanks. Approved.

@ltyDion
Copy link
Contributor Author

ltyDion commented Oct 20, 2018

I have fill in

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.

2 participants