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

Chase's WDI radio #13

Closed
wants to merge 24 commits into from
Closed

Chase's WDI radio #13

wants to merge 24 commits into from

Conversation

cpgruber
Copy link

I worked with Davis, it was good.

Completeness: 4 - i think we met all minimum requirements
Comfort level: 4 - starting to make better sense of Angular

@ghost
Copy link

ghost commented Dec 18, 2015

Completeness: 5
Comfort: 3

Chase and I paired on this. I navigated most of the week and offered advice where I could. It went well and I learned a lot from Chase.

link:function(scope){
scope.create = function(){
scope.song.$save(scope.song,function(song){
$state.go("songIndex",{},{reload:true})
Copy link
Contributor

Choose a reason for hiding this comment

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

do you need the second two arguments here?

@jshawl
Copy link
Contributor

jshawl commented Dec 27, 2015

👍 excellent job!

@jshawl jshawl closed this Dec 27, 2015
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.

None yet

2 participants