thoughtbot / carnival

An unobtrusive, developer-friendly way to add comments
MIT License
499 stars 30 forks source link

Fetch user data asynchronously #260

Closed pbrisbin closed 9 years ago

pbrisbin commented 9 years ago

Resolves #170, #248

jferris commented 9 years ago

Do we need to fire a request right when the script loads? That will increase the load on the server pretty dramatically, as most requests won't involve making comments.

pbrisbin commented 9 years ago

Do you think moving it to when/if the sidebar is opened will help enough (assuming it works)? That would still be more than is strictly required.

pbrisbin commented 9 years ago

Seems to still work moving it there. I could get close, but never quick enough to trigger an unnecessary login.

jferris commented 9 years ago

Looks good to me.