Skip to content

Add SFR Box icons#3975

Merged
frenck merged 1 commit into
masterfrom
epenet-patch-1
Dec 31, 2022
Merged

Add SFR Box icons#3975
frenck merged 1 commit into
masterfrom
epenet-patch-1

Conversation

@epenet
Copy link
Copy Markdown
Contributor

@epenet epenet commented Dec 29, 2022

Proposed change

Add icons for SFR Box 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)
  • 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)

@epenet epenet marked this pull request as ready for review December 29, 2022 15:22
@frenck frenck added the has-parent This PR has a parent PR in a other repo label Dec 29, 2022
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.

✅ Approved. Can be merged as soon as the parent PR gets merged.

@frenck frenck added the awaits-parent Awaits the merge of an parent PR label Dec 29, 2022
@epenet epenet added parent-merged The parent PR has been merged already and removed awaits-parent Awaits the merge of an parent PR labels Dec 31, 2022
@frenck frenck merged commit da8003e into master Dec 31, 2022
@frenck frenck deleted the epenet-patch-1 branch December 31, 2022 11:04
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.

2 participants