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

Master project ux improvement pat #22845

Closed

Conversation

pat-odoo
Copy link

@pat-odoo pat-odoo commented Feb 6, 2018

@C3POdoo C3POdoo added the RD research & development, internal work label Feb 7, 2018
@pat-odoo pat-odoo force-pushed the master-project-ux-improvement-pat branch 3 times, most recently from 6b57cc4 to ef5f5e9 Compare February 22, 2018 08:43
@pat-odoo pat-odoo force-pushed the master-project-ux-improvement-pat branch 3 times, most recently from 1e09a6e to 3c20e1b Compare March 20, 2018 10:35
@hmo-odoo hmo-odoo added the Services Project, timesheet, helpdesk,... label Mar 20, 2018
@mba-odoo mba-odoo force-pushed the master-project-ux-improvement-pat branch 3 times, most recently from 086e855 to f9410cb Compare April 24, 2018 11:05
@mba-odoo mba-odoo force-pushed the master-project-ux-improvement-pat branch from f9410cb to 4265854 Compare May 9, 2018 06:17
@mba-odoo mba-odoo force-pushed the master-project-ux-improvement-pat branch 3 times, most recently from 0f3ba1b to 2408439 Compare May 16, 2018 11:45
@mba-odoo
Copy link
Contributor

LGTM

@mba-odoo mba-odoo force-pushed the master-project-ux-improvement-pat branch 2 times, most recently from 9c2f4a0 to fce22a6 Compare May 31, 2018 05:27
@rdeodoo rdeodoo force-pushed the master-project-ux-improvement-pat branch from fce22a6 to 3816964 Compare April 19, 2019 05:27
@rdeodoo
Copy link
Contributor

rdeodoo commented Apr 19, 2019

@JKE-be This one can be merged, it had some conflicts since it was 8500 commits behind, hopefully runbot will be green.
I rebased and edited the commit messages

@rdeodoo rdeodoo force-pushed the master-project-ux-improvement-pat branch from 3816964 to 977de5f Compare April 19, 2019 10:13
@odoo odoo deleted a comment from robodoo Apr 19, 2019
@odoo odoo deleted a comment from robodoo Apr 19, 2019
@robodoo
Copy link
Contributor

robodoo commented Apr 19, 2019

I'm sorry, @JKE-be. r- makes no sense in the current PR state.

Parth Patel added 2 commits April 19, 2019 15:52
Fixing missed 'pivot_row_groupby' assigned to groupedBy in pivot_model.js in
Load function (Already done in reload function), which was missed in commit
42f22a2.
Added qunit test case for testing correct value.
Apply pivot_row_groupby on user in project

task-54027
Display field in two groups to display in four column.

Plus, email_from is used to 'Allow communication with a customer without
creating a partner'.
Since this field is not very useful, it is best to hide it to clean the UI.
Something was already done at aa33c81

task-54027
@rdeodoo rdeodoo force-pushed the master-project-ux-improvement-pat branch from 977de5f to ef451ce Compare April 19, 2019 10:23
@rdeodoo
Copy link
Contributor

rdeodoo commented Apr 19, 2019

Runbot should be green now, after rebase Qunit tests needed to be adapted to async/await

@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Apr 19, 2019
@JKE-be
Copy link
Contributor

JKE-be commented Apr 19, 2019

@robodoo r+

@JKE-be
Copy link
Contributor

JKE-be commented Apr 19, 2019

@robodoo retry r+ rebase-ff

@robodoo
Copy link
Contributor

robodoo commented Apr 19, 2019

Merge method set to rebase and fast-forward

@robodoo
Copy link
Contributor

robodoo commented Apr 19, 2019

I'm sorry, @JKE-be. Retry makes no sense when the PR is not in error. This PR is already reviewed, reviewing it again is useless.

@JKE-be
Copy link
Contributor

JKE-be commented Apr 19, 2019

@robodoo what are you waiting ?

robodoo pushed a commit that referenced this pull request Apr 19, 2019
Display field in two groups to display in four column.

Plus, email_from is used to 'Allow communication with a customer without
creating a partner'.
Since this field is not very useful, it is best to hide it to clean the UI.
Something was already done at aa33c81

task-54027

closes #22845

Signed-off-by: Jérémy Kersten (jke) <jke@openerp.com>
@robodoo
Copy link
Contributor

robodoo commented Apr 19, 2019

Merged, thanks!

@robodoo robodoo closed this Apr 19, 2019
@rdeodoo rdeodoo deleted the master-project-ux-improvement-pat branch April 21, 2019 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI 🤖 Robodoo has seen passing statuses RD research & development, internal work Services Project, timesheet, helpdesk,...
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants