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

[13.0][MIG] business_requirement_deliverable: Migration to 13.0 #333

Merged

Conversation

Tardo
Copy link
Member

@Tardo Tardo commented Jan 28, 2020

Depends #332

cc @Tecnativa TT19718

@pedrobaeza pedrobaeza added this to the 13.0 milestone Jan 30, 2020
@Tardo Tardo force-pushed the 13.0-mig-business_requirement_deliverable branch from a707cc0 to 7f1430c Compare January 30, 2020 16:14
@OCA-git-bot OCA-git-bot mentioned this pull request Mar 3, 2020
5 tasks
@pedrobaeza
Copy link
Member

You can rebase this

@Tardo Tardo force-pushed the 13.0-mig-business_requirement_deliverable branch from 7f1430c to 8e16583 Compare March 3, 2020 16:48
@Tardo
Copy link
Member Author

Tardo commented Mar 3, 2020

rebased, ready to review

@pedrobaeza
Copy link
Member

Red Travis

@Tardo Tardo force-pushed the 13.0-mig-business_requirement_deliverable branch from 8e16583 to f1b9eb5 Compare March 5, 2020 15:54
@Tardo
Copy link
Member Author

Tardo commented Mar 5, 2020

Now green

victor and others added 20 commits March 30, 2020 18:54
	new file:   business_requirement_deliverable/README.rst
	new file:   business_requirement_deliverable/__init__.py
	new file:   business_requirement_deliverable/__openerp__.py
	new file:   business_requirement_deliverable/models/__init__.py
	new file:   business_requirement_deliverable/models/business.py
	new file:   business_requirement_deliverable/security/ir.model.access.csv
	new file:   business_requirement_deliverable/static/description/icon.png
	new file:   business_requirement_deliverable/static/img/bus_req.png
	new file:   business_requirement_deliverable/static/img/bus_req_deliverable.png
	new file:   business_requirement_deliverable/static/img/bus_req_deliverable2.png
	new file:   business_requirement_deliverable/static/img/bus_req_resource.png
	new file:   business_requirement_deliverable/static/img/bus_req_tree.png
	new file:   business_requirement_deliverable/tests/__init__.py
	new file:   business_requirement_deliverable/tests/test_br.py
	new file:   business_requirement_deliverable/views/business_view.xml
…rs to improve code and functionality.

Also fix test cases for the new methods to test and remove fields no longer exist.
	modified:   business_requirement_deliverable/models/business.py
	modified:   business_requirement_deliverable/tests/test_br.py
	modified:   business_requirement_deliverable/views/business_view.xml
…avis warnings.

	modified:   business_requirement_deliverable/models/business.py
	modified:   business_requirement_deliverable/tests/test_br.py
…ated method Warning

	modified:   business_requirement_deliverable/models/business.py
…age for coveralls

	modified:   business_requirement_deliverable/tests/test_br.py
	modified:   business_requirement_deliverable/views/business_view.xml
…more than one deliverable lines.

	modified:   business_requirement_deliverable/models/business.py
[FIX] business_requirement: Update README to correct runbot address + figure
[IMP] business_requirement

* Improved and unified the README for the br_deliverable module
* Modified the openerp.py too remove unused images
* Finetuning
* Remove oca.cfg
pedrobaeza and others added 17 commits March 30, 2020 18:54
…less selection

If you track that field + hide prices for users, hidden information is revealed in chatter!
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: business-requirement-12.0/business-requirement-12.0-business_requirement_deliverable
Translate-URL: https://translation.odoo-community.org/projects/business-requirement-12-0/business-requirement-12-0-business_requirement_deliverable/
If you don't belong to the group, you shouldn't see prices, but even if belonging,
put it on name_get will make that messages coming from the BRD discloses the price
to everyone receiving the message, so better to remove it on each case.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: business-requirement-12.0/business-requirement-12.0-business_requirement_deliverable
Translate-URL: https://translation.odoo-community.org/projects/business-requirement-12-0/business-requirement-12-0-business_requirement_deliverable/
When creating a business requirement deliverable (BRD), auto-subscribe followers of the
parent business requirement (BR), similar to what is done on tasks/projects.

This is performed automatically thanks to mail mechanisms, having a mail.message.subtype
in BR having other subtype in BRD as parent. We put them as hidden, as that subtypes
don't have other function than that.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: business-requirement-12.0/business-requirement-12.0-business_requirement_deliverable
Translate-URL: https://translation.odoo-community.org/projects/business-requirement-12-0/business-requirement-12-0-business_requirement_deliverable/
Currently translated at 62.5% (75 of 120 strings)

Translation: business-requirement-12.0/business-requirement-12.0-business_requirement_deliverable
Translate-URL: https://translation.odoo-community.org/projects/business-requirement-12-0/business-requirement-12-0-business_requirement_deliverable/es/
Before this commit the form view on small screens doesn't display
the name of the record in a properly way when uses edition mode.
After this commit the form is fully usable on all modes and
screen sizes.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: business-requirement-12.0/business-requirement-12.0-business_requirement_deliverable
Translate-URL: https://translation.odoo-community.org/projects/business-requirement-12-0/business-requirement-12-0-business_requirement_deliverable/
@Tardo Tardo force-pushed the 13.0-mig-business_requirement_deliverable branch from f1b9eb5 to a56995a Compare March 30, 2020 16:59
@Tardo Tardo force-pushed the 13.0-mig-business_requirement_deliverable branch from a56995a to 5495897 Compare March 30, 2020 17:01
@Tardo
Copy link
Member Author

Tardo commented Mar 30, 2020

Changes done, green.

@Tardo
Copy link
Member Author

Tardo commented Apr 21, 2020

This PR need reviews

@pedrobaeza
Copy link
Member

Fast-tracking this for continuing in the dependency chain

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 13.0-ocabot-merge-pr-333-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 857820f into OCA:13.0 May 4, 2020
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 9115f10. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet