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

[8.0] add ngo_purchase_requisition #36

Merged
merged 10 commits into from
Nov 17, 2014

Conversation

yvaucher
Copy link
Member

This create a new view for purchase.requisition specific to NGO verticalization

Depends on:

@yvaucher yvaucher force-pushed the 8.0-add-ngo_purchase_requisition branch 2 times, most recently from a8a6563 to 64b3f54 Compare November 10, 2014 19:07
@coveralls
Copy link

Coverage Status

Coverage increased (+9.96%) when pulling 99fc059 on yvaucher:8.0-add-ngo_purchase_requisition into 19dfec7 on OCA:8.0.

This module serves as a base for NGO addons. It add a view
which provides better priority on fields for NGO needs.
@yvaucher yvaucher force-pushed the 8.0-add-ngo_purchase_requisition branch from 99fc059 to 3cf07b0 Compare November 12, 2014 12:35
@coveralls
Copy link

Coverage Status

Coverage increased (+9.96%) when pulling 3cf07b0 on yvaucher:8.0-add-ngo_purchase_requisition into 19dfec7 on OCA:8.0.

@lepistone
Copy link
Member

The dependency OCA/purchase-workflow#47 is now merged. I am restarting travis here.

@gurneyalex
Copy link
Member

👍

@yvaucher
Copy link
Member Author

AttributeError: Field dest_address_id does not exist

Missing dependency on purchase_requisition_delivery_address

@lepistone
Copy link
Member

@yvaucher I don't know why it didn't find the module, I just merged it. Do you have an idea?

@yvaucher yvaucher self-assigned this Nov 13, 2014
@yvaucher
Copy link
Member Author

@lepistone investigating

@yvaucher
Copy link
Member Author

Cannot reproduce maybe a cache issue ? I restart travis to see.

@coveralls
Copy link

Coverage Status

Coverage increased (+9.96%) when pulling cb22f49 on yvaucher:8.0-add-ngo_purchase_requisition into 19dfec7 on OCA:8.0.

@coveralls
Copy link

Coverage Status

Coverage increased (+9.96%) when pulling d4eeac3 on yvaucher:8.0-add-ngo_purchase_requisition into 19dfec7 on OCA:8.0.

@yvaucher
Copy link
Member Author

Error is in tests. Model do not have dest_address_id at this time.

self.pool['purchase.requisition'].fields_get(cr, user, None).get('dest_address_id')

@yvaucher yvaucher force-pushed the 8.0-add-ngo_purchase_requisition branch from c65e9d6 to ebbcf36 Compare November 13, 2014 18:27
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling ebbcf36 on yvaucher:8.0-add-ngo_purchase_requisition into c5886fd on OCA:8.0.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling ebbcf36 on yvaucher:8.0-add-ngo_purchase_requisition into c5886fd on OCA:8.0.

<div class="pull-left">
<label for="name" class="oe_edit_only oe_inline"/>
<h1>
<field name="name" class="oe_inline" attrs="{'readonly': [('state','not in',('draft'))]}"/>
Copy link
Member Author

Choose a reason for hiding this comment

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

Missing coma in tuple to replace by !=

@gurneyalex
Copy link
Member

👍 again

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling d05cbb9 on yvaucher:8.0-add-ngo_purchase_requisition into c5886fd on OCA:8.0.

@lepistone
Copy link
Member

👍 thanks!

lepistone added a commit that referenced this pull request Nov 17, 2014
@lepistone lepistone merged commit bdf7def into OCA:8.0 Nov 17, 2014
@yvaucher yvaucher deleted the 8.0-add-ngo_purchase_requisition branch November 17, 2014 09:42
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

4 participants