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

[IMP] sale_timesheet: add Overview button on form #48375

Closed
wants to merge 1 commit into from

Conversation

@sswapnesh
Copy link
Contributor

sswapnesh commented Mar 25, 2020

Fixes #48220

Before this commit, Checking overview of the Project was possible from only kanban view as the overview button was present on that view only.

With this commit, we add the Overview button on from view as well.

Description of the issue/feature this PR addresses:

Current behavior before PR:

Desired behavior after PR is merged:

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

Before this commit, Checking overview of the Project was possible from only kanban view as the `overview` button was present on that view only.

With this commit, we add the `Overview` button on from view as well.
@sswapnesh

This comment has been minimized.

Copy link
Contributor Author

sswapnesh commented Mar 25, 2020

On form view at
screenshot-localhost_8069-2020 03 25-21_48_15

Cc @tivisse

@robodoo robodoo added the CI 🤖 label Mar 25, 2020
@C3POdoo C3POdoo added the Sales label Mar 25, 2020
@tivisse

This comment has been minimized.

Copy link
Contributor

tivisse commented Mar 26, 2020

Hi @sswapnesh Thanks for the feedback.

I've created an internal task waiting for validation by our product owner.

Shouldn't be a big deal honestly, but we're working to finalize the freeze.

We'll keep in touch, have a nice day ;)

@tivisse tivisse closed this Mar 26, 2020
@robodoo robodoo added closed 💔 and removed CI 🤖 labels Mar 26, 2020
@sswapnesh

This comment has been minimized.

Copy link
Contributor Author

sswapnesh commented Mar 26, 2020

Thanks 👍

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

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.