Skip to content

Fix typos and grammar in strings.xml#16005

Closed
duvvuvenkataramana wants to merge 3 commits into
nextcloud:masterfrom
duvvuvenkataramana:patch-2
Closed

Fix typos and grammar in strings.xml#16005
duvvuvenkataramana wants to merge 3 commits into
nextcloud:masterfrom
duvvuvenkataramana:patch-2

Conversation

@duvvuvenkataramana
Copy link
Copy Markdown
Contributor

  • Tests written, or not not needed

Signed-off-by: duvvuvenkataramana <[email protected]>
Comment thread app/src/main/res/values/strings.xml Outdated
Comment thread app/src/main/res/values/strings.xml Outdated
@duvvuvenkataramana
Copy link
Copy Markdown
Contributor Author

@AndyScherzinger @rakekniven I changed and corrected some spellings. Review and Merge

Comment thread app/src/main/res/values/strings.xml Outdated
Comment thread app/src/main/res/values/strings.xml Outdated
Comment thread app/src/main/res/values/strings.xml Outdated
@AndyScherzinger
Copy link
Copy Markdown
Member

@duvvuvenkataramana thanks for the changes, please see my comments. This won't compile since changing the ID for a string basically breaks the ID-reference in the java/kotlin code, so the app won't even compile anymore.

<item quantity="other">You can upload up to %d files at once.</item>
</plurals>
<string name="sync_foreign_files_forgotten_explanation">As of version 1.3.16, files uploaded from this device are copied into the local %1$s folder to prevent data loss when a single file is synced with multiple accounts.\n\nDue to this change, all files uploaded with earlier versions of this app were copied into the %2$s folder. However, an error prevented the completion of this operation during account synchronization. You may either leave the file(s) as is and delete the link to %3$s, or move the file(s) into the %1$s folder and retain the link to %4$s.\n\nListed below are the local file(s), and the remote file(s) in %5$s they were linked to.</string>
<string name="sync_foreign_files_forgotten_explanation">As of version 1.3.16, files uploaded from this device are copied into the local %1$s folder to prevent data loss when a single file is sycned with multiple accounts.\n\nDue to this change, all files uploaded with earlier versions of this app were copied into the %2$s folder. However, an error prevented the completion of this operation during account synchronization. You may either leave the file(s) as is and delete the link to %3$s, or move the file(s) into the %1$s folder and retain the link to %4$s.\n\nListed below are the local file(s), and the remote file(s) in %5$s they were linked to.</string>
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.

This adds a typo and is not removing one!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

this is correct one

Comment thread app/src/main/res/values/strings.xml
@AndyScherzinger
Copy link
Copy Markdown
Member

Hi @duvvuvenkataramana quick question, are you vibe coding this? I am asking because

  • strings-xml is US-English and the GB/en equivalent in the repo runnign through a translation service would be British English
  • synced got changed to sycned now which is introducing typos
  • new commits add changes to String identifiers which I stated breaks the code from compiling!

Copy link
Copy Markdown
Member

@AndyScherzinger AndyScherzinger left a comment

Choose a reason for hiding this comment

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

needs fixing

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.

3 participants