Skip to content

docs(website): try to add opendal logo#2159

Merged
Xuanwo merged 4 commits intoapache:mainfrom
PsiACE:opendal-logo
Apr 29, 2023
Merged

docs(website): try to add opendal logo#2159
Xuanwo merged 4 commits intoapache:mainfrom
PsiACE:opendal-logo

Conversation

@PsiACE
Copy link
Copy Markdown
Member

@PsiACE PsiACE commented Apr 28, 2023

img

Just added the logo to the navbar, need some advice.

Signed-off-by: Chojan Shang <psiace@apache.org>
@PsiACE PsiACE requested a review from suyanhanx April 28, 2023 12:37
Copy link
Copy Markdown
Member

@suyanhanx suyanhanx left a comment

Choose a reason for hiding this comment

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

LGTM.

Comment thread website/docusaurus.config.js Outdated
@suyanhanx suyanhanx requested a review from Xuanwo April 28, 2023 12:44
Co-authored-by: Suyan <suyanhanx@gmail.com>
@PsiACE PsiACE marked this pull request as ready for review April 28, 2023 13:14
@PsiACE PsiACE requested review from ClSlaid and sundy-li as code owners April 28, 2023 13:14
Comment thread website/docusaurus.config.js
Signed-off-by: Chojan Shang <psiace@apache.org>
Comment thread README.md
@Xuanwo Xuanwo merged commit ea8e938 into apache:main Apr 29, 2023
@PsiACE PsiACE deleted the opendal-logo branch April 29, 2023 08:52
@Xuanwo Xuanwo mentioned this pull request May 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants