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: show full datetime #52193

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

sswapnesh
Copy link
Contributor

There is no point to hide time from the field while full date-time is visible on list view.

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

There is no point to hide time from the field while full date-time is visible on list view.
@robodoo robodoo added seen 🙂 CI 🤖 Robodoo has seen passing statuses labels May 29, 2020
@C3POdoo C3POdoo added the Sales Sales label May 29, 2020
@mart-e
Copy link
Contributor

mart-e commented Oct 19, 2020

There is no point to hide time from the field while full date-time is visible on list view.

Actually the widget="date" was added in the tree view at 900d162 by @aab-odoo so don't know if we want to have date everywhere or datetime in the form view is fine ?
Aaron, opinion?

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 Sales Sales
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants