Skip to content

Conversation

@paviad
Copy link
Contributor

@paviad paviad commented Aug 2, 2025

Proposed changes

Fix the README.md with the correct mobile app white labeling documentation link.

Issues

Closes #5795
https://rocketchat.atlassian.net/browse/COMM-109

Types of changes

  • Bugfix (non-breaking change which fixes an issue)
  • Improvement (non-breaking change which improves a current function)
  • New feature (non-breaking change which adds functionality)
  • Documentation update (if none of the other choices apply)

Checklist

  • I have read the CONTRIBUTING doc
  • I have signed the CLA
  • Lint and unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works (if applicable)
  • I have added necessary documentation (if applicable)
  • Any dependent changes have been merged and published in downstream modules

Summary by CodeRabbit

  • Chores
    • Updated documentation link for mobile app white-labeling resources.

✏️ Tip: You can customize this high-level summary in your review settings.

paviad and others added 2 commits August 2, 2025 16:50
Fix the mobile app white labeling link
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 30, 2025

Walkthrough

Updated a broken documentation link in README.md that was pointing to a 404 page. The link path for Mobile App White Labeling documentation was corrected from an outdated reference to the proper documentation path.

Changes

Cohort / File(s) Summary
Documentation Link Fix
README.md
Updated whitelabel documentation link from mobile-app-white-labelling to docs/mobile-app-white-labeling to resolve broken link (404) in the README's whitelabel section

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A link was broken, leading nowhere fast,
Now the path corrected, the docs at last!
No 404 shadows on this sunny day,
White labeling wisdom points the right way! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Linked Issues check ✅ Passed The pull request successfully addresses issue #5795 by updating the broken white labeling documentation link from an incorrect path to the correct one (docs/mobile-app-white-labeling).
Out of Scope Changes check ✅ Passed All changes are directly related to fixing the broken link issue; no out-of-scope modifications are present in this documentation-only update.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title clearly and accurately describes the main change: fixing a white labeling documentation link in the README, which aligns with the PR's core objective of correcting a broken link.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ 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

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.

@Rohit3523 Rohit3523 self-requested a review December 30, 2025 21:04
@Rohit3523 Rohit3523 requested a deployment to approve_e2e_testing January 7, 2026 17:29 — with GitHub Actions Waiting
@Rohit3523 Rohit3523 requested a deployment to official_ios_build January 7, 2026 17:32 — with GitHub Actions Waiting
@Rohit3523 Rohit3523 requested a deployment to experimental_ios_build January 7, 2026 17:32 — with GitHub Actions Waiting
@Rohit3523 Rohit3523 requested a deployment to official_android_build January 7, 2026 17:32 — with GitHub Actions Waiting
@Rohit3523 Rohit3523 requested a deployment to experimental_android_build January 7, 2026 17:32 — with GitHub Actions Waiting
@Rohit3523 Rohit3523 changed the title Update README.md docs: fix white labeling documentation link in readme Jan 7, 2026
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.

Broken link to Mobile App White Labeling in README.md

2 participants