the-farm: introduce interfaces concept #1638
Comments
ErikSchierboom
changed the title
|
@jmrunkle Do you want to continue to work on this and update the exercise to include the new concept or should we re-assign this task? |
|
Yes, sorry for the delay. Give me a couple more days. If I cannot finish by then we can reassign. |
|
@jmrunkle Could you give me an update whether I should unassign you or not? |
|
Sorry, still recovering from new baby. I'm nearly done with this so if you do not mind being a little patient, I can finish it. |
|
@jmrunkle I know how that feels 💙 and I would really like that you don't have this issue in your neck anymore. So if you don't mind, @andrerfcsantos offered to finish this up. If you could push whatever you have to some branch/ create a draft PR, he can pick it up from there. |
|
@jmrunkle I don't mind waiting a bit more, but like June said, I'll happily finish this so you can have more time with your family! |
|
@jmrunkle Can you give us an update on this? |
|
I just turned my in-progress code into a draft PR: #1976 I have lost a bit of context on where exactly I had left this, but I will try to make it complete today. |
|
Looks like the main thing that was missing was to copy the intro material about interfaces into the intro for the exercise. Let me patch that in and then send the PR out for review. |
|
@jmrunkle To avoid further delays (students keep asking for this), I would suggest @andrerfcsantos can directly push small fixes to your branch if anything comes up instead of writing review comments and waiting for them to be resolved. This way it can be merged quickly. If you object, let us know. |
|
Sounds fine to me. I don't mind either way |
The Farmshould unlockinterfaces. This will involve:interfacesto theconceptsfor thethe-farmexercise in theconfig.jsonThe text was updated successfully, but these errors were encountered: