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

[W16-3] Marketing Morph #68

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

Conversation

roanurquhart
Copy link

Copy link

@lestertj lestertj left a comment

Choose a reason for hiding this comment

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

GitHub project page

  • Please update the link at the top of your GitHub page

Readme Page

  • The site doesn't seem to be linked correctly; can you make your repo name uppercase? Comments from this point forth will be based on your .adoc files.
  • Please update your UI mockup to a computer-drawn version.
  • Point your Travis CI build badge to your repo, not S-EDU addressbook

About Us

  • Fill in your development responsibilities (components/roles)
  • Check that all your photos are showing up correctly (no 404)

You will be receiving your UG and DG comments separately.

cchj1995 pushed a commit to cchj1995/addressbook-level4 that referenced this pull request Mar 25, 2019
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