fix(docker): missing browse volume option [EE-7179]

[oscarzhou-portainer]

closes EE-7179

[chiptus]

I don't understand why your fix is changing anything. why is AccessDatatable related?

[oscarzhou-portainer]

I don't understand why your fix is changing anything. why is AccessDatatable related?

Because I noticed that the codebase for extendTableOption settings is not consistent throughout the project. Even in cases where there are "single extendTableOptions", you also have wrapped them with mergeOptions. An example can be found here 1261887#diff-fc05a87bac4bd4de2239bd6026a578ea922c7e8a7db63751d9dd00bc92933344R97-R102. Therefore, I believe it would be beneficial to add this wrapper to the remaining extendTableOption settings as well. Besides, it is intuitive for others to work on extendTableOption setting in the future.

[chiptus]

@oscarzhou-portainer I don't see the added value. where we have it's because it used to be multiple values. but how does it fixes the issue?

[oscarzhou-portainer]

@oscarzhou-portainer I don't see the added value. where we have it's because it used to be multiple values. but how does it fixes the issue?

The main fix for the issue can be found https://github.com/portainer/portainer/pull/11901/files#diff-ee4ca3e2b0441ac5ba3be0fa63c816a8f7df3597efba58805fdeea6086cac77dL72-R74. The problem was that two extend table options were added to the VolumesDatatable, but they were not wrapped with the mergeOptions function.

The reason why I suggest including mergeOptions in all extenTableOptions settings is that it makes it easier for other developers to grasp how to incorporate multiple values into extendTableOptions and reduces the likelihood of errors.

In this code snippet, you can see that the mergeOptions function is used even when there are no multiple values. You can find this example here: link. Another example can be found here: link. There are more instances of this usage in your original PR.

[chiptus]

I guess we could change extendTableOptions to accept an array of extenders.

LGTM

[chiptus]

@oscarzhou-portainer you need to run yarn format on this pr