Skip to content

[16.0][IMP] lighting_vertical: Fix warnings#78

Open
bjkumal wants to merge 16 commits into16.0from
16.0-imp-lighting_vertical
Open

[16.0][IMP] lighting_vertical: Fix warnings#78
bjkumal wants to merge 16 commits into16.0from
16.0-imp-lighting_vertical

Conversation

@bjkumal
Copy link
Contributor

@bjkumal bjkumal commented May 21, 2025

No description provided.

@bjkumal bjkumal requested a review from eantones May 21, 2025 12:57
@bjkumal bjkumal added the to review Must be reviewed label May 21, 2025
@bjkumal bjkumal force-pushed the 16.0-imp-lighting_vertical branch 2 times, most recently from 0b26efa to aaafe66 Compare May 21, 2025 13:13
@bjkumal bjkumal requested review from eantones and removed request for eantones May 21, 2025 13:13
Copy link
Member

@eantones eantones left a comment

Choose a reason for hiding this comment

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

You forgot the translations. Please check this commit out. I modified just the lighting module, please take this as an example to do the same in the other modules. We can discuss if you have doubts when a field string is duplicated.

@eantones eantones added changes requested and removed to review Must be reviewed labels May 26, 2025
@eantones eantones force-pushed the 16.0-imp-lighting_vertical branch from d800adb to 41623e9 Compare May 26, 2025 18:05
@bjkumal bjkumal force-pushed the 16.0-imp-lighting_vertical branch from 41623e9 to 9dee588 Compare May 28, 2025 11:07
@bjkumal bjkumal requested a review from eantones May 28, 2025 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants