-
Notifications
You must be signed in to change notification settings - Fork 54
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix query param deprecation and bump streamlit to 1.30 #41
Conversation
changed st.experimental_get_query_params to st.query_params - reason (st.experimental_get_query_params deprecation) https://docs.streamlit.io/library/api-reference/utilities/st.experimental_get_query_params
bumping streamlit version from 1.13 to 1.30 - requires testing for breaking changes
Update main.py
Update Pipfile.lock
@mt-krainski visibility |
@jrieke Are you available to take a look at this? |
Hi, |
@RazHadas In the meantime, you can maybe specify in your requirements.txt or pipfile or whichever dependency file you have to use a streamlit version that is prior to 1.30 |
Thanks for the quick response. I will check it. |
I messaged the author on Linkedin but no response so it would seem this project is not maintained... @jrieke visibility |
The analytics is streamlit is really problematic. I already try for few days to get unique page views. |
Hi @RazHadas |
Sounds good. I just find script that get ip and screen resolution from user in streamlit, I want to improve the analytics. |
Note: I am pushing this to my own fork but it seems be to included in this PR @RazHadas The GA4 integration is something I want to work on in the future |
Changes:
changed st.experimental_get_query_params to st.query_params
https://docs.streamlit.io/library/api-reference/utilities/st.experimental_get_query_params
bumping streamlit version from 1.13 to 1.30
This should resolve #39