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

could not finish #24

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

Conversation

maldonadoguitar
Copy link

first incomplete homework assignment. not happy bout this one

@Mataniko
Copy link

Christian,
Did you keep working on this past the deadline? I'd still like to do a code review even if you didn't finish on time.

What issues did you run into that you couldn't finish?

@maldonadoguitar
Copy link
Author

Hi,
Where do i start! i have a basic understanding for everything that's
going on but i get to a point where i don't know what to do for the next
step and so i refer back to older assignments to help me find answers. i
started to redo it yesterday as a way to practice and solve some of the
issues. i cloned a classmates project and she helped me to understand it
so i am currently doing it again. I am supposed to meet with a TA tomorrow
but he has not gotten in touch with me so i am not sure that is happening
still. I am at work so unfortunately i can;t go through all my issues.
perhaps you have some time we can do a one on one and you can help me out
(tho this week might not be good as i am getting married on Saturday, tho
tomorrow could work if Aaron does not get back in touch with me) any
thoughts on that?

On Mon, Aug 17, 2015 at 11:00 AM, Matan Bareket [email protected]
wrote:

Christian,
Did you keep working on this past the deadline? I'd still like to do a
code review even if you didn't finish on time.

What issues did you run into that you couldn't finish?


Reply to this email directly or view it on GitHub
#24 (comment)
.

@Mataniko
Copy link

So first things first, congratulations for your upcoming nuptials!

My reviews are due tomorrow, so at that point it's probably too late. I highly recommend attending office hours tonight and working through this with the TA present.

You were definitely on a good path based on what I've seen. Try moving all your data into a separate class and use that in your UI.

Read up on what prepareForSegue and cellForRowAtIndexPath do and implement them. Delete code that is not required.

Make sure you can pass information back from the second TVC to the first one.

I won't be around for office hours tonight, but you can just ping me with specific questions or we can try and find some time tonight remotely.

@maldonadoguitar
Copy link
Author

thanks for the advice. Mondays and Wednesdays are really hard for me to
get to office hours. i teach music after work so i can;t make it. I will
definitely try to make it to office hours should any lesson cancellations
occur but i will ping you one way or the other with my questions and i
really appreciate it thanks agian

On Mon, Aug 17, 2015 at 11:26 AM, Matan Bareket [email protected]
wrote:

So first things first, congratulations for your upcoming nuptials!

My reviews are due tomorrow, so at that point it's probably too late. I
highly recommend attending office hours tonight and working through this
with the TA present.

You were definitely on a good path based on what I've seen. Try moving all
your data into a separate class and use that in your UI.

Read up on what prepareForSegue and cellForRowAtIndexPath do and implement
them. Delete code that is not required.

Make sure you can pass information back from the second TVC to the first
one.

I won't be around for office hours tonight, but you can just ping me with
specific questions or we can try and find some time tonight remotely.


Reply to this email directly or view it on GitHub
#24 (comment)
.

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