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
Fixed #5181 Misleading warning title for duplicate file name #5185
Fixed #5181 Misleading warning title for duplicate file name #5185
Conversation
app/src/main/res/values/strings.xml
Outdated
@@ -192,6 +192,7 @@ | |||
<string name="location_permission_title">Requesting Location Permission</string> | |||
<string name="ok">OK</string> | |||
<string name="warning">Warning</string> | |||
<string name="duplicate_file_name">Duplicate File Name found</string> | |||
<string name="duplicate_image_found">Duplicate Image Found</string> |
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.
Is this duplicate_image_found still used somewhere?
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.
Nope.
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.
Seems like we can delete it
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.
Please remove it then 🙂
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.
Done!
@nicolas-raoul I guess this pr is good to go? |
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.
Thanks and sorry for the delay!
Description (required)
Fixes #5181
What changes did you make and why?
Added a new string resource to display in the alertdialog
Tests performed (required)
Tested locally on my device POCO X2, API Level 30
Screenshots (for UI changes only)