[16.0][CHG] fs_base_multi_image: drag and drop fixes

[benwillig]

• create in memory records instead of calling the ORM explicitly

[OCA-git-bot]

Hi @lmignon, some modules you are maintaining are being modified, check this out!

[lmignon]

@benwillig Why do we need to create the relation on drag and drop? Can we add a relation and save it when the parent is saved?

[benwillig]

@lmignon That's what I wanted to do, unfortunately I didn't find the right way to do it with the new framework.

[benwillig]

@lmignon I finally found the way to make it work as it was in the old widget. And it also works when saving a specific image directly on the relation record.

[lmignon]

@bealdav Can you take a look at this?

[bealdav]

Yes I 'll dive in.

[bealdav]

Thanks for this improvement even if don't understand the full details.

Nice to see Kanban view completed with some fields.

As product.product view doesn't allow to edit images, don't you think that this view could show kanban view instead of tree ? It's a better visual view for users.

[benwillig]

Yes we can display the kanban view by default. What do you think @lmignon ?

[lmignon]

Yes we can display the kanban view by default. What do you think @lmignon ?

Yes we can on the product.product view.

[bealdav]

It works nicely in prod db, really thanks

[bealdav]

Also this PR https://github.com/OCA/storage/pull/370 seems ok. What do you think ?

[OCA-git-bot]

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

[lmignon]

/ocabot merge patch

[OCA-git-bot]

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

[OCA-git-bot]

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