Skip to content

Solution (#3384): Arrange string resources based on screens #6335

Draft
TFGSUMIT wants to merge 1 commit into
oppia:developfrom
TFGSUMIT:fix/issue-3384
Draft

Solution (#3384): Arrange string resources based on screens #6335
TFGSUMIT wants to merge 1 commit into
oppia:developfrom
TFGSUMIT:fix/issue-3384

Conversation

@TFGSUMIT

Copy link
Copy Markdown

"This PR addresses the issue of arranging string resources based on screens in the Oppia Android app.

Changes Made:

  • Added a hierarchical structure to the string resources in strings.xml
  • Organized resources into categories such as "Screen/Toolbar Titles," "AppContentActivity," and "Shared Strings"

Testing Instructions:

  1. Run the app on a physical device or emulator to verify that all string resources are correctly categorized and accessible.
  2. Use a code review tool to check for any errors or inconsistencies in the app's UI management.
  3. Test the app's UI components to ensure that they are functioning correctly and consistently.

Note: This change is backward compatible and does not affect the app's functionality. However, it is recommended to review and update any existing code that relies on the previous string resource structure."

@github-actions

Copy link
Copy Markdown
Contributor

@TFGSUMIT this PR is being marked as draft because the PR description must contain 'Fixes #' or 'Fixes part of #' for each issue the PR is changing, and each one on its own line with no other text.

@github-actions
github-actions Bot marked this pull request as draft July 18, 2026 13:13
@coderabbitai

coderabbitai Bot commented Jul 18, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 017b7f7c-de4e-4408-9db7-e9399a9f1590

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@Neer-rn

Neer-rn commented Jul 18, 2026

Copy link
Copy Markdown
Collaborator

Same here, Please see my comment on #6336

@Neer-rn

Neer-rn commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

Hi @TFGSUMIT, it seems like #3384 is not assigned to you. Please don’t directly open a PR for an issue that hasn’t been assigned to you. First, go to the issue and explain your approach for solving it. After reviewing your approach, we can assign the issue to you, and then you can start working on it.

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