Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add note Brand Icon (#363) #364

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

agate-pris
Copy link

@agate-pris agate-pris commented Oct 16, 2021

This PR adds the note Brand Icon.

Fixes #363

image

@agate-pris agate-pris mentioned this pull request Oct 16, 2021
1 task
@agate-pris agate-pris force-pushed the note-com branch 2 times, most recently from 4273391 to 59e4c00 Compare October 17, 2021 06:16
@agate-pris
Copy link
Author

Sorry, this may violate the license, so please wait a moment.

@shinenelson
Copy link
Member

Until you can determine whether this is okay to be merged in, let us keep this pull request as a draft. You may mark it ready for review when you have confirmation that it can be included into the project.

@shinenelson shinenelson marked this pull request as draft October 24, 2021 20:46
@DougInAMug DougInAMug added the icon-request New icon request label Dec 2, 2021
@agate-pris
Copy link
Author

Hello. Sorry for the delay. I have been checked by the team in charge through the note Operations Office and they confirm everything is fine!

@agate-pris agate-pris marked this pull request as ready for review December 16, 2021 08:19
@agate-pris
Copy link
Author

I also rebased to the latest master and made sure that the unicode number wes not conflicting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
icon-request New icon request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Icon Request:note Brand Icon
3 participants