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

[ADD] gap_analysis #127

Closed
wants to merge 56 commits into from
Closed

[ADD] gap_analysis #127

wants to merge 56 commits into from

Conversation

xie8899
Copy link

@xie8899 xie8899 commented Nov 13, 2015

No description provided.

@dreispt dreispt added this to the 8.0 milestone Nov 13, 2015
{project_repo}/issues>`_.
In case of trouble, please check there if your issue has already been reported.
If you spotted it first, help us smashing it by providing a detailed and welcomed feedback `here <https://github.com/OCA/
{project_repo}/issues/new?body=module:%20

Choose a reason for hiding this comment

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

Modify accordingly to include the repo/module name

"author": "<AUTHOR(S)>, Odoo Community Association (OCA)",
'depends': [
'base',
'web_ckeditor4',
Copy link
Member

Choose a reason for hiding this comment

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

web_ckeditor4 has a bug, check OCA/web#151

Choose a reason for hiding this comment

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

@rafaelbn Thanks to the pointer. We might switch to standard odoo editor or etherpad (maybe as a separate module).

Copy link
Sponsor Member

Choose a reason for hiding this comment

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

I think this module should stick to the standard HTML fields/widgets, and then provide ckeditor as an option, through an additional module.

Choose a reason for hiding this comment

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

agree

readonly=True,
states={'draft': [('readonly', False)]}
)
odoo_gap = fields.Text(
Copy link
Sponsor Member

Choose a reason for hiding this comment

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

I dislike the odoo specific field names. Can you use just scenario and gap?

Choose a reason for hiding this comment

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

Good point: it will be more generic.

Luke Zheng and others added 10 commits March 14, 2016 16:00
[FIX] Fixed business requirement cost & sales price related issues
…_the_company_currency

[ADD] in BR/Cost control the total revenue should be changed into the company currency.
…12797-modify-module-business-requirement-to-add-link-in-task-and-link-in-br

[IMP] business_requirement: Improve module business_requirement based…
@coveralls
Copy link

Coverage Status

Coverage decreased (-3.5%) to 73.009% when pulling 5c70025 on xie8899:gap_analysis_8.0.2 into 148a25c on OCA:8.0.

string='Count',
compute='_sub_br_count'
)
linked_project = fields.Many2one(

Choose a reason for hiding this comment

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

@victormartinelicocorp This should be push to module business_requirement_project

@coveralls
Copy link

Coverage Status

Coverage decreased (-3.5%) to 73.009% when pulling 8081b9d on xie8899:gap_analysis_8.0.2 into 148a25c on OCA:8.0.

victor and others added 2 commits May 13, 2016 18:13
…BR Tree, fix business_view.xml file (need to upgrade module)
…Issue-3614-Incorrect-BI-view-for-BR-Tree

[FIX] business_requirement: Fix for Issue 3614 Incorrect BI view for …
victormmtorres pushed a commit to victormmtorres/project that referenced this pull request May 18, 2016
…A/project] [ADD] gap_analysis (OCA#127), add module "script_business_requirement_category" which contains erppeek script to migrate category information
@coveralls
Copy link

coveralls commented May 20, 2016

Coverage Status

Coverage decreased (-3.4%) to 73.133% when pulling 2fdd400 on xie8899:gap_analysis_8.0.2 into 148a25c on OCA:8.0.

@elicoidal
Copy link

replaced by #179

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

9 participants