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.

Sign up for GitHub

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

Jump to bottom

[MIG][16.0] business_requirement: Migration to 16.0 #369

Merged
merged 74 commits into from
Jul 13, 2023
Merged

[MIG][16.0] business_requirement: Migration to 16.0 #369

merged 74 commits into from
Jul 13, 2023

Conversation

AntoniRomera
Copy link

No description provided.

victor and others added 30 commits July 4, 2023 10:54
[ADD] business_requirement
bd1dd10 
	new file:   business_requirement/README.rst
	new file:   business_requirement/__init__.py
	new file:   business_requirement/__openerp__.py
	new file:   business_requirement/data/business_data.xml
	new file:   business_requirement/models/__init__.py
	new file:   business_requirement/models/business.py
	new file:   business_requirement/security/ir.model.access.csv
	new file:   business_requirement/security/security.xml
	new file:   business_requirement/static/description/icon.png
	new file:   business_requirement/static/img/bus_req.png
	new file:   business_requirement/static/img/bus_req_alias.png
	new file:   business_requirement/static/img/bus_req_approved.png
	new file:   business_requirement/static/img/bus_req_cancel.png
	new file:   business_requirement/static/img/bus_req_confirmed.png
	new file:   business_requirement/static/img/bus_req_cust_story.png
	new file:   business_requirement/static/img/bus_req_done.png
	new file:   business_requirement/static/img/bus_req_drop.png
	new file:   business_requirement/static/img/bus_req_module_diag.png
	new file:   business_requirement/static/img/bus_req_tags.png
	new file:   business_requirement/static/img/bus_req_tags2.png
	new file:   business_requirement/static/img/bus_req_tree.png
	new file:   business_requirement/static/img/module_diag.graphml
	new file:   business_requirement/tests/__init__.py
	new file:   business_requirement/tests/test_br.py
	new file:   business_requirement/views/business_view.xml
	new file:   business_requirement/wizards/__init__.py
	new file:   business_requirement/wizards/mail_compose_message.py

[FIX] Comments lines and Rst File
	modified:   business_requirement/README.rst
	modified:   business_requirement/models/business.py
Reduced outside margin of the icon
17a33e2 
Improved README and removed the part that is to be in repo README
[FIX] business_requirement:
05e30b8 
GitHub comments for inheritance message_post method to modify subject.
New fields for the review and request BR meaning.
Improve the message meaning for the status to be useful for internal projects
and not only for customer oriented projects.
	modified:   business_requirement/__init__.py
	modified:   business_requirement/models/business.py
	modified:   business_requirement/views/business_view.xml

[REM] business_requirement: Cause it's not longer necessary to inherit
the wizard I remove from the repository.
	deleted:    business_requirement/wizards/__init__.py
	deleted:    business_requirement/wizards/mail_compose_message.py
added the module in the image path in README
d2eab31
[IMP] business_requirement: Adapt current workflow.
1613c48 
If confirmed the user can't modify the first tab (story/scenario/gap)
but can still change the second one (Other information).
If approved, the user can't change the first and second tab.
	modified:   business_requirement/models/business.py

[IMP] business_requirement: Modify business requirement categories
to be simplify the name to "Categories".
Change business requirement "description" to full screen wide.
	modified:   business_requirement/models/business.py
	modified:   business_requirement/views/business_view.xml

[IMP] business_requirement: Change Text field by Html to avoid use Html widget on the view.
Remove sequence field on the Tree view. Next version will use sequence for the Kanban view.
	modified:   business_requirement/models/business.py
	modified:   business_requirement/views/business_view.xml

[FIX] business_requirement: Change some model functions name to follow guideline
and remove not necessary isolated functions.
	modified:   business_requirement/models/business.py
@elicoidal
Update README to correct runbot address + figure (OCA#12)
6ba52a4 
[FIX] business_requirement: Update README to correct runbot address + figure
[IMP] business_requirement: Improvement on master project related to …
1272462 
…BR. (OCA#17)

* [IMP] business_requirement: Improvement on master project related to BR.
	modified:   business_requirement/__openerp__.py
	modified:   business_requirement/models/__init__.py
	new file:   business_requirement/models/project.py
	new file:   business_requirement/views/project.xml

* [FIX] business_requirement: Increase Coverall regards the new model inherited.
	modified:   business_requirement/tests/__init__.py
	new file:   business_requirement/tests/test_project.py

* [FIX] business_requirement: Modify style and format for python and xml files.
	modified:   business_requirement/models/project.py
	modified:   business_requirement/views/project.xml

* [FIX] business_requirement: Fix version number.
	modified:   business_requirement/__openerp__.py

* [FIX] business_requirement: Fix format on XML attributes for views.
	modified:   business_requirement/views/project.xml
@elicoidal
Improve the README and added functional workflow (OCA#18)
553a7d7 
[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

Update image link as they where not visible in app

link to this issue: OCA/maintainer-tools#221
@ pedro can we FT this one?

Added double image handling

testing OCA/maintainer-tools#221
@RawEvan
[IMP]business_requirement: add test case field (OCA#28)
fe048be
@astirpe
[8.0] Fix Duplicate Function for Business Requirements
bb4a3e4 
This PR only updates module business_requirement.
Other affected modules will be fixed with separate PRs.
Fixes OCA#36 (business_requirement).

[8.0] business-requirement: fix link in README.rst

[IMP] Added reference and name_get for Business Requirement

In tests use / for the default ref value
@noahzaozao
[FIX] fix the business_requirement message body empty issue
dad5d0d 
[FIX] fix the business_requirement super method message_post didnt pass parameters
[FIX] business_requirement
de63281 
Need to add Stakeholder Group for this State Approval.
	modified:   business_requirement/__openerp__.py
	modified:   business_requirement/security/security.xml
	modified:   business_requirement/views/business_view.xml
@sbidoul
[UPD] fix website url in manifest
bb5aa7f
@YogeshMahera-SerpentCS
[IMP] followers of master project to BR
50da92b
@kl-adonze
[8.0][FIX] Wizard to generate the project (OCA#42)
c9255c9 
* [FIX]Move when button to generate the project appears. It should get displayed when the stage is set to Stakeholder approval (rename to Approved by Stakeholders)

* [FIX]Move when button to generate the project appears. It should get displayed when the stage is set to Stakeholder approval (rename to Approved by Stakeholders)

* [Fix] Fix unit test

* [IMP] Improved the UT to fix the Travis.

* [IMP] Improved UT for increase Code Coverage.

* [IMP] Added UT to increase coveralls.

* [IMP] Added UT to increase coveralls.

* [IMP] Added UT to increase coveralls.

* [FIX] stakeholder_approval

* [FIX] stakeholder_approval
@serpentcs-dev1
[FIX] Fixed travis
f852476 
[IMP] Changed Workflow.
[IMP][MIG] business_requirement (OCA#112)
1591711 
Need to add reponsible and reviewers.
Responsible should be required and populate on upgrade version
Reviewers are users who are involved on the BR process

[FIX][business_requirement] Name formating improvement (OCA#116)

* [FIX] business_requirement
To fix OCA#114

* [FIX] business_requirement
Fix name_get method as peer code review comments
Need to force string to unicode before replace
@sudhir-serpentcs
[8.0][IMP] Transfer the Gross Profit fields to the Deliverable lines (O…
82a8d31 
…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.

[ADD][business_requirement] Added To be reviewed field in BR (OCA#120)

* [ADD]Added To be reviewed field in BR

* [IMP] Improved form view as per requirement.

* [IMP] Improved code as per comment.

* [FIX] Fixed issues.

* [IMP]Improved Code.

* [IMP] IMproved code.

OCA Transbot updated translations from Transifex
@serpentcs-dev1
[FIX]144 In stages missing the stage drop (OCA#165)
1e71e32 
[IMP]Improved BR-fields position (OCA#166)

* [IMP]Improved BR-fields position

* exchanged requested and responsible

[IMP]default sort on BR list (OCA#167)

[FIX] kanban state editable in BR (OCA#169)

[IMP][BR1030][22333]-Modify layout of BR Form view (OCA#210)
@sudhir-serpentcs
[FIX][8.0-i181-BR001028-T22316] BR ordering on Tree view (OCA#201)
5f619a9 
* [FIX] Fixed order in tree view.

* [FIX] Fixed order
@serpentcs-dev1
[8.0]-[BR1030]-[ADD/IMP][Reference field improvements] (OCA#208)
4eb870f 
* [8.0][BR1030][ADD][Reference field improvements]

* [IMP]Improved code
@YogeshMahera-SerpentCS
[Issue-197][IMP]Improved String (OCA#214)
6e1726d
@sudhir-serpentcs
[IMP] [8.0-i195-BR001028-T22321] Added code to search BR based on Nam…
88934d0 
…e or Description in m2o fields (OCA#203)

* [IMP] Added code to search BR based on Name or Description in m2o fields.

* [IMP] Added UT to improved code coverage.

* [IMP] Modified the variables names as per GIT review.

[8.0-i190] [ADD] Added filter in BR search to allow user to search BR from Name or Description. (OCA#220)
@serpentcs-dev1
[IMP][8.0][22070][ISSUE#115]-Modify projects and BR id relationship (O…
81d7f71 
…CA#192)

* [IMP][8.0][22070][ISSUE#115]-Improved code as per changes

* [IMP]Improved domain and added copy=False in linked_project

* [ADD/IMP]Linked Project with BR and DL with Smartbutton

* [IMP]Improved code

* [IMP]Count number of linked project in BR,removed count function and veriable from project

* [IMP]Improved method name

* [FIX]Fix Travis

* [IMP]Improved code

* [IMP]Improved code

* [IMP] Improved method name to field
@YogeshMahera-SerpentCS
[8.0][I-215][BR1030]-Set Default Values (OCA#219)
1c758b2 
* [IMP][I215][BR1030]Set Default Values

* [IMP]Improved test-cases
@serpentcs-dev1
[WIP][8.0][BR001024] - Added demo data for BR modules (OCA#212)
ce94c85 
* [WIP]:Added Demo Data.

* [ADD]:Added demo data.

* [ADD]Added demo datas for BR modules

* [REM]Removed unnecessary demo data

* [IMP]Improved Demo datas

* [WIP/IMP]Improved demo data code

* [IMP]Improved code as per commments

* [IMP]Improved demo data

* [IMP/ADD]Improved Code and Added Products Demo data for Resource lines

* [IMP]Update noupdate = 1

* [IMP]Demo data improvements.

* [IMP]Improved demo data

* [IMP]Improved demo data as per comment

* [IMP]Improved code

* [IMP]Sale order on draft stage

* [IMP]Improved code

* [IMP]Modified Demo data file name

* [IMP]Improved Identation
@YogeshMahera-SerpentCS
[8.0][IMP]-Improved Origin field (OCA#226)
55d8cb8 
* [8.0][IMP]-Improved Origin field

* [IMP]Improved field Type for Origin field

[8.0]-IMP-Stages as clickable in BR (OCA#227)

* [IMP]Added Stages as clickble removed buttons

* [IMP]Improved code

* [IMP]Improved code

* [IMP]Improved code

* [Added docstring in readgroup method]
@elicoidal
init branch 9.0
6699f50 
moved to __manifest__.py
@YogeshMahera-SerpentCS
[MIG]Migrated Business requirement module from 8.0 to 10.0 (OCA#223)
7b898ba 
* [MIG]Migrated Business requirement module from 8.0 to 10.0

* [FIX]Fix Pylint

* [IMP]Added new blankline in security file

* [IMP]Added Test-Cases for Coverage

* [IMP]Added report,improved fields in py and xml

* [IMP]Improved code

* [IMP] Removed default dimensions from pivot view

* [FIX]Fixed Pylint

* [IMP]Improved code

* [IMP]Improved code

* [FIX]Fixed Travis

* [IMP]Test-cases Improved

* [IMP]Improved view as per OCA standards

* [IMP]Improved assert in test-cases

* [IMP]-Added assertEqual in test-cases

* [IMP]Improved Test-cases

* [IMP]Improved Code as per OCA Standard in XML

* [IMP/REM]Cover whole BR-code and removed unnecessary test-cases.

* [IMP]Cover test-cases default_comany function
@sudhir-serpentcs
Kanban view move from BRP to BR (OCA#243)
506fa6d 
* [WIP]Kanban view move from BRP to BR

* [IMP]AMENDED Changes as per PR 224

* [IMP]AMENDED Changes as per PR 224,Also improved demo data

* [IMP]Improved code

* [IMP]Added read group method for expand empty stages

* [IMP]Removed dimensions

* [IMP]Moved demo data from BR to BRD

* [IMP]Removed fields from pivot views

* [ADD]Added Issue in README.rst
OCA-git-bot and others added 7 commits July 4, 2023 10:54
@OCA-git-bot
[IMP] update dotfiles [ci skip]
10271cd
@victoralmau
[FIX] business_requirement: portal access error
d56e865 
When the user has no access rights the `search` call returns an AccessError.
TT37203
@OCA-git-bot
business_requirement 15.0.1.0.1
650b601
@victoralmau
[IMP] business_requirement: Use _prepare_home_portal_values() function.
337d3cd
@OCA-git-bot
business_requirement 15.0.1.0.2
19e77df
@sysadminmatmoz
Translated using Weblate (Slovenian)
fc66110 
Currently translated at 34.8% (46 of 132 strings)

Translation: business-requirement-15.0/business-requirement-15.0-business_requirement
Translate-URL: https://translation.odoo-community.org/projects/business-requirement-15-0/business-requirement-15-0-business_requirement/sl/
@sysadminmatmoz
Translated using Weblate (Slovenian)
100bc5b 
Currently translated at 100.0% (132 of 132 strings)

Translation: business-requirement-15.0/business-requirement-15.0-business_requirement
Translate-URL: https://translation.odoo-community.org/projects/business-requirement-15-0/business-requirement-15-0-business_requirement/sl/
@AntoniRomera AntoniRomera changed the title 16.0 mig business requirement [MIG][16.0] business_requirement: Migration to 16.0 Jul 4, 2023
@AntoniRomera AntoniRomera force-pushed the 16.0-mig-business_requirement branch 2 times, most recently from ac3c6e6 to 3828342 Compare July 4, 2023 09:11
MRomeera
MRomeera approved these changes Jul 4, 2023
View reviewed changes
Copy link

@MRomeera MRomeera left a comment

Choose a reason for hiding this comment

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

ok

@AntoniRomera AntoniRomera force-pushed the 16.0-mig-business_requirement branch from 3828342 to cb85786 Compare July 5, 2023 12:31
@AntoniRomera AntoniRomera force-pushed the 16.0-mig-business_requirement branch from cb85786 to deda95f Compare July 6, 2023 09:08
@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@AntoniRomera
Copy link
Author

@pedrobaeza Can anyone merge?

@pedrobaeza
Copy link
Member

/ocabot migration business_requirement
/ocabot merge nobump

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Jul 13, 2023
@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 16.0-ocabot-merge-pr-369-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot mentioned this pull request Jul 13, 2023
Open
2 tasks
@OCA-git-bot OCA-git-bot merged commit c429266 into OCA:16.0 Jul 13, 2023
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at da7257b. 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
Reviewers

@FernandoRomera FernandoRomera FernandoRomera approved these changes

@MRomeera MRomeera MRomeera approved these changes

@Christian-RB Christian-RB Awaiting requested review from Christian-RB

Assignees
No one assigned
Labels
approved merged 🎉 ready to merge
Projects
None yet
Milestone
16.0
Development

Successfully merging this pull request may close these issues.

None yet