-
Notifications
You must be signed in to change notification settings - Fork 10
Syncing from upstream odoo/odoo (tmp.saas-18.4) #34144
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
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Steps to reproduce: Traceback occurs when printing the eWaybill. When the eWaybill expiry date is not present in the JSON response, any usage of date.strftime(...) caused a traceback. This fix ensures the date is checked before formatting to prevent such errors. Possible issue after the post migration of the database to saas-18.1 opw-4919575 X-original-commit: 074a7b7 Part-of: #217587 Signed-off-by: Harsh Modi (hamo) <[email protected]>
- The E-Waybill report currently displays the E-Waybill generation date in The 'Document Details' section of the report. Steps to reproduce: 1) Enable the Indian E-Waybill configuration from Settings. 2) Create an invoice with a past/future date. 3) Print the E-Waybill. 4) The 'Document Details' section incorrectly shows the E-Waybill date. - This fix ensures that the document date is displayed instead of the E-Waybill date. closes #217587 X-original-commit: d31702b Signed-off-by: Harsh Modi (hamo) <[email protected]>
This commit prevents the delivery date from being set to invoice date by default in French localization. We need the delivery date to be only set by the users. task-4900043 closes #217600 X-original-commit: 18f7191 Signed-off-by: Maximilien La Barre (malb) <[email protected]> Signed-off-by: Hesham Saleh (hsal) <[email protected]>
…ogging BUG - when you select 2 companies in the multi company widget (A & B, default : A) and I create a work entry for an employee of company B. The work entry will be related to company A. Expected behavior: - The default company should be the employee one if has a one FIX - check if the employee has a company add it to the create vals_list closes #217533 Task: 4781100 X-original-commit: 6e76b27 Related: odoo/enterprise#89541 Signed-off-by: Yannick Tivisse (yti) <[email protected]> Signed-off-by: Abdelrahman Mahmoud (amah) <[email protected]>
Following the fix in d927a7b, we broke the default behaviour of empty text/chart fields. While the server returns the value `false` when they're empty, we want to display an empty string in the `ODOO.LIST` formulas. Task-4897690 closes #216305 X-original-commit: 7a40285 Signed-off-by: Pierre Rousseau (pro) <[email protected]> Signed-off-by: Rémi Rahir (rar) <[email protected]>
Before this commit: - go to Accounting > Configuration > Chart of Accounts - in the search bar, you cannot group accounts by account root. After this commit: - You can group accounts by account root ('Root'). We do this by implementing `_field_to_sql` for `account.account.root_id` which makes the field groupable. task-none closes #217567 X-original-commit: 4bdc5af Signed-off-by: William André (wan) <[email protected]>
The POS supports to get your invoice using a QR code present on a POS ticket after the fact. Support for it is not easily done for public users and will be supported later; but for logged in users we can display the fields on their portal to allow them to fetch the invoice correctly by themselves. X-original-commit: fc8488f Part-of: #217607 Signed-off-by: Nicolas Viseur (vin) <[email protected]>
Adds support for sending consolidated invoices to MyInvois from the PoS. task-4558045 closes #217607 X-original-commit: fb484f9 Signed-off-by: Nicolas Viseur (vin) <[email protected]>
Steps to Reproduce: - Install website and website_event module. - Navigate to Events -> Attendees -> Calendar View. Issue: - While accessing the calendar view, we are getting an error because that view needs the event_end_date field to be searchable. We have added search method for field event_begin_date https://github.com/odoo/odoo/pull/210532/files#diff-b8b0a21217ded7346150f1dd8353fbe2932e83f5e3d3755fd6e1c8281bb50c64R62 but missed it for event_end_date. opw-4892275 closes #217510 X-original-commit: a3c783a Signed-off-by: Jérémy Hennecart (jeh) <[email protected]> Signed-off-by: Karna Modasiya (kmod) <[email protected]>
The test "Can edit message comment in chatter" fails often non-determinically on runbot. This happens because this test heavily uses the edit message feature, which has client-side code receiving store data from both RPC returns and bus notifications. When CPU load is high which happens frequently on runbot, test shows race-condition where bus notifications are received much later than RPC return part and thus discuss state becomes wrong by having outdated data. This is an architectural issue that takes time to solve, so this test is skipped in the meantime. Note that while the problem occurs quite a lot on runbot in HOOT tests, in practice this happens quite rarely: bus notifications should be heavily throttled. Also HOOT test fails because of unfortunately case of receiving exactly outdated message body data before starting message edition. When message is being edited, any following store data with change of message body is ignored because the text in composer is intended to not change while editing the message. Kinda fixes runbot 227618 closes #217590 X-original-commit: c24c4ac Signed-off-by: Matthieu Stockbauer (tsm) <[email protected]> Signed-off-by: Alexandre Kühn (aku) <[email protected]>
On mobile, opening the message actions requires precise tapping, as the user has to touch a very small icon. This PR replaces that with a long press, a more intuitive and classical way to access these actions. Part-of: #217310 Related: odoo/enterprise#89402 Signed-off-by: Alexandre Kühn (aku) <[email protected]>
On mobile, the message action menu is a modal, as are the view/add reactions panels. Modals add a dark background overlay to the page. In this case, the background becomes darker when accessing a submenu and lighter when returning from it. This change in background intensity feels a bit odd. This commit ensures that there is at most one modal background. Part-of: #217310 Related: odoo/enterprise#89402 Signed-off-by: Alexandre Kühn (aku) <[email protected]>
The message reaction menu shows who reacted to a message. Currently, it opens as a modal in the middle of the screen, unlike other modals which appear at the bottom (e.g., add reaction). This also hides the active message highlight. Additionally, the menu closes immediately when clicking away, unlike other submenus, making navigation harder. This commit moves the reaction menu to the bottom like add reaction, tweaks its style for clarity on mobile, and changes click-away to return to the main menu—improving mobile usability. Part-of: #217310 Related: odoo/enterprise#89402 Signed-off-by: Alexandre Kühn (aku) <[email protected]>
Now that longpress is used to open the message actions menu, it's not possible to select text inside a message anymore. This commit adds a "copy" option that allows to copy the message text.tmp Part-of: #217310 Related: odoo/enterprise#89402 Signed-off-by: Alexandre Kühn (aku) <[email protected]>
Looks best with discuss style that uses rounded shapes. closes #217310 Related: odoo/enterprise#89402 Signed-off-by: Alexandre Kühn (aku) <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
bt_gitbot