Skip to content

ADFA-2788 Change warning message from AndroidIDE to Code On the Go#977

Merged
hal-eisen-adfa merged 2 commits intostagefrom
ADFA-2788-Change-warning-message-from-AndroidIDE
Feb 13, 2026
Merged

ADFA-2788 Change warning message from AndroidIDE to Code On the Go#977
hal-eisen-adfa merged 2 commits intostagefrom
ADFA-2788-Change-warning-message-from-AndroidIDE

Conversation

@hal-eisen-adfa
Copy link
Collaborator

Changed AndroidIDE to CodeOnTheGo in some debug messages
Moved to logsender's strings.xml where possible

@hal-eisen-adfa hal-eisen-adfa requested a review from a team February 13, 2026 03:58
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 13, 2026

📝 Walkthrough

Release Notes

Changes

  • Rebranding messaging: Updated user-facing debug and warning messages to reference "Code On The Go" instead of "AndroidIDE" across multiple components

    • LogSenderService: Updated log messages and notification text
    • AbstractModelBuilder: Updated error messages for AGP version compatibility warnings
    • msg_bind_service_failed: Updated service binding failure message
  • Resource management improvements: Migrated hardcoded notification strings to resource file for better maintainability and localization support

    • Added notification_title, notification_text, and notification_channel_name to strings.xml
    • Updated LogSenderService to reference notification strings from resources instead of hardcoded values

Files Modified

  • logsender/src/main/java/com/itsaky/androidide/logsender/LogSenderService.java
  • logsender/src/main/res/values/strings.xml
  • subprojects/tooling-api-impl/src/main/java/com/itsaky/androidide/tooling/impl/sync/AbstractModelBuilder.kt

Considerations

  • Potential consistency issue: Verify that all occurrences of "AndroidIDE" in user-facing messages have been updated across the entire codebase to avoid inconsistent branding
  • String resource migration: Ensure that other hardcoded strings in the codebase follow the same pattern of being moved to resource files for consistency

Walkthrough

This pull request updates application branding and notification strings across the logsender service and tooling API modules. Hardcoded notification values are replaced with string resources, and all references to "AndroidIDE" are updated to "Code On The Go" throughout the codebase.

Changes

Cohort / File(s) Summary
Notification Strings and Resources
logsender/src/main/java/com/itsaky/androidide/logsender/LogSenderService.java, logsender/src/main/res/values/strings.xml
Notification title, text, and channel name moved from hardcoded strings to dynamic resource values. Notification construction updated to reference new string resources (notification_title, notification_text, notification_channel_name).
Branding Updates
logsender/src/main/res/values/strings.xml, subprojects/tooling-api-impl/src/main/java/com/itsaky/androidide/tooling/impl/sync/AbstractModelBuilder.kt
All user-facing strings changed from "AndroidIDE" to "Code On The Go" in bind failure messages, AGP version warnings, and sync issue reports.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Suggested reviewers

  • jatezzz
  • dara-abijo-adfa

Poem

🐰 Strings now dance in resource files so bright,
"Code On The Go" shines with rebranding light,
Notifications leap from hardcoded chains,
Through config and localization plains,
The app hops forward with style to gain!

🚥 Pre-merge checks | ✅ 3 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 16.67% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main objective of the PR: changing warning messages from AndroidIDE to Code On The Go across multiple files.
Description check ✅ Passed The description is related to the changeset, accurately capturing that AndroidIDE was replaced with CodeOnTheGo and strings were moved to logsender's strings.xml.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into stage

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch ADFA-2788-Change-warning-message-from-AndroidIDE

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.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
logsender/src/main/java/com/itsaky/androidide/logsender/LogSenderService.java (1)

37-37: ⚠️ Potential issue | 🟡 Minor

Javadoc still references "AndroidIDE".

The class-level Javadoc on line 37 says "sends logs to AndroidIDE" — should be updated to "Code On The Go" for consistency with the rest of this PR.

-/**
- * A {`@link` Service} which runs in the background and sends logs to AndroidIDE.
+/**
+ * A {`@link` Service} which runs in the background and sends logs to Code On The Go.

@hal-eisen-adfa hal-eisen-adfa merged commit 73a2da0 into stage Feb 13, 2026
2 checks passed
@hal-eisen-adfa hal-eisen-adfa deleted the ADFA-2788-Change-warning-message-from-AndroidIDE branch February 13, 2026 07:52
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