search

OCA / spreadsheet

GNU Affero General Public License v3.0
46 stars 47 forks source link

[16.0][IMP] spreadsheet_oca: Allow to edit Odoo graphs properly #15

Closed etobella closed 1 year ago

etobella commented 1 year ago

Fixes #9

etobella commented 1 year ago

@sami96h can you explaine how did you obtain the error from https://github.com/tegin/spreadsheet/commit/08b297b95b40735f2ef042e3d4c060830404f737#commitcomment-121196974 ? I am unable to reproduce it locally

sami96h commented 1 year ago

@etobella

I added a pie graph from the sales order view, It is added successfully to the sheet but when i click on the burger button on the graph in order to edit it ,it raises the error .

sami96h commented 1 year ago

Also there is another issue,in the filters when i try to add relation it raises an error

etobella commented 1 year ago

@sami96h I am unable to reproduce the issue on runboat. Editing the filters work too. did you try it in runboat?

etobella commented 1 year ago

/ocabot merge patch

OCA-git-bot commented 1 year ago

This PR looks fantastic, let's merge it! Prepared branch 16.0-ocabot-merge-pr-15-by-etobella-bump-patch, awaiting test results.

OCA-git-bot commented 1 year ago

Congratulations, your PR was merged at 744e250b9847a72de06e882762322c7c22f256a2. Thanks a lot for contributing to OCA. ❤️