Conversation
chapters/network/chapter.md
Outdated
|
@bakercp I fixed all the issues you pointed out. Thank you. |
|
ping @arturoc |
1 similar comment
|
ping @arturoc |
|
@arturoc could you please take a look at my changes? |
|
@tpltnt i've taken a look at your changes several times but it's complicated to merge, there's things that are very valuable like grammar corrections or the introduction to ip which was missing and it's important to understand but it's somehow too technical for the level i was aiming for, originally. Then there's annotations that need to be removed. There's also parts that are just wrong or too technical. I haven't had time to fix everything before merging. In general there's too many things that i don't consider correct from a technical or pedagogical point of view and don't have the time to give you feedback on all of them. From my part i consider the chapter is finished so don't merge anything without the permission of the original authors unless is small fixes that are obviously wrong on the originals. If you want to give a try at removing the annotations and explaining things in a simpler way i can try to give you some feedback but right now there's too many things that i don't consider correct to be able to give you feedback. Otherwise i can close this PR. Let me know what you prefer to do. |
|
@arturoc thank you for the feedback. I will try to address the annotations and will come back to you. |
|
@arturoc I removed the annotation and cleaned up a few things. Are you ok with merging? |
|
ping @arturoc |
Hi @arturoc
please check my changes and merge if applicable.
Cheers,
tpltnt