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

Modify card and readme.md #81

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

sreyaarc
Copy link

@sreyaarc sreyaarc commented Jul 2, 2023

  • Fixed the linkedin typo, aligned the card-text and modified the a tag.
  • Modified README.md by adding proper links to each of the card animations.

@UsmanAhmadSaeed
Copy link
Owner

Thanks for the contribution. We've full schedules currently so will review and approve later. Meanwhile, feel free to work on landing page or other stuff in this repo if you want. Also, please try to make one PR for all changes you make. For example, the previous PR which has not been merged yet can be safely ignored since its changes are available in this PR as well.

@sreyaarc sreyaarc changed the title Modify readme.md Modify card and readme.md Jul 8, 2023
@sreyaarc
Copy link
Author

sreyaarc commented Jul 8, 2023

Ok. Done. The previous PR has been closed. All the changes are present here now.

@sreyaarc
Copy link
Author

@UsmanAhmadSaeed could you please review the code and merge the PR. That way it would be motivating to work on further issues.

@UsmanAhmadSaeed
Copy link
Owner

@sreyaarc I'll review as soon as I could. It's been pretty busy couple of months. Sorry for the delays, and we truly appreciate your input and contributions.

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.

None yet

2 participants