Skip to content

made cowsay say what i type #4

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

noahgribbin
Copy link

No description provided.

<section class="container" ng-controller="CowsayController">
<h2>History</h2>
<pre>{{history}}</pre>
<button ng-click="history = {{cowsayCtrl.speak(cowsayCtrl.text)}}">Log</button>

Choose a reason for hiding this comment

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

On this ng-click it should look like the one on line 12. This was a syntax error not allowing you to log correctly. Correct and re-submit for full points.

# dotenv environment variables file
.env


Choose a reason for hiding this comment

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

should be able to enter build/ to ignore the build

Choose a reason for hiding this comment

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

▓⚗_⚗▓

"scripts": {
"build": "./node_modules/webpack/bin/webpack.js",
"watch": "./node_modules/webpack-dev-server/bin/webpack-dev-server.js --inline --hot"
},

Choose a reason for hiding this comment

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

the assignment required a lint script such as in addition "lint": "./node_modules/eslint/bin/eslint.js .", plug a lint script in

let cowsayCtrl = $scope.cowsayCtrl = {};
// whats goin on on this line?
// is cowsayCtrl bneing attached onto the $scope object?

Choose a reason for hiding this comment

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

It is attaching onto the $scope object and then adding functionality below.

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