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

PK's Challenge Submission #8

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

PK's Challenge Submission #8

wants to merge 2 commits into from

Conversation

pablok47
Copy link

@pablok47 pablok47 commented Apr 4, 2016

Pablo Kaparis is using reactjs to build "Stock Manager".

Installed as dependencies:
babel-preset-react
babelify
watchify
react

This react app is a Stock Manager. It will be an application where one could keep track of the stocks they are interested in. They will be able to create several lists based on each sector (ex: Tech, Energy, Financials, etc.).

The app is like a shopping list app only the list contains stocks of companies one likes instead of shopping items.

Features besides list feature includes:
Digital clock,
Note Pad,
Nav Bar with link to Yahoo's Financials Website set to the Logo on the top left corner of the nav bar.

@superchris
Copy link
Member

Pablo,

Here's some impressions after reviewing your submission:

After manually installing babel-preset-survivejs-kanban and babel-preset-es2015 npms I was able to get your application to load. I noticed that you had implemented an stock tracking application instead of a shopping list. One of the things we wanted to see is if you were able to build an application that was just a bit more interesting that the example app we pointed you at. Specifically, we'd like to see a list item have fields of it's own such as name and price, rather than just being a simple string.

If you'd like to continue on with this application, I'd be happy to take another look. We've selected a few candidates for initial interviews, and based on these may or may not be doing more interviews for the first pair of apprentices. Regardless of the outcome, we expect to hire an additional pair in the next month or two and would be happy to consider your submission then.

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