Skip to content

Conversation

@fw-bot
Copy link
Contributor

@fw-bot fw-bot commented Apr 21, 2024

Steps to reproduce:

  • wrap a pivot function inside a IFERROR e.g. =IFERROR(PIVOT("1", "probability"), 42)
  • reload the spreadsheet
  • before the pivot is loaded (throttle the network in the dev tools): right click the cell
  • click on "See records" menu item => boom

Task: 3847477
X-original-commit: odoo/enterprise@31401c75baa4252917fe5d808be0abd5fc23a906


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

Forward-Port-Of: #162759
Forward-Port-Of: #162568

@robodoo robodoo added forwardport This PR was created by @fw-bot conflict There was an error while creating this forward-port PR labels Apr 21, 2024
@robodoo
Copy link
Contributor

robodoo commented Apr 21, 2024

Pull request status dashboard.

@fw-bot
Copy link
Contributor Author

fw-bot commented Apr 21, 2024

@LucasLefevre cherrypicking of pull request #162568 failed.

stdout:

Auto-merging addons/spreadsheet/static/src/pivot/plugins/pivot_ui_plugin.js
CONFLICT (content): Merge conflict in addons/spreadsheet/static/src/pivot/plugins/pivot_ui_plugin.js

stderr:

13:03:08.213585 git.c:463               trace: built-in: git cherry-pick c94abfd5c14d11bfc132be1d67520e94e1ed48d6
error: could not apply c94abfd5c14d... [FIX] documents_spreadsheet: fix See record menu if not loaded
hint: After resolving the conflicts, mark them with
hint: "git add/rm <pathspec>", then run
hint: "git cherry-pick --continue".
hint: You can instead skip this commit with "git cherry-pick --skip".
hint: To abort and get back to the state before "git cherry-pick",
hint: run "git cherry-pick --abort".
----------
status:

Either perform the forward-port manually (and push to this branch, proceeding as usual) or close this PR (maybe?).

In the former case, you may want to edit this PR message as well.

⚠️ after resolving this conflict, you will need to merge it via @robodoo.

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

Steps to reproduce:

- wrap a pivot function inside a IFERROR e.g. =IFERROR(PIVOT("1", "probability"), 42)
- reload the spreadsheet
- before the pivot is loaded (throttle the network in the dev tools): right click the cell
- click on "See records" menu item
=> boom

Task: 3847477
X-original-commit: 7351f42
@LucasLefevre LucasLefevre force-pushed the saas-17.1-16.0-15.0-see-record-shadowed-error-lul-DAVs-fw-bis-iQkN-fw branch from 96df9d0 to 7fa72ec Compare April 21, 2024 11:23
@C3POdoo C3POdoo added the RD research & development, internal work label Apr 21, 2024
@LucasLefevre
Copy link
Contributor

robodoo r+

robodoo pushed a commit that referenced this pull request Apr 21, 2024
Steps to reproduce:

- wrap a pivot function inside a IFERROR e.g. =IFERROR(PIVOT("1", "probability"), 42)
- reload the spreadsheet
- before the pivot is loaded (throttle the network in the dev tools): right click the cell
- click on "See records" menu item
=> boom

closes #162761

Task: 3847477
X-original-commit: 7351f42
Signed-off-by: Lucas Lefèvre (lul) <lul@odoo.com>
@robodoo robodoo closed this Apr 21, 2024
@fw-bot fw-bot deleted the saas-17.1-16.0-15.0-see-record-shadowed-error-lul-DAVs-fw-bis-iQkN-fw branch May 5, 2024 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

conflict There was an error while creating this forward-port PR forwardport This PR was created by @fw-bot RD research & development, internal work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants