Add an option to the addItemToToolbar config action to skip adding if the item already exists

Added a test - that was easy! This is good to go.

From a UX perspective, I have a doubt here. I agree with @thejimbirch about this:

in most cases, you will only want a single instance.

In light of that, maybe it would make more sense to invert the logic here and to make skip-if-exists the default behavior, with an allow_duplicate: true flag to override that.

Thoughts?

I tested and it works as is, but I do feel like this would be the more normal behavior:

skip-if-exists the default behavior, with an allow_duplicate: true flag to override

Done.

That works also as expected. This is much better behavior to not duplicate by default.

Committed to 11.x - thanks!

Could this be backported to 10.4.x? Recipes are still experimental, and this would be quite helpful to Starshot.

Backported to 11.1.x, 10.5.x and 10.4.x

Automatically closed - issue fixed for 2 weeks with no activity.