Skip to content

Change resource string (feedback from #25903)#26131

Merged
OmarTawfik merged 1 commit intodotnet:masterfrom
OmarTawfik:fix-25944-resources-nit
Apr 13, 2018
Merged

Change resource string (feedback from #25903)#26131
OmarTawfik merged 1 commit intodotnet:masterfrom
OmarTawfik:fix-25944-resources-nit

Conversation

@OmarTawfik
Copy link
Copy Markdown
Contributor

Closes #25944

cc @dotnet/roslyn-compiler

<target state="new">Invalid object creation</target>
<note />
</trans-unit>
<trans-unit id="FTL_InvalidInputFileName">
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is it possible to keep the existing translations for these, since all that was changed was the id of the string? Or does it have to be "retranslated"? cc @tmeschter

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't think there's an automatic way to keep the existing translation under a new ID, but it isn't a big deal. On their next pass, the Loc team will find the existing translation in their master database and reuse that for the "new" resource--no real work by a human will be required.

Copy link
Copy Markdown
Contributor

@AlekseyTs AlekseyTs left a comment

Choose a reason for hiding this comment

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

LGTM (iteration 1)

@OmarTawfik OmarTawfik merged commit d187f90 into dotnet:master Apr 13, 2018
@OmarTawfik OmarTawfik deleted the fix-25944-resources-nit branch April 13, 2018 21:49
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.

4 participants