Skip to content
This repository was archived by the owner on Oct 26, 2020. It is now read-only.

Week3/patrick #83

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

Week3/patrick #83

wants to merge 18 commits into from

Conversation

pchan2
Copy link

@pchan2 pchan2 commented Jul 6, 2020

Your Details

Your Name: Patrick Chan
Your City: Birmingham
Your Slack Name: Patrick Chan (from #westmidlands-class1)

Homework Details

Module: JavaScript Core 1
Week: 3

wonder woman added 18 commits June 17, 2020 21:46
1-exercises
2-mandatory
3-extra
function validate(num)
function tidyUpString(strArr) - bug
function remove(arr, index) - passed
function formatPercentage(arr) - bug
function works but the TEST output is FAILED - not sure why
… 3 functions

Next step: I need to get the three functions to work the whole array instead of just one array item.
I am not sure if I have answered correctly Q6 and Q7 because it adds up to 8, not 9 as per Debug Console result.
I used the built-in Javascript filter and sort methods instead of using Hard Mode.
revised tidyUpString
and formarPercentage

Created new js files to test solutions with different collaborators.
completed part 1: function getAvailableStations()
some completed, some have errors
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant