Skip to content

Add assets for LD2410 BLE integration#3925

Merged
frenck merged 1 commit into
home-assistant:masterfrom
930913:add_ld2410_ble_integration
Jan 6, 2023
Merged

Add assets for LD2410 BLE integration#3925
frenck merged 1 commit into
home-assistant:masterfrom
930913:add_ld2410_ble_integration

Conversation

@930913
Copy link
Copy Markdown
Contributor

@930913 930913 commented Dec 13, 2022

Proposed change

Add icon and logo for the ld2410_ble integration

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)
  • Replace an existing icon or logo with a higher quality version
  • Removing an icon or logo

Additional information

Checklist

  • The added/replaced image(s) are PNG
  • Icon image size is 256x256px (icon.png)
  • [n/a] 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)
  • [n/a] hDPI logo image size has min 256px, but max 512px, on the shortest side (logo@2x.png)

@930913 930913 force-pushed the add_ld2410_ble_integration branch from fda0004 to b7cb577 Compare December 13, 2022 18:57
@frenck frenck added the has-parent This PR has a parent PR in a other repo label Dec 28, 2022
@bdraco bdraco added the parent-merged The parent PR has been merged already label Jan 6, 2023
@bdraco
Copy link
Copy Markdown
Member

bdraco commented Jan 6, 2023

Are there higher quality images available for this? If not you might reach out to Hi-link as they have been pretty responsive in the past

@930913
Copy link
Copy Markdown
Contributor Author

930913 commented Jan 6, 2023

Are there higher quality images available for this? If not you might reach out to Hi-link as they have been pretty responsive in the past

Unfortunately not. I reached out to them on this subject, and they just could not seem to understand what I was asking for. Eventually I gave up and settled for what I raised here.

Copy link
Copy Markdown
Member

@frenck frenck left a comment

Choose a reason for hiding this comment

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

Alright, thanks @930913 👍

Let's stick with these for now. If a better quality image can be found / shows up / is provided, that would be awesome 👍

../Frenck

@frenck frenck merged commit faf4c34 into home-assistant:master Jan 6, 2023
tsunglung pushed a commit to tsunglung/brands that referenced this pull request Sep 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

has-parent This PR has a parent PR in a other repo parent-merged The parent PR has been merged already

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants