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

Update documentation #29

Open
DavidLemayian opened this Issue Sep 12, 2017 · 4 comments

Comments

Projects
None yet
3 participants
@DavidLemayian
Member

DavidLemayian commented Sep 12, 2017

The documentation for this extension could be better so let's add comments into the code as docstrings and inline where needed.

@andela-onnenanya

This comment has been minimized.

Show comment
Hide comment
@andela-onnenanya

andela-onnenanya Sep 12, 2017

Contributor

Hi @DavidLemayian I will want to work on this issue, but I need some clarifications.

When you talk about documentation, is it the code documentation or the user guide? If it's the code documentation, that means I will have to check all applicable code/files and add proper documentation.

Contributor

andela-onnenanya commented Sep 12, 2017

Hi @DavidLemayian I will want to work on this issue, but I need some clarifications.

When you talk about documentation, is it the code documentation or the user guide? If it's the code documentation, that means I will have to check all applicable code/files and add proper documentation.

@DavidLemayian

This comment has been minimized.

Show comment
Hide comment
@DavidLemayian

DavidLemayian Sep 13, 2017

Member

@andela-onnenanya This would be for the code documentation but we should definitely do both ( #27 ).

Member

DavidLemayian commented Sep 13, 2017

@andela-onnenanya This would be for the code documentation but we should definitely do both ( #27 ).

@andela-onnenanya

This comment has been minimized.

Show comment
Hide comment
@andela-onnenanya

andela-onnenanya Sep 14, 2017

Contributor

Ok, I'm picking up this issue.

Contributor

andela-onnenanya commented Sep 14, 2017

Ok, I'm picking up this issue.

@andela-onnenanya

This comment has been minimized.

Show comment
Hide comment
@andela-onnenanya

andela-onnenanya Sep 18, 2017

Contributor

The PR for this issue is waiting for reviews and merging.

Contributor

andela-onnenanya commented Sep 18, 2017

The PR for this issue is waiting for reviews and merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment