1. 17 May, 2017 4 commits
  2. 16 May, 2017 12 commits
    • Sean Roberts's avatar
    • Sean Roberts's avatar
      Cleaning up: moving constructor to be first function in Guest class and... · 8419426e
      Sean Roberts authored
      Cleaning up: moving constructor to be first function in Guest class and removing unused wrapper html field
      8419426e
    • Sean Roberts's avatar
      Merge pull request #380 from evidentpoint/remove-annotator-dep · 4efa180a
      Sean Roberts authored
      Remove dependency Annotator.js
      4efa180a
    • Sean Hammond's avatar
      Merge pull request #359 from hypothesis/yarn · 5e284a96
      Sean Hammond authored
      Switch package management from npm to yarn
      5e284a96
    • Sean Hammond's avatar
      Merge pull request #384 from hypothesis/fix-refresh-access-token-when-laptop-suspends · 13bd6744
      Sean Hammond authored
      Fix refreshing access tokens when laptop suspends
      13bd6744
    • Sean Hammond's avatar
      Fix refreshing access tokens when laptop suspends · 361d3381
      Sean Hammond authored
      This commit fixes an issue that refresh token requests were being sent
      too late if a laptop was suspended, even if the laptop was resumed again
      in time to do the refresh.
      
      For example if a laptop was suspended for 10 mins, then the refresh
      request would be sent 10 mins later than it should have been and the
      access token and refresh token would have already expired before the
      refresh request was attempted.
      
      The bug was due to a misunderstanding about how `window.setTimeout()`
      works:
      
      If you, say, use `setTimeout` to call a function in 10 minutes time,
      then wait 5 minutes, then put the laptop to sleep for an hour, then
      wake it up, **it will wait another 5 minutes and then call the
      function**.
      
      That is, the time that the laptop spent asleep does not count towards
      the timeout, it pauses the timer and then continues the timer again when
      the laptop wakes. This will also happen if the browser or OS suspends
      the tab or app.
      
      I've verified this behaviour in Chrome and Firefox on Linux, and it also
      agrees with [this StackOverflow
      answer](http://stackoverflow.com/questions/6346849/what-happens-to-settimeout-when-the-computer-goes-to-sleep/6347336#6347336)
      and [the HTML5 spec on
      `setTimeout`](https://www.w3.org/TR/2010/WD-html5-20101019/timers.html#timers):
      "wait until the Document associated with the method context has been
      fully active for a further timeout milliseconds (not necessarily
      consecutively)."
      
      This means that if you want to definitely call a function at a given
      time (say 55 mins in the future) you can't just use `setTimeout` to set
      a single timer for 55 mins like our code was doing - if the laptop is
      suspended during those 55 mins then the timer won't go off until 55 mins
      + however long the laptop was suspended for.
      
      To fix this change the code to repeatedly poll, every few seconds,
      whether the current time is later than the access token expiry time
      minus 5 minutes. If it is, then try to refresh the access token.
      
      I'm using repeated calls to `window.setTimeout()` to implement this,
      rather than one call to `window.setInterval()`, because with
      `setTimeout()` it's easier to do a couple of things that the previous
      implementation was already doing:
      
      1. Not make any more refresh token requests while one is still in flight
      2. Stop making further refresh token requests when one fails
      361d3381
    • Robert Knight's avatar
      Update yarn lockfile · d8f60dc3
      Robert Knight authored
      Update lockfile with result of running `yarn install` with yarn v0.23.2.
      d8f60dc3
    • Robert Knight's avatar
      e3e7760e
    • Robert Knight's avatar
      Update to latest Node v6.x release · ea3cf49f
      Robert Knight authored
      Package installation fails with Node v6.2 due to an incompatible
      "check-dependencies" package. See
      https://travis-ci.org/hypothesis/client/jobs/228937886
      ea3cf49f
    • Robert Knight's avatar
      Re-create Yarn lockfile from scratch · ea4c57e4
      Robert Knight authored
      There were a couple of issues with the initial Yarn lockfile created via
      `yarn import`:
      
      1. `yarn check` reported several inconsistencies.
      
      2. `phantomjs-prebuilt` needed to be upgraded to resolve a compatibility
         issue with Yarn.
      
      This commit resolves these issues by just removing and recreating the
      Yarn lockfile. In the process all dependencies have been upgraded to the
      latest versions compatible with the constraints in package.json.
      ea4c57e4
    • Robert Knight's avatar
      Switch from npm to yarn for package management · e8f08d46
      Robert Knight authored
      Yarn provides much faster package installation, better resilience to npm
      registry connection issues and most importantly for us, a much better
      lockfile format.
      
      This commit:
      
      1. Replaces the npm shrinkwrap with the yarn lockfile using the
         following steps:
      
         1. Checkout latest version of master
      
         2. Remove node_modules folder and re-run `npm install`
      
         3. Run `yarn import` to generate a lockfile
      
      2. Modifies the Makefile, Jenkinsfile, package.json and .travis.yml
         scripts to run `yarn` instead of `npm`.
      e8f08d46
    • Robert Knight's avatar
      1.16.0 · 7d041803
      Robert Knight authored
      7d041803
  3. 15 May, 2017 3 commits
  4. 12 May, 2017 7 commits
  5. 10 May, 2017 1 commit
    • Juan Corona's avatar
      Fix the scoping of the initial value for the `plugins` property · acddfb66
      Juan Corona authored
      This fixes the remaining failing tests.
      When compiled from coffeescript the initial `{}` value was at the prototype level instead of scoped at the function level.
      It looked like I was declaring it as an instance variable, but it was more like a class variable.. coffeescript is weird to me..
      acddfb66
  6. 09 May, 2017 8 commits
  7. 04 May, 2017 2 commits
  8. 27 Apr, 2017 2 commits
    • Sean Hammond's avatar
      f2f5775f
    • Sean Hammond's avatar
      Improve error when flagging when logged out · cd22ae61
      Sean Hammond authored
      Improve the error message that's shown to the user when trying to flag
      an annotation while logged out.
      
      Instead of showing a "404 Not Found. Either the resource you requested
      doesn't exist, or you are not currently authorized to see it." error
      from the server, show a friendlier "You must be logged in to report an
      annotation" message.
      
      This is done client-side by checking whether the user is logged in when
      they click the flag button, and if not showing an error instead of
      sending the flag request to the API. This is because the API doesn't
      respond with a unique "You must be logged in to flag" error that the
      client could depend on, it just returns a 404, which could be for a
      number of reasons (e.g. the annotation no longer exists).
      cd22ae61
  9. 26 Apr, 2017 1 commit
    • Sean Hammond's avatar
      Clarify in tooltip what flag button does · e64fa192
      Sean Hammond authored
      The flag button on an annotation has a tooltip that says simply "Flag"
      (or "Annotation has been flagged" if it's already flagged) but there's
      no indication to the user what flagging is or does (even when they click
      the button, all that happens is the flag turns red).
      
      "Flagging" can certainly be misinterpreted. I think some apps use it as
      a way to keep track of "special" or "favorite" items (usually called
      "starring" in most modern apps), or have a variety of different "flags"
      that you can apply to an email or whatever it is (important, work,
      personal, etc, this is usually called "tagging" or "labelling" in most
      modern apps).
      
      This commit changes the tooltips to clarify that what the button does is
      report the annotation to the moderators. I've used the verb "report"
      rather than "flag" as I think "Report this annotation to the moderators"
      sounds better than "Flag this annotation to the moderators".  Either way
      I think "to the moderators" is important to clarify that this button
      sends a message to the moderators and isn't, for example, just flagging
      for personal use like starring / tagging / labeling.
      e64fa192