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

Update Readme with helpful links for the Umami demo. #8

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

Conversation

dgading
Copy link

@dgading dgading commented Oct 16, 2017

I wanted to open this pull request to add some extra information relevant to the README. I looked through Angular and Ember projects to find similar data that was useful for this project, which were mostly links to resources. I wasn't sure on any build or contribution guidelines so I left those off for now.

Since this build is also using patterns from Atomic Design, I decided to link to a blog post that gives a nice overview of it. I didn't know if there is a better source, but this is the one I normally send to other developers looking into it.

README.md Outdated
- [Contenta](http://www.contentacms.org/)
- [Sample Source Data](http://live-contentacms.pantheonsite.io/)
- [Sample Source Data APIs](http://live-contentacms.pantheonsite.io/api)
- [Demo Implementation Guidelines](https://github.com/contentacms/contenta_jsonapi/blob/8.x-1.x/docs/demos-consumers-implementation-guidelines.md)
Copy link
Member

Choose a reason for hiding this comment

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

@dgading
Copy link
Author

dgading commented Oct 17, 2017

@e0ipso That is a much better link, thanks. Since it has all the information for APIs included, I went ahead and removed the redundant Demo source API link.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants