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

[FIX] missing translated terms due to #245 #249

Merged
merged 2 commits into from Sep 15, 2016

Conversation

Projects
None yet
4 participants
@JonathanNEMRY
Copy link

commented Aug 1, 2016

some missing terms into #245

@JonathanNEMRY JonathanNEMRY force-pushed the acsone:8.0-fix-purchase_request_to_rfq_terms-jne branch from 30707bb to 114293d Aug 1, 2016

@JonathanNEMRY

This comment has been minimized.

Copy link
Author

commented Aug 1, 2016

@JonathanNEMRY JonathanNEMRY force-pushed the acsone:8.0-fix-purchase_request_to_rfq_terms-jne branch 2 times, most recently from 91d1c7c to 6fdcec4 Aug 2, 2016

@sbidoul

This comment has been minimized.

Copy link
Member

commented Aug 10, 2016

👍

@pedrobaeza

This comment has been minimized.

Copy link
Contributor

commented Aug 10, 2016

Please remove pot file

@JonathanNEMRY

This comment has been minimized.

Copy link
Author

commented Aug 10, 2016

Aren't .pot the base for transiflex files generation?

@pedrobaeza

This comment has been minimized.

Copy link
Contributor

commented Aug 10, 2016

They are generated on the fly on the Travis build, so keeping an outdated copy only confuses contributors.

@yvaucher

This comment has been minimized.

Copy link
Member

commented Sep 14, 2016

👍 with removal of .pot

@JonathanNEMRY JonathanNEMRY force-pushed the acsone:8.0-fix-purchase_request_to_rfq_terms-jne branch from 6fdcec4 to 2b1f398 Sep 14, 2016

@JonathanNEMRY

This comment has been minimized.

Copy link
Author

commented Sep 14, 2016

done

@yvaucher yvaucher merged commit 9b8928d into OCA:8.0 Sep 15, 2016

4 checks passed

ci/runbot runbot build 3154422-249-2b1f39 (runtime 121s)
Details
codecov/patch Coverage not affected.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.04%) to 74.673%
Details
@JonathanNEMRY

This comment has been minimized.

Copy link
Author

commented Sep 15, 2016

Thanks @yvaucher ! 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.