-
Notifications
You must be signed in to change notification settings - Fork 111
F.prototype #231
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
F.prototype #231
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello @flaviohblima ,
Here is a review.
Two files in the "1-js/08-prototypes/02-function-prototype/4-new-object-same-constructor" folder seem not yet translated.
Please, feel free to comment where you might disagree.
Thank you,
Osvaldo.
1-js/08-prototypes/02-function-prototype/1-changing-prototype/solution.md
Outdated
Show resolved
Hide resolved
1-js/08-prototypes/02-function-prototype/1-changing-prototype/solution.md
Outdated
Show resolved
Hide resolved
1-js/08-prototypes/02-function-prototype/1-changing-prototype/task.md
Outdated
Show resolved
Hide resolved
Please make the requested changes. After it, add a comment "/done". |
/done |
Hello @flaviohblima , I hope to check it as soon as possible. Probably, within a few days. However, I still see my review notes. I believe I shouldn't if they were resolved and committed. Osvaldo. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1-js/08-prototypes/02-function-prototype/4-new-object-same-constructor/solution.md
Outdated
Show resolved
Hide resolved
1-js/08-prototypes/02-function-prototype/4-new-object-same-constructor/solution.md
Outdated
Show resolved
Hide resolved
1-js/08-prototypes/02-function-prototype/4-new-object-same-constructor/solution.md
Outdated
Show resolved
Hide resolved
1-js/08-prototypes/02-function-prototype/4-new-object-same-constructor/solution.md
Outdated
Show resolved
Hide resolved
1-js/08-prototypes/02-function-prototype/4-new-object-same-constructor/solution.md
Outdated
Show resolved
Hide resolved
Please make the requested changes. After it, add a comment "/done". |
…structor/solution.md Co-authored-by: Osvaldo Dias dos Santos <vadodsantos@yahoo.com>
…structor/solution.md Co-authored-by: Osvaldo Dias dos Santos <vadodsantos@yahoo.com>
…structor/solution.md Co-authored-by: Osvaldo Dias dos Santos <vadodsantos@yahoo.com>
…structor/solution.md Co-authored-by: Osvaldo Dias dos Santos <vadodsantos@yahoo.com>
Co-authored-by: Osvaldo Dias dos Santos <vadodsantos@yahoo.com>
/done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello @flaviohblima ,
I reopened a conversation and I still am insisting on that one with 'de'.
Would you mind if I ask you to review it again?
Please, I continue open to your appreciation of my review.
Thank you,
Osvaldo.
Please make the requested changes. After it, add a comment "/done". |
Co-authored-by: Osvaldo Dias dos Santos <vadodsantos@yahoo.com>
Hello @odsantos , If you insist, I see no problem fixing it. I already commited the change. Thank you, |
/done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello @flaviohblima ,
I appreciate the patience you had.
By the rules, we need a second reviewer for your work to be merged.
Let's hope that either @luizclr, @Mr-Saxobeat, @adriavieira314, or @kiniggit have time for that.
Best regards,
Osvaldo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I recorded some notes
Note: the images (SVG) have not been translated.
I didn't add the above note in the review, but I was wondering if you could perform the script translation procedure? As per image translation guidance at https://github.com/javascript-tutorial/server#translating-images
If not, no problem, I perform the procedure separately in another Pull Request.
1-js/08-prototypes/02-function-prototype/4-new-object-same-constructor/task.md
Outdated
Show resolved
Hide resolved
1-js/08-prototypes/02-function-prototype/4-new-object-same-constructor/solution.md
Outdated
Show resolved
Hide resolved
1-js/08-prototypes/02-function-prototype/1-changing-prototype/solution.md
Outdated
Show resolved
Hide resolved
Please make the requested changes. After it, add a comment "/done". |
/done |
I applied the corrections I requested by contributing to the PR to be able to accept the request |
Thank you 💖 I updated the Progress Issue #1 🎉 🎉 🎉 |
No description provided.