Skip to content

Website logo added#67

Open
vedikaag99 wants to merge 6 commits intoBALaka-18:masterfrom
vedikaag99:patch-1
Open

Website logo added#67
vedikaag99 wants to merge 6 commits intoBALaka-18:masterfrom
vedikaag99:patch-1

Conversation

@vedikaag99
Copy link

@vedikaag99 vedikaag99 commented Mar 8, 2021

Description of the change

Fixes #62
I was not sure on where to add the logo so I have made a seperate folder named asset and have added the folder there. @BALaka-18 kindly review.

Related Issues & Documents (if needed)

What type of PR is this? (check all applicable)

  • Refactoring existing code
  • Adding tests to existing code
  • New Feature
  • Bug Fix
  • Optimization
  • Documentation Update

QA Instructions(if necessary), Screenshots of Output or Recordings

Screen Shot 2021-03-08 at 7 30 19 PM
Screen Shot 2021-03-08 at 7 29 56 PM

_Please replace this line with instructions on how to test your changes

Added tests?

  • Yes
  • No, because they aren't needed
  • No, because I need help

Added to documentation / readme or updated requirements.txt(if needed)?

  • Changed documentation in readme
  • No documentation needed
  • Updated requirements.txt

@restyled-io restyled-io bot mentioned this pull request Mar 8, 2021
@vedikaag99 vedikaag99 changed the title Patch 1 Website logo added Mar 8, 2021
@vedikaag99
Copy link
Author

vedikaag99 commented Mar 8, 2021

@BALaka-18 I have added a backgroundless logo, if you want I can give the logo a white background as shown in the issue. And I can also change the size of the logo as required. Please review.

@BALaka-18
Copy link
Owner

@vedikaag99 I don't think the path is being read correctly, because the logo isn't showing in the README when I viewed the files.

@vedikaag99
Copy link
Author

@vedikaag99 I don't think the path is being read correctly, because the logo isn't showing in the README when I viewed the files.

Can you recheck again? Because the logo can be seen in my branch.

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.

Website logo

2 participants