Do not submit a pull request to this repository. Your PR will be rejected and your submission ignored. To be safe do not fork this repository, if you do you will be tempted to create a PR.
To properly submit a coding challenge you must:
-
Create a blank (empty) repo in the public git service of your choice (GitHub, GitLab, Bitbucket)
-
Clone this repo to your local workstation
-
Reset the remote origin to point to your newly created empty repo
-
Push the master branch up to your repo
-
Make necessary changes
-
Push changes to your origin
-
Send address of your copy to T-Mobile
We will review your copy online before and during your interview.
There are two apps: stocks
and stocks-api
.
stocks
is the front-end. It uses Angular 7 and Material. You can run this usingyarn serve:stocks
stocks-api
uses Hapi and has a very minimal implementation. You can start the API server withyarn serve:stocks-api
A proxy has been set up in stocks
to proxy calls to locahost:3333
which is the port that the Hapi server listens on.
You need to register for a token here: https://iexcloud.io/cloud-login#/register/ Use this token in the
environment.ts
file for thestocks
app.
The charting library is the Google charts API: https://developers.google.com/chart/
Please provide a short code review of the base master
branch:
- What is done well?
- What would you change?
- Are there any code smells or problematic implementations?
Make a PR to fix at least one of the issues that you identify
Accessibility is an important feature of all public facing websites.
Make a PR to add accessibility features to the web application
Business requirement: As a user I should be able to type into
the symbol field and make a valid time-frame selection so that
the graph is refreshed automatically without needing to click a button.
Make a PR from the branch feat_stock_typeahead
to master
and provide a code review on this PR
Add comments to the PR. Focus on all items that you can see - this is a hypothetical example but let's treat it as a critical application. Then present these changes as another commit on the PR.
Business requirement: As a user I want to choose custom dates
so that I can view the trends within a specific period of time.
Implement this feature and make a PR from the branch feat_custom_dates
to master
.
Use the material date-picker component
We need two date-pickers: "from" and "to". The date-pickers should not allow selection of dates after the current day. "to" cannot be before "from" (selecting an invalid range should make both dates the same value)
Technical requirement: the server `stocks-api` should be used as a proxy
to make calls. Calls should be cached in memory to avoid querying for the
same data. If a query is not in cache we should call-through to the API.
Implement the solution and make a PR from the branch feat_proxy_server
to master
It is important to get the implementation working before trying to organize and clean it up.