- 14 Sep, 2018 3 commits
-
-
Robert Knight authored
Add "requestConfigFromFrame" string setting
-
Robert Knight authored
Replace "var" with "const" or "let" everywhere
-
Robert Knight authored
This setting will cause the sidebar app to fetch its configuration via a postMessage request to frames with the given origin on startup. The setting is not actually used yet. This is coming in future commits.
-
- 13 Sep, 2018 5 commits
-
-
Robert Knight authored
To ensure consistency going forwards, enforce usage of const/let
-
Robert Knight authored
Fix the uses which js-codemod's "no vars" transform was unable to change automatically.
-
Robert Knight authored
This is the result of using jscodeshift [1] to apply the "no-vars" transform from js-codemod [2] on all code in scripts/ and to gulpfile.js [1] https://github.com/facebook/jscodeshift [2] https://github.com/cpojer/js-codemod
-
Robert Knight authored
-
Robert Knight authored
This is the result of using jscodeshift [1] to apply the "no-vars" transform from js-codemod [2] on all code in src/ except for src/{annotator, sidebar}/vendor. [1] https://github.com/facebook/jscodeshift [2] https://github.com/cpojer/js-codemod
-
- 04 Sep, 2018 1 commit
-
-
Sean Hammond authored
Add documentation for externalContainerSelector
-
- 30 Aug, 2018 1 commit
-
-
Robert Knight authored
Update to Node 10 for Jenkins and Travis builds
-
- 22 Aug, 2018 5 commits
-
-
Robert Knight authored
-
Robert Knight authored
The official Docker node builds now have Yarn preinstalled, so use that instead of a community-provided image.
-
Robert Knight authored
Use `yarn version --minor` to bump the package version. This requires Yarn >= v1.7.0.
-
Robert Knight authored
-
Robert Knight authored
-
- 21 Aug, 2018 1 commit
-
-
Sean Hammond authored
Inject a default organization into groups objects without organizations
-
- 15 Aug, 2018 2 commits
-
-
Lyza Danger Gardner authored
-
Hannah Stepanek authored
Create annotation when clicking "Annotate" button label
-
- 10 Aug, 2018 11 commits
-
-
Robert Knight authored
-
Robert Knight authored
-
Robert Knight authored
Hide "Public" group for unauthenticated users (alternate implementation)
-
Robert Knight authored
-
Robert Knight authored
-
Robert Knight authored
Implement a business requirement that the "Public" group should be hidden from the groups list if all these conditions are met: - The user is logged-out - The current document has groups associated with it - The user has not visited a direct-link to an annotation in "Public" --- This requirement introduces a complexity in that deciding which group to focus and fetch annotations for after initially fetching groups now depends on the current profile. However the profile and groups are fetched concurrently to reduce the number of API round-trips before we can show annotations in the client. Fortunately we don't need to know the exact userid to know which group to focus, only whether the list of groups was fetched as a logged-in user or not. This can be determined from the access token that was sent with the `/api/groups` request.
-
Sheetal Umesh Kumar authored
Remove obsolete JSDoc comments
-
Robert Knight authored
Add an option to return metadata in the response as well as just the JSON response from the server. This will initially be used to determine the access token that was used to make the request and therefore whether it was authenticated or not. In future it could also return information from HTTP headers or the status code if useful.
-
Robert Knight authored
-
Robert Knight authored
Remove unnecessary extra `/api/groups` call on startup
-
Robert Knight authored
Respond to PR feedback.
-
- 08 Aug, 2018 4 commits
-
-
Robert Knight authored
-
Robert Knight authored
This event is never broadcast anywhere. A comment referenced this being called when a user leaves a group, but this is handled in the `streamer` service by calling `groups.load()` directly.
-
Robert Knight authored
On startup an initial `FRAME_CONNECTED` event is fired when the main frame connects to the sidebar. This triggered an extra `/api/groups` fetch. Avoid this by recording the main frame URI used in the last groups fetch in `load()`. The existing test in the `FRAME_CONNECTED` handler then skips the `load()` call if this URI has not changed.
-
Robert Knight authored
The group service does not have `add` or `remove` methods any more.
-
- 02 Aug, 2018 7 commits
-
-
Robert Knight authored
-
Robert Knight authored
-
Robert Knight authored
Fix `groups.focused()` error if localStorage is empty when client loads
-
Robert Knight authored
-
Robert Knight authored
-
Robert Knight authored
Use a pointer cursor for highlights
-
Robert Knight authored
If the client started and local storage did not contain a saved last-viewed group ID, then `groups.load()` would attempt to call `store.focusGroup(null)`. As a result subsequent calls to `groups.focused()` would return null and code in various places in the application expects that there will always be a focused group. Fix the problem by not setting the focus group after groups are fetched unless the previously-focused group was set and exists in the new groups list. Also improve the behavior of `store.focusGroup` by making it leave the focused group untouched if the specified group has not been loaded, but log an error so we'll know (via Sentry) if it happens again. Fixes #750
-