Skip to content

ADFA-2986-Fix-error-msg-typo-Create-new-project-page#985

Merged
hal-eisen-adfa merged 2 commits intostagefrom
ADFA-2986-Fix-error-msg-typo-Create-new-project-page
Mar 4, 2026
Merged

ADFA-2986-Fix-error-msg-typo-Create-new-project-page#985
hal-eisen-adfa merged 2 commits intostagefrom
ADFA-2986-Fix-error-msg-typo-Create-new-project-page

Conversation

@Elissa-AppDevforAll
Copy link
Collaborator

"to long" and "is not a valid" are fixed

too long

is not valid

"to long" and "is not a valid" are fixed
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 16, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: a59720b6-0ec8-493b-aa1a-a265685adf92

📥 Commits

Reviewing files that changed from the base of the PR and between d42f1e4 and 61a929a.

📒 Files selected for processing (1)
  • resources/src/main/res/values/strings.xml

📝 Walkthrough
  • Fixed user-facing validation/error message typos on the Create New Project page:
    • "Package name is to long" → "Package name is too long"
    • "Package name is not a valid" → "Package name is not valid"
    • "A constructor with same parameter types is already available" → "A constructor with the same parameter types is already available" (fixed grammar/article)
  • Files changed:
    • resources/src/main/res/values/strings.xml

Risks / Best-practice notes:

  • Risk: Low — only string resource values were updated; no code logic or control flow changes.
  • Best practices: Changes are limited to localization resources. Ensure translations (if any) are updated for other locales to keep messages consistent.

Walkthrough

This pull request corrects three grammatical and spelling errors in Android string resources: fixes "not a valid" → "not valid", "to long" → "too long", and adds "the" in a constructor-related message.

Changes

Cohort / File(s) Summary
String Resource Corrections
resources/src/main/res/values/strings.xml
Updated three string literal values to correct grammar and spelling: msg_package_is_not_valid, msg_package_is_to_long, and msg_constructor_available. No functional or control-flow changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • hal-eisen-adfa
  • jatezzz

Poem

🐰 I hopped through strings with careful cheer,
Found "too" and "the" that didn't belong here,
A nibble, a tweak, a tidy small fix,
Now messages bounce in grammatical mix,
Hooray — clean text, and a happy rabbit's cheer! 🥕

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the pull request as fixing error message typos on the Create New Project page, which directly matches the changeset of correcting grammar errors in string resources.
Description check ✅ Passed The description is directly related to the changeset, identifying the specific typos being fixed ('to long' -> 'too long' and 'is not a valid' -> 'is not valid').
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch ADFA-2986-Fix-error-msg-typo-Create-new-project-page

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@hal-eisen-adfa hal-eisen-adfa merged commit 6b8af67 into stage Mar 4, 2026
2 checks passed
@hal-eisen-adfa hal-eisen-adfa deleted the ADFA-2986-Fix-error-msg-typo-Create-new-project-page branch March 4, 2026 02:51
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.

3 participants