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][15.0][MIG]business_requirement_deliverable:Migration to 15.0 #365

Open
wants to merge 85 commits into
base: 15.0
Choose a base branch
from

Conversation

bizzappdev
Copy link

No description provided.

victor and others added 30 commits March 4, 2023 22:53
	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
* [ADD] BI report for the business requirements

* [FIX] pylint

* [IMP] Added Access rights

* [FIX] travis

* [FIX] made requested changes

* [FIX] Fixed travis

* [IMP] Added Br report file and improved code

* [IMP] Changed as per comments.

* [IMP] Improved code.

* [IMP]Improved SQL readabilityby submethods.

* [IMP]Improved Indentation.
… RL (OCA#113)

* [Add/IMP] Added Smart Buttons and Search views

* [IMP]Fixed Travis

* [WIP] Count Rsource lines and Deliverble lines in smartbuttons also improved code as per comments

* [IMP]Improved access rules as per OCA#103 matrix

* [IMP]Fix Travis

* [IMP] Added graph/pivot views inDL/RL and improved code

* [IMP] fields_view_get by mistake removed method

* [IMP]Improved Code.

* [IMP]In DL Added Sales and prices at RL level.

* [IMP]Improved Indentation

* [IMP]Improved Code.

* [IMP]Improved Code.
…to the customer (OCA#117)

* [IMP] business_requirement_deliverable
Improve customer model to add additional pricelist for estimations

* Update README.rst

* Bumped version number

* [ADD] business_requirement_deliverable
Propery on new API is mean company dependent

* [ADD] business_requirement_deliverable
Add company dependent (property) pricelist for estimation

* Refreshed the README

* Removed the single quotes
…CA#105)

* [IMP] Transfer the Gross Profit fields to the Deliverable lines

* [IMP] Changed calculation for Gross Profit.

* [IMP]Transfer the Gross Profit fields to the Deliverable lines

* [ADD]Added Description field

* [IMP] Improved code as per comment.

* [IMP]changed the position of the fields.

* [IMP] Improved code as per comment.

* [IMP]Improved pivot views.

* [FIX] Fixed Travis & Changed code as per comments.

* [FIX] made requested changes.
* [FIX] Fixed deliverable issues

* [FIX] Improved code for UT

OCA Transbot updated translations from Transifex
* [WIP] business_requirement_deliverable
- Fix the meaning of internal field level later will have same name for different meaning
  First means Sales price on one module and later is meaning Cost
- Fix the wrong module to achieve the requirement

* [FIX] travis

* [FIX] business_requirement_deliverable_cost
- Fix Travis

* [FIX] business_requirement_deliverable_cost
- Transfer fields from module BRD to BRDCost
- Fix Travis

* [FIX]travis

* [FIX] business_requirement_deliverable_cost/views/business.xml]
- Fix Travis
- Wrong view model

* [FIX] business_requirement_deliverable_cost
- Fix Travis regards UT
* [FIX] business_requirement_deliverable
- Fix double factor conversion when UoM
- Fix Total revenue should be recomputed when sales price on DL changes

* [FIX] business_requirement_deliverable
- Fix Travis by UT fix

* [FIX] business_requirement_deliverable
- Fix coveralls and UT
* [FIX] business_requirement_deliverable
- Add the estimation pricelist from customer to Deliverable
calculation.

* [FIX] business_requirement_deliverable
- Fix remove unnecessary else
…ription if previously filled (OCA#132)

* FIX don't change deliverable description if previously filled

* FIX travis with test modification suggested by @darshan-serpent

* FIX travis with test modification suggested by @darshan-serpent

* Revert "FIX travis with test modification suggested by @darshan-serpent"

This reverts commit 6df49d5.

* Revert "FIX travis with test modification suggested by @darshan-serpent"

FIX don't change deliverable description if previously filled

* FIX travis as recommended by @darshan-serpent

* FIX product_id onchange tests

* fixed L81 description
@pedrobaeza
Copy link
Member

/ocabot migration business_requirement_deliverable

@OCA-git-bot OCA-git-bot added this to the 15.0 milestone Mar 5, 2023
@OCA-git-bot OCA-git-bot mentioned this pull request Mar 5, 2023
5 tasks
@bizzappdev bizzappdev force-pushed the 15.0-mig-business-requirement-deliverable-BAD-SIP branch 5 times, most recently from b2c23c7 to f715c30 Compare March 6, 2023 12:11
@bizzappdev bizzappdev force-pushed the 15.0-mig-business-requirement-deliverable-BAD-SIP branch from 3f70ec1 to 77c5c22 Compare March 7, 2023 06:47
@dreispt
Copy link
Sponsor Member

dreispt commented Apr 27, 2024

/ocabot merge major

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 15.0-ocabot-merge-pr-365-by-dreispt-bump-major, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Apr 27, 2024
Signed-off-by dreispt
@OCA-git-bot
Copy link
Contributor

@dreispt your merge command was aborted due to failed check(s), which you can inspect on this commit of 15.0-ocabot-merge-pr-365-by-dreispt-bump-major.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

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