444B / streamlit-analytics2

👀 Track & visualize user interactions with your streamlit app
MIT License
14 stars 2 forks source link

Bump streamlit to 1.30 #2

Closed 444B closed 4 months ago

444B commented 4 months ago

Hi, I am making a PR as discussed. I acknowledge that there are multiple changes here and can split this into 4 different PRs if you prefer. Also AFAIK, this wont yet make any changes to downstream pip but I rely on you to let me know how that works since I have less experience with creating pypi packages. Please find below a summary of changes, all tested and verified.

Streamlit -> 1.30

All adjustments were made with streamlit 1.30 in the requirements.txt

fixed st.query_params

st.experimental_get_query_params() was replaced with st.query_params. I made sure to test the changes first on my local env files before making an edit to the /streamlit_analytics/main.py file and both work fine, with no more deprecation notice. Screenshot below [1]

added .github/contributing.md

A CONTRIBUTING.md was added to detail the following 1) How to contribute to the project 2) How to set up a test environment to work on any contributions This was the methodology I followed and if it could be improved or has flaws, please share

multipage example/

To facilitate easier testing of features, a sidebar with a multipage selection was added

software version printing

To ensure collaborators are not unaligned with the software versions they are using in testing, a few lines were added to the /example/minimal.py page to print the software version

Please let me know your thoughts I could not find any errors when using all features and look forward to your feedback

1