Skip to content

Maximilian Wagner#151

Open
MWagner3 wants to merge 11 commits intoturingschool:mainfrom
MWagner3:main
Open

Maximilian Wagner#151
MWagner3 wants to merge 11 commits intoturingschool:mainfrom
MWagner3:main

Conversation

@MWagner3
Copy link

Here is the Mod 1 prework

@toppings = toppings
end

def add_topping
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'll give you a nudge for this one that may also help with the change_protein method -

Currently, the add_topping method adds more of the original topping. Could you rework it so the user can choose a NEW topping to add to their burrito? How can information be added dynamically?

Copy link
Author

Choose a reason for hiding this comment

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

Or would we be looking to change the attribute reader to accessor? Then do something like

def change_protein(protein)
@protein = protein
end

Copy link
Author

Choose a reason for hiding this comment

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

I think that would work, and the same logic could be applied to the add_topping.

@ameseee
Copy link
Collaborator

ameseee commented Nov 2, 2021

Hey there @MWagner3 - your work so far looks good, I left a couple comments I'd like you to read through and update as appropriate!

From what I can see, I don't believe you've completed (or at least pushed up) the finalprep section of the prework - specifically the technical work. The boxes were checked but the files were untouched. Is that correct?

@MWagner3
Copy link
Author

MWagner3 commented Nov 3, 2021

Hey Amy! In the morning I will take a look at your comments and see what the issue was in the final prep section. I thought I had submitted everything so I will go back and take a look. Thanks

@MWagner3
Copy link
Author

MWagner3 commented Nov 3, 2021

Decided just to take a quick look now and just pushed the last portion through. I am not sure how I missed that, yikes.

@ameseee
Copy link
Collaborator

ameseee commented Nov 4, 2021

Looks good Max, I've marked you as Turing Ready! Good luck in Mod 1 - you've got this.

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