Closed blackary closed 1 year ago
Seems worth it for usability.
Not sure if it would be overkill to allow some event handler functions and use the current ones as defaults. Ex:
import streamlit as st
from st_stripe import require_auth, show_login_button, render_subscribe
require_auth(
on_anonymous = show_login_button,
on_unsubscribed = render_subscribe,
on_subscribe = st.balloons)
closing this with the PR merge!
And then make a POC demo app that uses the package. This seems like a really helpful way to see how it feels for the end-user, and whether the public API makes sense.