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

added the assets folder and changed the links accordingly #58

Merged
merged 1 commit into from
May 27, 2021

Conversation

mohit200008
Copy link
Contributor

Fixes Issue

This PR fixes the following issues :
#example

Changes proposed

Here comes all the changes proposed through this PR

Check List (Check all the boxes which are applicable)

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.
  • This PR does not contain copyrighted content.
  • The title of my pull request is a short description of the requested changes.

Screenshots

Add all the screenshots which support your changes

Copy link
Contributor

@vinzvinci vinzvinci left a comment

Choose a reason for hiding this comment

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

Looks good to me 👍 , wait for the maintainer for the final review. Thanks!

Copy link
Owner

@kaiwalyakoparkar kaiwalyakoparkar left a comment

Choose a reason for hiding this comment

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

LGTM 🚀 Sorry for getting back to you late. Thank you very much for your contribution and time. We would love to see many more contributions from you. I appreciate your efforts :)

@kaiwalyakoparkar kaiwalyakoparkar merged commit 996f5a7 into kaiwalyakoparkar:main May 27, 2021
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.

3 participants