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
feat: use cash.min.js on the page instead of CDN #18
Conversation
This is live in production BTW. Making the PR for review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
@raisedadead, sorry I'm just seeing this now. Thanks for deploying such a quick fix 👍 |
Yep - I do not like it though. |
I know what you mean. I'd like to take another pass at this and just use vanilla JS. If I remember correctly, Cash is just hiding or showing some elements, and maybe fetching something, so it should be possible. |
Sounds like a plan. But no rush, I know you are pretty swamped already. |
Awesome, sounds good. I'll add it to the Trello board so we have a place to track progress on this. |
The CDN is potentially having some CORS errors. Using the script on the auth page instead for now.
/cc @scissorsneedfoodtoo @ojeytonwilliams