web-app icon indicating copy to clipboard operation
web-app copied to clipboard

WEB-457 Tenant aware feature

Open JaySoni1 opened this issue 2 months ago • 8 comments

Changes Made :-

-Logged user must be aware which is the Tenant that he has signed in.

WEB-457

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.

JaySoni1 avatar Dec 04 '25 14:12 JaySoni1

[!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_status to false in the CodeRabbit configuration file.

[!NOTE]

.coderabbit.yaml has unrecognized properties

CodeRabbit 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, titlecase usage, 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.

❤️ Share

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

coderabbitai[bot] avatar Dec 04 '25 14:12 coderabbitai[bot]

@JaySoni1 there are some items to resolve

IOhacker avatar Dec 08 '25 15:12 IOhacker

@IOhacker Ok I will resolve this and will update the PR soon

JaySoni1 avatar Dec 09 '25 08:12 JaySoni1

@JaySoni1 please let me know once it is ready to review it

IOhacker avatar Dec 10 '25 21:12 IOhacker

@IOhacker Ok

JaySoni1 avatar Dec 10 '25 22:12 JaySoni1

@JaySoni1 squash and commit please :)

IOhacker avatar Dec 11 '25 00:12 IOhacker

@IOhacker OK

JaySoni1 avatar Dec 11 '25 00:12 JaySoni1

@IOhacker I have updated the PR please review

JaySoni1 avatar Dec 17 '25 14:12 JaySoni1

@IOhacker Thank You for the review

JaySoni1 avatar Dec 17 '25 16:12 JaySoni1