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

[13.0] [REF] Web: form widget max upload size #54

Open
wants to merge 8 commits into
base: 13.0
Choose a base branch
from

Conversation

ThomasBinsfeld
Copy link
Member

No description provided.

nle-odoo and others added 8 commits April 28, 2021 14:57
Scenario:

- add a file in mass mailing editor => an icon is added linking the file
- save
- edit => the icon is replaced by a generic icon
- save
- send mail => no icon is shown in email received

The system is using eg. `data-mimetype="pdf"` to show the icon, before
12.0 this worked but in saas-12.3 the system uses mailing.mailing
body_arch's field that is sanitizing attributes and remove it on save,
so the icon is only working in received email if you save one and only
one time after adding file inside the mass mailing.

opw-2474053 (ticket for similar issue in 14.0)

closes odoo#70042

X-original-commit: 7e14515
Signed-off-by: Nicolas Lempereur (nle) <[email protected]>
Using a fiscal position to map a tax to a price-included tax doesn't work.

closes odoo#68997

Signed-off-by: Antoine Vandevenne (anv) <[email protected]>
… a user still linked to it

Description of the issue/feature this PR addresses and current behavior before PR:
There is a discrepancy in the wording of the error message provided. The error reads that an 'internal user' is linked but really its a 'portal user.'.

Desired behavior after PR is merged:
The wording of the error is clearer, and it now says 'portal user' instead of 'internal user'.

OPW: 2498139

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

closes odoo#69916

X-original-commit: cdd94cd
Signed-off-by: Simon Goffin (sig) <[email protected]>
Signed-off-by: Kamen Zhekov <[email protected]>
Some aria attributes were missing or incorrectly set on these new
buttons:
- The button "Export All" was missing an `aria-label` attribute
- The button to show/hide optional columns was also missing an
  `aria-label` attribute, and its menu items (checkboxes to show/hide
  columns) were missing a proper role
- Ever since the debug manager was added in frontend on 73327db,
  the `aria-label` attribute was not being rendered correctly on the
  button to open developer tools

closes odoo#67803

Signed-off-by: Nicolas Lempereur (nle) <[email protected]>
…rtner

Display the default name (the one displayed in the kanban view) for followers replacing
the False value for "other address" partners, this way the user won't get
confused, furthermore, this fix won't require the name to be mandatory for this
type of contacts.

Task-2514244

closes odoo#70340

X-original-commit: 759054a
Signed-off-by: Thibault Delavallee (tde) <[email protected]>
Have in a purchase order a first row with unit price 0
Export the PO adding unit price in the list of fields to export
The unit price is not reported

This occur because the first line of the order lines is meld with the
purchase order line but in the process the 0.0 values is discarded

opw-2510917

closes odoo#70353

X-original-commit: 242bf50
Signed-off-by: agr-odoo <[email protected]>
When the user creates a new account analytic line, he should not be able
to create a new journal item.

To reproduce the error:
(Use demo data)
1. In Settings, enable "Analytic Accounting"
2. Accounting > Configuration > Analytic Accounts > Administrative >
Cost/Revenue
3. Create a new one
4. Click on "Journal Item" list

Error: the list contains the "Create and Edit..." option. Such an action
should not be possible.

OPW-2481197

closes odoo#69881

closes odoo#70376

X-original-commit: f3ef533
Signed-off-by: Quentin De Paoli (qdp) <[email protected]>
Signed-off-by: Adrien Widart <[email protected]>
@ThomasBinsfeld ThomasBinsfeld force-pushed the 13.0-ref_web_form_widget_max_upload_size_tbi branch from 23b272c to f0aee98 Compare May 5, 2021 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants