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

[T09-2] Something Daily #76

Open
wants to merge 675 commits into
base: master
Choose a base branch
from

Conversation

arty9
Copy link

@arty9 arty9 commented Oct 15, 2018

No description provided.

@nishantbudhdev
Copy link

nishantbudhdev commented Oct 23, 2018

@arty9 @SHININGGGG @feijunzi @XiaoYunhan

Good work on keeping the documents updated.

  • Unable to see all member's photos through reposense. Please ensure proper format when naming the images.

User guide:

  • Remove reference to addressbook in Section 2.
  • Remove reference to addressbook in the download link.
  • FAQ section is empty.

Developer guide:

  • Some updated diagrams still refer to Addressbook. Consider renaming them to reflect you app name.
  • Activity diagram figure not numbered.

Code:

  • Remove code instead of just commenting out entire methods.

auskure added a commit to auskure/main that referenced this pull request Nov 3, 2018
Fixed missing clearNotes command bug, and bug where edit could not change only Enrolled Modules of a particular contact.
arty9 and others added 30 commits November 12, 2018 20:30
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.

6 participants