Update running-idom.rst

[jmtaysom]

The run function does not have a config kwarg but instead server_config

[rmorshea]

Thanks @jmtaysom for the contribution!

A note on idom.run, we're thinking about making some changes to the run function since it encourages bad habits - namely that you should use a production WSGI or ASGI web server instead of the idom.run function. Given this, in the future, idom.run will become a tool for convenience while developing rather than something that should be used in production.

[jmtaysom]

I saw some of the chatter in the issues about that. I am definitely all for that. I have also had issues with the shared runtime in the tutorial. So that may be a section to clean up in the future if it doesn't get cleaned up automatically by moving away from the run command.

…

On Wed, Mar 16, 2022, 8:36 PM Ryan Morshead ***@***.***> wrote: Thanks @jmtaysom <https://github.com/jmtaysom>, a note on this, we're thinking about making some changes <#657> to the run function since it encourages some bad habits - namely that you should use a production WSGI <https://wsgi.readthedocs.io/en/latest/servers.html> or ASGI <https://asgi.readthedocs.io/en/latest/implementations.html> web server instead of the run function. In the future idom.run will become a tool for convenience while developing rather than something that should be used in production. — Reply to this email directly, view it on GitHub <#710 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AB337P47JSTFAGIZYVI37Q3VAJ47PANCNFSM5Q5NKOMQ> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. You are receiving this because you were mentioned.Message ID: ***@***.***>

[rmorshea]

I have also had issues with the shared runtime in the tutorial

Can you link to particular sections of the docs that you found to be problematic? I'll create an issue to track that.

[jmtaysom]

https://idom-docs.herokuapp.com/docs/guides/getting-started/running-idom.html?highlight=server%20name#available-server-types When I ran the above section about SharedClientStateServer I get the following error sanic.exceptions.SanicException: Sanic instance named 'idom.server.sanic[1]' uses an invalid format. Names must begin with a character and may only contain alphanumeric characters, _, or -. .I ran the same section with an explicitly created and named sanic app and passed that app into the run function which let it work but I still couldnt get two windows to update each other as I moved the slider

…

On Wed, Mar 16, 2022 at 8:40 PM Ryan Morshead ***@***.***> wrote: I have also had issues with the shared runtime in the tutorial Can you link to particular sections of the docs that you found to be problematic? I'll create an issue to track that. — Reply to this email directly, view it on GitHub <#710 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AB337PZFD2YDVO3AV5H4WWDVAJ5QZANCNFSM5Q5NKOMQ> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. You are receiving this because you were mentioned.Message ID: ***@***.***>

[rmorshea]

It looks like there may be two separate problems here. I'll open a discussion so we can tease them out.