Skip to content

Brian's exercise#3

Open
BrianPiere wants to merge 14 commits into
GasStationTV:masterfrom
BrianPiere:brians-excercise
Open

Brian's exercise#3
BrianPiere wants to merge 14 commits into
GasStationTV:masterfrom
BrianPiere:brians-excercise

Conversation

@BrianPiere
Copy link
Copy Markdown

The Project Readme asked for the following versions in the pull request.

Node version: 4.4.3
MongoDB Version: Not sure how to find this with Mlab.com. Their Mongo Shell is limited.
You'll see the DB "connection string" within a text file in Git
/server/dbconnection

Remaining Issues
This task was a lot of fun and it pushed me to the limit as far as what I could accomplish within this time frame. It was hard to get the ball rolling without an app creator. There are three things that I came up short on.

  • Date Pickers: This was the big one I missed because it was a project requirement. However the application is still working correctly with manual dates (using text-fields). The validation routines wouldn't change after integrating a Date Picker component because they would simply populate the text fields (as an add-on helper).
  • Unit Testing: It wasn't technically a requirement on the Readme.md but still it's expected on any serious project. The remaining work is left as a set of un-checked entries on my Task List.
  • CSS Styling: I was going to use Styled Components as an exercise (first time). I just put some really basic stuff inline for now. I'm sure that I won't get any points on looks :)

My Task List
I committed updates to my task list as I went along. You'll find it within Git at...
/Brian Task List.txt
Just in case you're interested, I put the newest entries at top at the root level. Indented entries are sorted with the oldest entries on top.

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.

1 participant