Laifen

[UrbanTechIO]

Proposed change

Adding Link to the Laifen custom Integration
https://github.com/UrbanTechIO/Laifen

Type of change

• Add a new logo or icon for a new core integration
• Add a missing icon or logo for an existing core integration
• Add a new logo or icon for a custom integration (custom component)
  • I've added a link to my custom integration repository in the PR description
  • I've opened up a PR for my custom integration on the Home Assistant
    Python wheels repository
• Replace an existing icon or logo with a higher quality version
• Replace an existing icon or logo after a branding change
• Removing an icon or logo

Additional information

• This PR fixes or closes issue: fixes #
• Link to code base pull request:
• Link to documentation pull request:
• Link to integration documentation on our website:
• Link to custom integration repository:

Checklist

• The added/replaced image(s) are PNG
• Icon image size is 256x256px (icon.png)
• hDPI icon image size is 512x512px for (icon@2x.png)
• Logo image size has min 128px, but max 256px, on the shortest side (logo.png)
• hDPI logo image size has min 256px, but max 512px, on the shortest side (logo@2x.png)

[klaasnicolaas]

You've currently added product photos but this repo collects branding logos, so I would expect you to add the logo and icon of laifen. Also missing a link to your custom integration

./Klaas

[home-assistant]

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

[UrbanTechIO]

@klaasnicolaas My apologies, this is the first time i am doing this, I have updated the icon and logo to meet the requirements, and added the link to my custom integration.

[UrbanTechIO]

Hello @klaasnicolaas, any chance of getting a bit of your time to review the changes?, thanks.

[frenck]

Hello @klaasnicolaas, any chance of getting a bit of your time to review the changes?, thanks.

Please don't ping for reviews. There are almost a thousand PRs in this org, that all want their stuff merged. Please have a little patience.

../Frenck

[frenck]

The logos used seem to differ from what the manufacturer is using for their branding on their website:

Please make sure the branding is unmodified.

../Frenck

[UrbanTechIO]

Hello @klaasnicolaas, any chance of getting a bit of your time to review the changes?, thanks.

Please don't ping for reviews. There are almost a thousand PRs in this org, that all want their stuff merged. Please have a little patience.

../Frenck

Got it, thanks for pointing out, I have modified the logos according to company branding

[frenck]

Thanks, @UrbanTechIO 👍

../Frenck

Comment addressed