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

Adding missing PHP classes #11

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

Conversation

sommobilitat
Copy link

Adding missing PHP classes:

  • ApiClient
  • Catalog
  • PortalExample
  • UserInfo

@vokimon
Copy link
Member

vokimon commented Sep 20, 2018

So you are also working with PHP in sommobilitat?? Nice!

In theory, php classes should pass the same tests cases than the equivalent Python classes.

Do you want the PR to be merge or are you still working on it?

@bernatixer
Copy link
Contributor

Yes! We plan to use the PHP version of intercoop

We are still working on the tests, feel free to implement them if you want to

@vokimon
Copy link
Member

vokimon commented Sep 21, 2018 via email

@daniquilez
Copy link
Collaborator

We'll keep working on the necessary tests and keep you posted about our advance!

As we previosly commented, our idea is to use intercoop as the protocol for internal communication between apps.

We'll have odoo as our central data repository so, when a new user signs in in our coorporate website (wordpress - php) we'll use it as portal to send registration data to odoo. And we'll use our client area (flask - python) as portal also to retrieve user data from odoo.

That's why we need both implementations. I hope this architecture idea makes sense to you :)

Thanks for the followup!

@vokimon
Copy link
Member

vokimon commented Sep 21, 2018 via email

@daniquilez
Copy link
Collaborator

Indeed is going to be used for a slightly different purpose as it was designed but at the end we though our goal (send data between entities) doesn't differ that much from the original idea.

We wanted to ask, maybe meet, to see if you're planning to use a completely different approach to implement your internal API of communication between client area and ERP. Would be good if we join development efforts to achieve same goals and try to create tools that can be easily adapted for other cooperatives from the social economy.

We'll be happy to use and try to help as much as we can.

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.

4 participants