Skip to content

auto-install=True on donation_recurring_tax_receipt#9

Merged
gurneyalex merged 6 commits intoOCA:8.0from
akretion:8.0-auto-install
Dec 31, 2015
Merged

auto-install=True on donation_recurring_tax_receipt#9
gurneyalex merged 6 commits intoOCA:8.0from
akretion:8.0-auto-install

Conversation

@alexis-via
Copy link
Contributor

No description provided.

@pedrobaeza
Copy link
Member

Can you check red status?

@alexis-via
Copy link
Contributor Author

This red status is still a mystery for me. I added logs, but I still don't understand why my YAML test sometimes fails with this bad date...

@yvaucher
Copy link
Member

@alexis-via I would bet on a timezone issue.

Your test ran at 2015-06-14 22:02:44,906

It says that tax_receipt_id.date=2015-06-15

So when setting the timestamp it might have interpreted it as 2015-06-15 00:02

Maybe you could test getting the date with fields.Date.today() ?

@alexis-via
Copy link
Contributor Author

@yvaucher I can't use "fields." in YAML test, even with new API

@yvaucher
Copy link
Member

@alexis-via you need to do from openerp import fields first.

See an old API exemple which is working in addons/project_timesheet/test/work_timesheet.yml

@alexis-via
Copy link
Contributor Author

I plan to merge it very soon now, so as to update the travis config file with the appropriate version numbers.

@MOSAMAD
Copy link

MOSAMAD commented Oct 31, 2015

👍

1 similar comment
@brother-bernard
Copy link
Contributor

👍

gurneyalex added a commit that referenced this pull request Dec 31, 2015
auto-install=True on donation_recurring_tax_receipt
@gurneyalex gurneyalex merged commit 565546f into OCA:8.0 Dec 31, 2015
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.

6 participants