[CLOSED] Update strings.js

[core-ai-bot]

Issue by Denisov21
Sunday Feb 23, 2014 at 10:31 GMT
Originally opened as adobe/brackets#6980

---

Update Italian translator.

---

Denisov21 included the following code: https://github.com/adobe/brackets/pull/6980/commits

[core-ai-bot]

Comment by Denisov21
Sunday Feb 23, 2014 at 11:21 GMT

---

@chrispecoraro Hello, Sorry but I am a new user in Brackets. Now what do I do? I need to close this "bug" by clicking on "close and comment". I would like the changes I made are accepted!

[core-ai-bot]

Comment by chrispecoraro
Sunday Feb 23, 2014 at 11:22 GMT

---

I'm new as well...

On Sun, Feb 23, 2014 at 12:21 PM, Denisov21 notifications@github.comwrote:

@chrispecoraro https://github.com/chrispecoraro Hello, Sorry but I am a
new user in Brackets. Now what do I do? I need to close this "bug" by
clicking on "close and comment". I would like the changes I made are
accepted!

Reply to this email directly or view it on GitHubhttps://github.com/adobe/brackets/pull/6980#issuecomment-35829357
.

Not sent from an iPhone

[core-ai-bot]

Comment by Denisov21
Sunday Feb 23, 2014 at 12:03 GMT

---

Ok, all fix!!

[core-ai-bot]

Comment by ingorichter
Tuesday Feb 25, 2014 at 00:12 GMT

---

Hi@Denisov21, perhaps you should add the changes from PR #6973 and #6974 into this PR, since the review started here already. Thank you.

[core-ai-bot]

Comment by Denisov21
Tuesday Feb 25, 2014 at 20:03 GMT

---

Hi@ingorichter Yes, of course. It's a big idea. Thank you for your help me!

[core-ai-bot]

Comment by Denisov21
Friday Mar 14, 2014 at 11:21 GMT

---

@ingorichter Can you include in Spint 37 my changes?

[core-ai-bot]

Comment by ingorichter
Thursday Mar 20, 2014 at 00:35 GMT

---

@Denisov21 I'm sorry that we are not able to include this in Sprint 37 anymore. We will target this for Sprint 38.@Denisov21@artoale are you done with the review? I'd like to merge it. Thank you.

[core-ai-bot]

Comment by artoale
Thursday Mar 20, 2014 at 01:13 GMT

---

LGTM :)
On Mar 20, 2014 1:36 AM, "Ingo Richter" notifications@github.com wrote:

@Denisov21 https://github.com/Denisov21 I'm sorry that we are not able
to include this in Sprint 37 anymore. We will target this for Sprint 38.
@Denisov21 https://github.com/Denisov21`@`artoalehttps://github.com/artoaleare you done with the review? I'd like to merge it. Thank you.

Reply to this email directly or view it on GitHubhttps://github.com/adobe/brackets/pull/6980#issuecomment-38124270
.

[core-ai-bot]

Comment by Denisov21
Thursday Mar 20, 2014 at 17:56 GMT

---

@ingorichter Sorry I know I made some mistakes but how can I merge? And how can I fix them?

[core-ai-bot]

Comment by Denisov21
Thursday Mar 20, 2014 at 17:58 GMT

---

@artoale Hello, What do you mean "LGTM"? Can you help?

[core-ai-bot]

Comment by ingorichter
Monday Mar 24, 2014 at 17:18 GMT

---

@Denisov21 It's not bad to make mistakes. This is part of a learning process. I can go ahead and merge this PR if everybody is okay with the current translation. Regarding the two other PR's, I'll commenting on them and give you some instructions how to get them into one PR. BTW, I don't know the meaning of LGTM. ;-)

[core-ai-bot]

Comment by njx
Monday Mar 24, 2014 at 17:24 GMT

---

LGTM = "looks good to me" :) Means it's good.

[core-ai-bot]

Comment by Denisov21
Monday Mar 24, 2014 at 20:52 GMT

---

@ingorichter Hello, Thank you very much for your help. As I have written are a beginner, you will learn better with time using this tool you!

[core-ai-bot]

Comment by Denisov21
Monday Mar 24, 2014 at 20:54 GMT

---

@njx What does it mean PR?

[core-ai-bot]

Comment by njx
Monday Mar 24, 2014 at 21:20 GMT

---

@Denisov21 Pull Request.