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

JavaScript Animations #136

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

Conversation

karlasamantha
Copy link

No description provided.

@CLAassistant
Copy link

CLAassistant commented May 20, 2020

CLA assistant check
All committers have signed the CLA.

7-animation/3-js-animation/article.md Outdated Show resolved Hide resolved
7-animation/3-js-animation/article.md Outdated Show resolved Hide resolved
7-animation/3-js-animation/article.md Outdated Show resolved Hide resolved
7-animation/3-js-animation/article.md Outdated Show resolved Hide resolved
7-animation/3-js-animation/article.md Outdated Show resolved Hide resolved
@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@wnleao
Copy link
Contributor

wnleao commented May 22, 2020

Hi, Karla! Doing fine? Please don't translate or change the source code in the examples. Could you take a look into that? Like, you should keep the ';', spaces and variable names. Cheers!

@karlasamantha
Copy link
Author

Hi, Karla! Doing fine? Please don't translate or change the source code in the examples. Could you take a look into that? Like, you should keep the ';', spaces and variable names. Cheers!

Hey, yeah. I'm addressing feedback now. My linter changed the code, fixing that :)

@karlasamantha
Copy link
Author

/done

@javascript-translate-bot javascript-translate-bot requested a review from a team May 26, 2020 03:46
@karlasamantha karlasamantha requested review from wnleao and removed request for a team May 26, 2020 03:46
@odsantos
Copy link
Contributor

odsantos commented Dec 8, 2020

Hi @karlasamantha would you please also resolve the conflicts, or kindly inform if unavailable to? Thank you.

@odsantos
Copy link
Contributor

Hi @karlasamantha , please I wonder if you could resolve the conflicts, or kindly inform if unavailable to? Thank you.

@karlasamantha
Copy link
Author

Hi @karlasamantha , please I wonder if you could resolve the conflicts, or kindly inform if unavailable to? Thank you.

Hi @odsantos , sorry for the delay! I'll try to fix the conflicts today!

@karlasamantha
Copy link
Author

/done

@javascript-translate-bot javascript-translate-bot requested a review from a team May 25, 2021 21:17
@karlasamantha
Copy link
Author

/done

@odsantos
Copy link
Contributor

Hi @karlasamantha , thank you.

Copy link
Contributor

@odsantos odsantos left a comment

Choose a reason for hiding this comment

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

Hi @karlasamantha ,

The article review is still missing, hopefully to another time.
Please, feel free to leave a note on something you may not agree.

Thank you.

7-animation/3-js-animation/1-animate-ball/solution.md Outdated Show resolved Hide resolved
7-animation/3-js-animation/1-animate-ball/solution.md Outdated Show resolved Hide resolved
7-animation/3-js-animation/1-animate-ball/solution.md Outdated Show resolved Hide resolved
7-animation/3-js-animation/2-animate-ball-hops/solution.md Outdated Show resolved Hide resolved
7-animation/3-js-animation/2-animate-ball-hops/task.md Outdated Show resolved Hide resolved
7-animation/3-js-animation/2-animate-ball-hops/task.md Outdated Show resolved Hide resolved
7-animation/3-js-animation/text.view/index.html Outdated Show resolved Hide resolved
7-animation/3-js-animation/text.view/index.html Outdated Show resolved Hide resolved
@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

Copy link
Contributor

@odsantos odsantos left a comment

Choose a reason for hiding this comment

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

Hi @karlasamantha ,

Here is the whole review.
Please, feel free to leave a note where you might not agree.

Thank you,
Osvaldo.

7-animation/3-js-animation/article.md Outdated Show resolved Hide resolved
7-animation/3-js-animation/article.md Outdated Show resolved Hide resolved
7-animation/3-js-animation/article.md Outdated Show resolved Hide resolved
7-animation/3-js-animation/article.md Outdated Show resolved Hide resolved
7-animation/3-js-animation/article.md Outdated Show resolved Hide resolved
7-animation/3-js-animation/article.md Outdated Show resolved Hide resolved
7-animation/3-js-animation/article.md Outdated Show resolved Hide resolved
7-animation/3-js-animation/article.md Outdated Show resolved Hide resolved
7-animation/3-js-animation/article.md Outdated Show resolved Hide resolved
7-animation/3-js-animation/article.md Outdated Show resolved Hide resolved
@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@odsantos
Copy link
Contributor

Hello @karlasamantha the English article was updated and this PR now has some conflicts. The changes are small, though. Thank you.
Osvaldo.

@karlasamantha karlasamantha requested review from odsantos and wnleao and removed request for wnleao and odsantos January 26, 2023 21:16
@karlasamantha
Copy link
Author

/done

@nazarepiedady
Copy link
Member

@odsantos, @jonnathan-ls, @Peruibeloko, what do you recommend colleagues to be done in this pull request?

@nazarepiedady
Copy link
Member

@karlasamantha, are you available to update and apply the changes requested?

@karlasamantha
Copy link
Author

@karlasamantha, are you available to update and apply the changes requested?

@nazarepiedady sure, but I don't see any pending changes requested. Am I missing something here?

@nazarepiedady
Copy link
Member

@karlasamantha, I would like to invite you to compare the original English content to the Portuguese content translated by you to see if there are any updates needed or sentences that need to be applied.

I also invite you to check if the reviews did by @odsantos are observed or resolved.

@nazarepiedady
Copy link
Member

@jonnathan-ls, do you have time to review and handle this pull request?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants