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 overview #11

Merged
merged 3 commits into from
Dec 4, 2014
Merged

Update overview #11

merged 3 commits into from
Dec 4, 2014

Conversation

fosrias
Copy link
Contributor

@fosrias fosrias commented Nov 24, 2014

@smizell @zdne Please review and merge. Updated README and a little bit of organization a cleanup.

![Representor Pattern Diagram](assets/representor_pattern_diagram.png?raw=true)

The heart of _The Hypermedia Project_ is developing a suite of libraries in different languages that provide
tooling for implmenting the _Representor Pattern_. For details on how this pattern can be applied to full-stack
Copy link
Member

Choose a reason for hiding this comment

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

implmenting -> implementing

@zdne
Copy link
Member

zdne commented Nov 25, 2014

Done with the review. Great job @fosrias !

However I would like to see https://github.com/the-hypermedia-project/charter/blob/update_overview/design/representor.md with this PR as well. Is this the plan?

@fosrias
Copy link
Contributor Author

fosrias commented Nov 25, 2014

@zdne So, part of me wants to consolidate ideas in representor.md that evolve from actually implementing them vs. theorizing the interface. Thus, it is sparse. I was not planning to do more in this PR, but if you and anyone else (@kylef @smizell) want me to, can do that. Please vote.

@fosrias fosrias mentioned this pull request Nov 25, 2014
@smizell
Copy link
Contributor

smizell commented Dec 4, 2014

👍

This is good to merge. If all is good for you, I can merge.

@fosrias
Copy link
Contributor Author

fosrias commented Dec 4, 2014

@smizell Please do.

smizell added a commit that referenced this pull request Dec 4, 2014
@smizell smizell merged commit 95d8c7a into master Dec 4, 2014
@smizell
Copy link
Contributor

smizell commented Dec 4, 2014

Merged!

@smizell smizell deleted the update_overview branch December 4, 2014 01:49
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

3 participants