streamlit-extras icon indicating copy to clipboard operation
streamlit-extras copied to clipboard

✨ [IDEA] - Streamlit Query Strings

Open Asaurus1 opened this issue 1 year ago • 6 comments

Description

Hi!

I am working on a small Streamlit add-on over at https://github.com/Asaurus1/streamlit-qs to make integrating data from the URL query string easier. At its core, this relies on the experimental experiemental_get_query_string and experimental_set_query_string API's. I'm wondering whether streamlit-extras would be open to including this library once it is a little more polished, or if your preference would be to wait for the streamlit APIs to get out of their experimental stage first.

Example: https://query-string.streamlit.app/

Cheers, A

Asaurus1 avatar Jun 05 '23 08:06 Asaurus1

Hey @Asaurus1! That is a great library, thanks a lot for sharing it here. Make sure to share it also in Streamlit's forum at discuss.streamlit.io - makes me think of one blog post our data team posted some time ago. Definitely keen on adding it here to streamlit-extras, let me know if you want to do it and throw a PR.

Best,

arnaudmiribel avatar Jun 05 '23 08:06 arnaudmiribel

@arnaudmiribel I'd be happy to throw it on a PR once I've made a few more tweaks. It absolutely does look like what's on that blog-post, just packaged a bit more cleanly to control direct interactions with the query string.

You'll have to forgive me, I am very familiar with git but only within a corporate context and not so much with OSS and GitHub. Should I create a branch in this project for the PR, or fork the entire project and create a PR from that?

Asaurus1 avatar Jun 05 '23 08:06 Asaurus1

It absolutely does look like what's on that blog-post, just packaged a bit more cleanly to control direct interactions with the query string.

Exactly! That's what I like about it 💯

Should I create a branch in this project for the PR, or fork the entire project and create a PR from that?

Best is to fork and create a PR from that!

arnaudmiribel avatar Jun 05 '23 08:06 arnaudmiribel

Great! And it seems like I would want to add the project like the annotated_text one, where there's a thin wrapper around the imported functions and some examples, correct?

If so, I will need to publish this to pypi first (which I was going to do anyways), so I will get back to you here in a bit when that's ready. Thanks for the quick responses and help!

Asaurus1 avatar Jun 05 '23 09:06 Asaurus1

You can follow our contribution guidelines here to get things properly set up, hope that helps!

I would want to add the project like the annotated_text one

Exactly, the annotated_text is a good example to follow where the source code lives somewhere else (in your repo, and published in a standalone PyPI package) instead of pasting the code within streamlit-extras. Great thing about your package is that there's only streamlit as a dependency, so that's great!

arnaudmiribel avatar Jun 05 '23 09:06 arnaudmiribel

Alright the PR is ready

Asaurus1 avatar Jun 07 '23 18:06 Asaurus1