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

People get the idea? #100

Closed
sagikazarmark opened this issue Dec 21, 2015 · 9 comments
Closed

People get the idea? #100

sagikazarmark opened this issue Dec 21, 2015 · 9 comments
Labels

Comments

@sagikazarmark
Copy link
Member

Is it just me or people really don't get the idea behind httplug?

This is just one example:

DigitalOceanPHP/Client#75 (comment)

@dbu
Copy link
Contributor

dbu commented Dec 22, 2015 via email

@sagikazarmark
Copy link
Member Author

There are a few arguments which are totally acceptable in my opinion: not being stable, messy documentation, etc.

But people claiming to understand what HTTPlug is are being reluctant about using it and mention arguments which are either not true or reflect that they actually don't get the idea behind.

Wonder if it would be different if it was a PSR.

@dbu
Copy link
Contributor

dbu commented Dec 22, 2015 via email

@Nyholm
Copy link
Member

Nyholm commented Dec 22, 2015

Sure it would be different with a PSR but that is far from the only solution. The situation is that every api client uses at least one http client. If it is a well built library (api client) they have adapters for Guzzle5 and 6 etc.

What we need to do is to explain why the current situation is a bad idea. But not in terms on "more code", "hard to maintain", "keep up with changes in guzzle" etc. I think we need to be more clever than that. Let's put on the hat of data scientist and explain that current situation violates the common reuse principle, dependency inversion principle and by doing adapters yourself you risk violating the liskov substitution principle etc etc. I do also believe that we need to show package dependencies in a graph/images.

The message we want to send is: "You focus on your api, we focus on the transportation of data. Trust us, we know our shit."

There are a few arguments which are totally acceptable in my opinion: not being stable, messy documentation, etc.

I believe these are signs for that developers do not trust us.

@sagikazarmark
Copy link
Member Author

Nice summarizing @Nyholm

@ddeboer
Copy link
Contributor

ddeboer commented Dec 29, 2015

Yes, good summary. In our docs, we should answer the questions:

  • Why should my library/framework depend on Httplug instead of a concrete HTTP client? Answer: everything mentioned by @Nyholm – in one word: decoupling.
  • What do I gain by using Httplug? Answer: you can stop worrying about supporting any concrete clients.

@ddeboer
Copy link
Contributor

ddeboer commented Jan 3, 2016

Documentation has improved a lot over the last couple of days. Please have a look at http://php-http.readthedocs.org/en/latest/ and let me know what we can do to make them even clearer.

@sagikazarmark
Copy link
Member Author

I had an idea to create a two-step migration process: php-http/documentation#57

The idea is that you can use HTTPlug API internally without breaking BC at first.

@sagikazarmark
Copy link
Member Author

Let's solve this. 😄

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

No branches or pull requests

4 participants