-
Notifications
You must be signed in to change notification settings - Fork 74
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
Adjust Like & Comment such that they funnel logged-out users to create an account #992
Comments
FYI in case it's helpful - number of likes/comments for NEAR social daily is low hundreds. I'd assume Blog likes/comments would be some subset of that From Dash |
@charleslavon this issue was closed by near/near-discovery-components#673. However I think the changes made to use the modal is impacting the desired experience. Here is a quick video for reference: https://www.loom.com/share/8996f5ff9d6c48d8b2e4e04712a7fef0?sid=bc2e8978-8739-4f2f-8d1b-8a5603aedb08 thoughts on how to proceed here? |
hey @jackson-harris-iii Let's trigger the new signin/signup modal, to allow users to authenticate with the modal/wallet and get redirected back to this page (instead of going to the applications page). You may want to pass the a reference to this hook to the like/comment buttons. |
The blogPostPage is a bos component, so I currently don't have access to the hooks used in the near discovery repo. 🤔 |
@jackson-harris-iii is that a blocker? can the required functions be passed into the bos components from the gateway? |
The text was updated successfully, but these errors were encountered: