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

[WIP][DRAFT][MIG][13.0] connector magento #312

Closed
wants to merge 72 commits into from

Conversation

bizzappdev
Copy link

No description provided.

OCA-git-bot and others added 30 commits September 27, 2018 02:04
* Add binding on sales orders
* Ensure that we always have the Connector Manager group on the
  Connector tabs (most are done in the parent views of
  connector_ecommerce)
* Fixed inheritance of the delivery.carrier view
* Use vcr.py to record the exchanges instead of a self-made recorder
* Use new base Components test case classes
* Separate export tests in 2 phases: trigger of the export which check
  that the jobs are delayed, and test the job itself in a second test
I removed it earlier, but it is required when we import products
* Rename it because it was shadowing another test
* Compute in the location's context
The wizard is available on the backend.
@bizzappdev
Copy link
Author

@StefanRijnhart sure will do the needful.

@bizzappdev
Copy link
Author

bizzappdev commented Mar 8, 2021

Not sure why it gets closed I just rebased on 12.0. can anyone reopen again? or I have to create another PR.

@StefanRijnhart
Copy link
Member

You rebased on 12.0 instead of on what I called, for lack of a better word, a new 'format-patch' of this module in 12.0 (with which I tried to refer to this process: https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-13.0#technical-method-to-migrate-a-module-from-120-to-130-branch). So now, the branch histories of your branch and 13.0 are not related anymore as indicated when trying to open a new PR:
image

@bizzappdev
Copy link
Author

New PR created #317
Sorry for the mess up.

@StefanRijnhart
Copy link
Member

Did you try force pushing your branch so that we can continue on this PR?

@bizzappdev
Copy link
Author

I tried but it is not working that way so created new PR

@StefanRijnhart
Copy link
Member

Ah ok, thanks!

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