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

Updated README.md for better understanding #15

Merged
merged 5 commits into from
Apr 14, 2020

Conversation

Kuljeet-123
Copy link
Contributor

Fixes issue:- #3 .

Changes:-

Updated the documentation or README for better understanding.

Copy link
Member

@arpit9667 arpit9667 left a comment

Choose a reason for hiding this comment

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

@Kuljeet-123 I think you skipped the CSS part given in the current version of the Readme.md. You should take a look at that.

@Kuljeet-123
Copy link
Contributor Author

@arpit9667 I have Updated the code.Please review it and merge it.

Copy link
Collaborator

@plxity plxity left a comment

Choose a reason for hiding this comment

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

@Kuljeet-123 Code of conduct is not present. Please raise an issue for it. We can't merge this before that file because it is mentioned in the README.md but no file is actually present.

@Kuljeet-123
Copy link
Contributor Author

@plxity I have commented in the issue #19 ...Please assign this to me,So that I can add the link of Code of conduct int README.

@plxity
Copy link
Collaborator

plxity commented Apr 8, 2020

@plxity I have commented in the issue #19 ...Please assign this to me,So that I can add the link of Code of conduct int README.

Done 👍

@Kuljeet-123
Copy link
Contributor Author

@plxity @yellowwoods12 I have Updated the README file.Please review it and merge it also.

@Kuljeet-123
Copy link
Contributor Author

@plxity @yellowwoods12 Please review it .

@plxity plxity merged commit 9c76252 into Awesome-React-Modules:master Apr 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants