Looks like we need account context after all...

bushwallyta271828 / logic_graph_scalable

This repository should support the logicgraph.dev website, taking over from logic_graph.

MIT License

0 stars 0 forks source link

Looks like we need account context after all... #178

Closed bushwallyta271828 closed 5 months ago

bushwallyta271828 commented 5 months ago

TODO:

Make it compile
Test everything!
Write down / formalize how I'm managing account state, cookies, and redirects/refreshes in the Issue.

bushwallyta271828 commented 5 months ago

Okay, I think I could have ManageAccount pass in an "after submitting" function to AccountForm as a parameter. That "after submitting" function could update the displayed username.

bushwallyta271828 commented 5 months ago

Better idea: Inside AccountForm, let's check if the form has a "username" field, and if it does then let's setAccount using that. This modification should reduce code duplication across everything except the update password form.

bushwallyta271828 commented 5 months ago

TODO:

Display success message on password change (since the user currently doesn't get any confirmation).
Make it compile
Test everything!
Write down / formalize how I'm managing account state, cookies, and redirects/refreshes in the Issue.

bushwallyta271828 commented 5 months ago

I think it's actually working!

TODO for future branches:

Display success message on password change (since the user currently doesn't get any confirmation).
Write down / formalize how I'm managing account state, cookies, and redirects/refreshes in the Issue.