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.

Sign up for GitHub

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

Jump to bottom

Screenshots added to the readme file #112

Closed
wants to merge 1 commit into from
Closed

Screenshots added to the readme file #112

wants to merge 1 commit into from

Conversation

ZER-0-NE
Copy link
Member

shortlog: Readme.md

Added screenshots for the readme.md file

Fixes #95

@PaliwalSparsh
Copy link
Collaborator

@ZER-0-NE thanks for the PR.
@djmgit we didn't address this PR. I was thinking about adding a gif instead of 3 screenshots taking so much place in the README. What do you think ?

@akshatnitd
Copy link
Member

@PaliwalSparsh Yes, that is a good idea. It would be a cleaner approach. Issue open at #107 .

@djmgit
Copy link
Member

djmgit commented Nov 3, 2017

@PaliwalSparsh @akshatnitd Yes, GIF will be a better option.
@ZER-0-NE Thanks for showing interest in this project :) Can you please checkout the other issues. I would recommend you to see #96, #97 (a the issues contain links to resources from where you can learn how to solve it), #116 to get started. Hope to see more contributions from you 👍

@ZER-0-NE
Copy link
Member Author

ZER-0-NE commented Nov 3, 2017

@djmgit will be glad to contribute more. 👍

@PaliwalSparsh
Copy link
Collaborator

Closing the PR #107 fixed this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@ZER-0-NE @PaliwalSparsh @akshatnitd @djmgit