- 08 Nov, 2018 1 commit
-
-
Sean Hammond authored
Convert viewFilter tests to JS
-
- 07 Nov, 2018 4 commits
-
-
jenkins-hypothesis authored
-
Robert Knight authored
Remove support for real-time updates in the /stream view
-
Robert Knight authored
It is not particularly clear from the naming of these modules what they are for and how they relate to things the user can do in the application. Add some comments to clarify this.
-
Robert Knight authored
Convert the tests for the `viewFilter` class to JS. This is mostly a straightforward conversion except that `viewFilter.fields` has been made a private variable and the direct tests for ithave been removed, since the contents of this field were an implementation detail of the class.
-
- 06 Nov, 2018 1 commit
-
-
jenkins-hypothesis authored
-
- 05 Nov, 2018 1 commit
-
-
Hannah Stepanek authored
Adjust spinner positioning and convert to component
-
- 02 Nov, 2018 10 commits
-
-
jenkins-hypothesis authored
-
Robert Knight authored
Set a useful title on the sidebar iframe
-
Robert Knight authored
This was untested code, some of the last CoffeeScript in the sidebar app, that supports a feature which gets little use. The stream itself will still be available at https://hypothes.is/stream but visitors will need to refresh the page to see updates.
-
jenkins-hypothesis authored
-
Robert Knight authored
This went unnoticed because it turns out that `yarn version` does not actually run the postversion hook if `--no-git-tag-version` is specified. See https://github.com/yarnpkg/yarn/issues/5358
-
Robert Knight authored
Deploy QA release of client as part of master branch builds
-
Robert Knight authored
This makes it easier to identify the purpose of the frame when navigating the page with assistive technology. Fixes https://github.com/hypothesis/product-backlog/issues/784
-
Robert Knight authored
- Remove a semantically incorrect `<button>` wrapping the loading spinner and a redundant wrapping `<span>`. - Vertically center the spinner by adding a wrapper `<div>` in the spinner component itself, so that there is less of a visual shift when switching between displaying the loading indicator and the search bar. - Add a meaningful title attribute to the spinner. Properly indicating when the sidebar's content is changing/being loaded to ATs requires additional work which is not tackled here.
-
Robert Knight authored
This is the last remaining directive which should be a component.
-
Robert Knight authored
The short-lived max-age value set previously was overridden by the higher value of 1800 (30 mins) set in CloudFlare's "Browser Cache TTL" setting under Page Rules for cdn.hypothes.is. Resolve this by simply disabling caching of the entry point for qa deployments. This means that https://cdn.hypothes.is/hypothesis@qa will always return the boot script for the latest client QA release.
-
- 01 Nov, 2018 2 commits
-
-
Robert Knight authored
Add a new build stage to Jenkins CI builds of the master branch which deploys a "QA" build to S3. This QA build: - Has a pre-release version number of the form `X.Y.Z-{commit hash}` - Has a stable URL at https://cdn.hypothes.is/hypothesis@qa (as opposed to https://cdn.hypothes.is/hypothesis for the prod build) - Loads the sidebar app from qa-h (https://qa.hypothes.is/app.html) - Does not update the changelog or create a Git tag or GitHub release This build is currently not deployed to npm at all, but we might need to do that in future if we were to create a QA deployment of the browser extension with it. --- - Add a QA deploy step to the Jenkins build. - Skip GitHub release creation and changelog updates for pre-release versions used for QA deployments. - Remove the preversion.sh script, which is no longer needed since the builds always run in Jenkins and not on a developer's system. - Support specifying the npm tag to check in `wait-for-npm-release.sh`, though this is not used in the final version of this commit.
-
Robert Knight authored
Replace the generic Docker-based s3-npm-publish tool with a JS script in the client repo which is tailored more specifically to the needs of client deployments: - Avoid creating the `hypothesis@X.Y` and `hypothesis@X` aliases which we never used. - Support uploading a copy of the entry point under a version-indepenent alias, regardless of whether this is a pre-release or not. This will enable creating a stable URL which points to the current QA version of the client. - Deploy the package directly from the current working directory. This saves the need to wait for publication to npm to complete, or the need to actually publish the package to npm at all in the case of the QA release.
-
- 31 Oct, 2018 4 commits
-
-
jenkins-hypothesis authored
-
Robert Knight authored
Add service.groups setting to filter groups
-
Robert Knight authored
This option is being added initially to support the LMS application where it is important that students annotate in the correct group for the course that the active assignment belongs to.
-
Hannah Stepanek authored
-
- 25 Oct, 2018 3 commits
-
-
jenkins-hypothesis authored
-
Robert Knight authored
Wait for npm release to complete before deploying to CDN
-
Robert Knight authored
Use build milestones to automatically abort old builds
-
- 23 Oct, 2018 7 commits
-
-
Robert Knight authored
npm packages are not immediately available after "npm publish" returns. Wait until the package has been released before attempting to deploy it to the CDN. This should fix the failure seen in https://jenkins.hypothes.is/job/client/job/master/745/console.
-
Robert Knight authored
Use milestones to automatically abort older builds when a newer build reaches the same step of the pipeline. This mirrors the use of milestones in h's Jenkins pipeline.
-
jenkins-hypothesis authored
-
Robert Knight authored
Add missing polyfill for IE 11 compatibility when annotating PDFs
-
Robert Knight authored
This package was already depended upon indirectly but following the previous commit is now a direct dependency of the client.
-
Robert Knight authored
The `seek` function from the `dom-seek` package expects its `NodeIterator` argument to have a `referenceNode` property, which is missing in IE 11. Use the dom-node-iterator package recommended by the dom-seek README [1] to polyfill optional-ness of arguments to `document.createNodeIterator` and the missing properties on the returned `NodeIterator` object. dom-node-iterator provides several ways to use the polyfill. This commit uses the method that prefers the browser's native implementation if possible and does not pollute the global scope, which could affect JS code outside of the Hypothesis client. There is one other place where we use `createNodeIterator` but in that context the missing properties are not used so the polyfill is not needed. Fixes #762 [1] https://github.com/tilgovi/dom-seek/blob/master/README.md
-
Robert Knight authored
Fix "Log in" links in sidebar content area not opening login prompt.
-
- 22 Oct, 2018 5 commits
-
-
Hannah Stepanek authored
The loggedout-message controller should bind onLogin. The sidebar-content controller should also bind onLogin and call vm.login(). Prevously this behavior was broken.
-
jenkins-hypothesis authored
-
Robert Knight authored
Fix IE 11 JS error which annotating PDFs.
-
Robert Knight authored
Skip builds of automated commits on Jenkins
-
Sheetal Umesh Kumar authored
The `whatToShow`, `filter` and `expandEntityReferences` arguments for `Document.createNodeIterator()` are mandatory in IE although optional according to the spec. Pass default values to fix JS console errors. See: https://github.com/hypothesis/client/issues/762
-
- 19 Oct, 2018 2 commits
-
-
Robert Knight authored
As part of the deployment process Jenkins creates a new commit on master in order to bump the npm package version. There is no point in creating an automated build of these commits, which can be identified by the commiter name.
-
jenkins-hypothesis authored
-