WEB-457 Tenant aware feature
Changes Made :-
-Logged user must be aware which is the Tenant that he has signed in.
Summary by CodeRabbit
-
New Features
- Tenant name now appears in the home page header and footer for clearer context.
- Home page uses tenant-specific branding images for a personalized look.
- Added tenant-related translation strings (e.g., "Tenant" and "To") across supported locales to improve localization.
✏️ Tip: You can customize this high-level summary in your review settings.
[!IMPORTANT]
Review skipped
More than 25% of the files skipped due to max files limit. The review is being skipped to prevent a low-quality review.
189 files out of 296 files are above the max files limit of 100. Please upgrade to Pro plan to get higher limits.
You can disable this status message by setting the
reviews.review_statustofalsein the CodeRabbit configuration file.
[!NOTE]
.coderabbit.yamlhas unrecognized propertiesCodeRabbit is using all valid settings from your configuration. Unrecognized properties (listed below) have been ignored and may indicate typos or deprecated fields that can be removed.
⚠️ Parsing warnings (1)
Validation error: Unrecognized key(s) in object: 'pre_merge_checks'⚙️ Configuration instructions
- Please see the configuration documentation for more information.
- You can also validate your configuration using the online YAML validator.
- If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation:
# yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json
Walkthrough
Adds tenant-aware UI: Home and Footer components now read a tenant identifier from SettingsService and expose it; home header and image are updated to use tenant data; footer shows tenant; translation keys "Tenant" and "To" added across multiple locale files.
Changes
| Cohort / File(s) | Summary |
|---|---|
Home component src/app/home/home.component.ts, src/app/home/home.component.html |
Injected SettingsService, added public tenant: string and helper tenantIdentifier(). Template updated: multi-line mat-card-title with translations and username, and card image source changed to assets/images/{{ tenant }}_home.png. |
Footer component src/app/shared/footer/footer.component.ts, src/app/shared/footer/footer.component.html |
Added public tenant: string and tenantIdentifier(); initialized on component init. Template gains a new table row displaying tenant name with titlecase. |
Translations (13 locales) src/assets/translations/{cs-CS,de-DE,en-US,es-CL,es-MX,fr-FR,it-IT,ko-KO,lt-LT,lv-LV,ne-NE,pt-PT,sw-SW}.json |
Added two translation keys in each file: "Tenant" and "To" with locale-specific values. |
Estimated code review effort
🎯 2 (Simple) | ⏱️ ~10–15 minutes
- Check SettingsService injection and
tenantIdentifier()defaulting behavior in both components. - Verify template bindings: translation keys exist,
titlecaseusage, and the dynamic image path resolves as expected. - Confirm all 13 locale files include both
"Tenant"and"To"keys with correct JSON syntax.
Suggested reviewers
- alberto-art3ch
- IOhacker
- gkbishnoi07
Pre-merge checks and finishing touches
✅ Passed checks (3 passed)
| Check name | Status | Explanation |
|---|---|---|
| Description Check | ✅ Passed | Check skipped - CodeRabbit’s high-level summary is enabled. |
| Docstring Coverage | ✅ Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. |
| Title check | ✅ Passed | The title 'WEB-457 Tenant aware feature' clearly relates to the main changes in the PR, which add tenant-specific functionality (tenant-aware home and footer components, tenant image paths, and multi-language translations for tenant-related text). |
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.
Comment @coderabbitai help to get the list of available commands and usage tips.
@JaySoni1 there are some items to resolve
@IOhacker Ok I will resolve this and will update the PR soon
@JaySoni1 please let me know once it is ready to review it
@IOhacker Ok
@JaySoni1 squash and commit please :)
@IOhacker OK
@IOhacker I have updated the PR please review
@IOhacker Thank You for the review