Story sorting

[snowystinger]

Issue: #6439

Story sorting now takes place in story_store. This is so that the Object.values returns the correct order and sorting is not needed elsewhere.

What I did

Added a Param to the global params called storySort which takes a sort function. If none is provided then the default alphabetical sort is used.

How to test

• Is this testable with Jest or Chromatic screenshots? don't know yet
• Does this need a new example in the kitchen sink apps? Is showcased in react kitchen sink
• Does this need an update to the documentation? yes

If your answer is yes to any of these, please make sure to include it in your PR.

[vercel]

This pull request is automatically deployed with Now.
To access deployments, click Details below or on the icon next to each push.

Latest deployment for this branch: https://monorepo-git-fork-snowystinger-feature-story-sorting.storybook.now.sh

[shilman]

@snowystinger Thanks for taking a go at this. I think the issue here is that config.js is running in the "preview" (user code) and it serializes the options over the channel to the "manager" (storybook UI code) and the function is not serializable.

I think @ndelangen is trying to address this with a unified config that makes configuring the preview and the manager easier.

[snowystinger]

@shilman I thought that might be an issue, but got tripped up because the sorting function in config.js is getting called. It's just not returning anything. Where is @ndelangen working on this? I'd like to follow along if possible.

[ndelangen]

@snowystinger I could brief you next week if you'd like.

I could use some company, possibly some pair programming would do me some good.

Tomorrow I'll be at React Amsterdam Conference, weekend I'll be resting, or at the very least trying to rest.

[ndelangen]

What do you want to do with this PR?

[snowystinger]

@ndelangen happy to help out in any way :) and have time next week.
If this stuff happens in another PR, that's cool

One thought I did have was that I could take out the addParams bit and just auto sort alphabetically, then at least there'd be a deterministic way to deal with this until the sorting is exposed.

[Armanio]

Good job! Thank you!
I left the mini notice. Check it, please.
Could you update docs?

[snowystinger]

@Armanio updated. Should I also remove the logic for pulling it from the parameters object for now?

[Armanio]

No, I think everything is fine. 👍
Thank you, good job!
Let's wait other opinions.

[ndelangen]

Related to monoconfig