Skip to content

Conversation

@Aneesh-0108
Copy link
Contributor

Fixes (#6515)

This Pull Request resolves a minor capitalization inconsistency in the "Send Thanks" status messages. The capitalization was corrected to align with the application's standard Sentence Case convention for UI text.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update (You can also check this if you consider string files as documentation)

Description and Changes Made

The issue was in the file strings.xml ,where it was visible in the lines 496,499 and 501 and action taken are:

String Name Original Value Corrected Value
send_thank_success_title "Sending **T**hanks: Success" "Sending **t**hanks: Success"
send_thank_failure_title "Sending **T**hanks: Failure" "Sending **t**hanks: Failure"
send_thank_toast "Sending **T**hanks for %1$s" "Sending **t**hanks for %1$s"

Checklist

(Ensure you check all these boxes before submitting)

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code where necessary
  • I have added tests that prove my fix is effective or that my feature works (Not needed for this type of textual fix)
  • New and existing unit tests pass locally with my changes (Not needed for this type of textual fix)
  • I have added documentation wherever appropriate

<!--accessibility UI description strings-->
<string name="commons_facebook">Commons Facebook Page</string>
<string name="commons_github">Commons Github Source Code</string>
<string name="commons_github">Commons GitHub Source Code</string>
Copy link
Member

Choose a reason for hiding this comment

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

Great catch! :-)

<string name="send_thank_failure_title">Sending thanks: Failure</string>

<string name="send_thank_toast">Sending Thanks for %1$s</string>
<string name="send_thank_toast">Sending thanks for %1$s</string>
Copy link
Member

Choose a reason for hiding this comment

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

Good job finding different patterns of this bug, rather than just what I wrote in the bug description!

@nicolas-raoul nicolas-raoul merged commit 714e5f8 into commons-app:main Oct 17, 2025
@Aneesh-0108
Copy link
Contributor Author

Thank you @nicolas-raoul

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants