Fix passing info to wrapping Story component

[danielkcz]

I am sorry. I was certain that had I had fixed it in #37, but apparently not...

[nross83]

I just ran into this same issue, currently there is no way to see the info content. This PR looks like it should fix it.

[Nickman87]

I made a somewhat cleaner fix with consistent naming: #45

[roonyh]

@FredyC @Nickman87 Thanks! Even with this fix, looks like if options are specified without info they will be ignored. Like, addWithInfo(storyName, storyFn, options). If possible could you make a fix for that as well?

[danielkcz]

@roonyh Makes sense, should be fairly easy to fix.

@Nickman87 I surely like your PR better, so if you can add that fix, I would close this one. Should be possible just by adding options = _storyFn on line 25.

[Nickman87]

I added the swap for the options as well, but I think this is a very dirty way of doing things...
Why not just pass '' as the info when you don't want info? Behind the scenes the info gets changed to empty string anyway and it's not that much of a hassle to type it? :)

Now you make the interface very confusing as you use the same method with different sets of parameters and it 'just works'.
Just my two cents, fixing this is first priority.

[danielkcz]

You are partially right, but it makes harder to switch between regular add and this one if you leave there '' parameter. Not even mentioning it feels very strange to be passing empty string there within the story as that argument is not named.

[danielkcz]

Closed in favor of #45

[roonyh]

Agree with both of you. @FredyC we are working on an api for addons that works better with regular add and with other addons :)