Description
Problem
A user writes a detailed review on a notebook and their browser crashes before they could post the review. Well written comments are lost. This is not the experience we want our users to have.
Solution
Draft review can be stored on our backend server or localStorage of the browser. Some considerations --
-
We do not store any pull request comments on our servers, we merely act as a passthrough to GitHub i.e. we post all PR comments to GitHub directly. So storing draft comments on our server is against our promise of
"we do not store any of your PR comments". Draft comments are only cached until the review is posted to GitHub so this might be an acceptable exception. -
localStorage of the browser is a preferable place for draft comments as it'll only store comments locally & the draft review will still be available across sessions. But this might not work in case of incognito mode or across browsers or if user clears browser history and such. More research to be done on the possible failure cases for localStorage & see if those are acceptable.