Bumps [jquery](https://github.com/jquery/jquery) from 3.3.1 to 3.4.0.
- [Release notes](https://github.com/jquery/jquery/releases)
- [Commits](https://github.com/jquery/jquery/compare/3.3.1...3.4.0)
Signed-off-by: dependabot[bot] <support@dependabot.com>

Fix npm dependencies make target not re-installing if yarn lockfile changed

The make target used to ensure that npm dependencies are installed when
they change depended on `yarn.lock` but didn't actually run `yarn
install` if only that file, and not package.json, changed.

The target relied on a tool called check-dependencies which pre-dates
yarn. `yarn install` is fast if nothing changed, so just run it
unconditionally.

Open the sidebar when direct-linked group

Convert '<group-list-section>' to React

Add annotations:group:GROUP_ID fragment url

- Add an api call to get the linked to group
- Add the linked to group to the featuredGroups list.
- Pass the directLinkedGroup into the filterGroups function.
- Take into account the directLinkedGroup when deciding whether to
filter out the "Public" group or not.
- If there is a directLinkedGroup focus it.

 - Rename `sectionGroups` to the slightly more obvious name `groups`
 - Add a comment

These components are no longer within an Angular component but are
used by GroupSection instead.

Fix feature flag race condition

Previously, a test that should have failed passed when uri=null, because
none of the test groups had uri patterns to check against. Make the
test groups more realistic by adding in the scopes attribute for at
least one of them.

In cases like: https://hypothes.is/a/rGSnNwCKSuiG3VT__k3VvA where the
sidebar is not being used, the document uri is set to null in the groups
service and passed to the combineGroups function. This means that the
isScopedToUri logic must handle the case where uri=null.

Since the feature flag isn't available until the profile api request
completes, and the profile api request is run asynchronously with the
group api requests, it doesn't make sense to load conditionaly. Instead,
call all the group api requests and filter the groups for the old groups
menu later. In this case, filtering is done in the groups.all
conditionally based on the feature flag.

Since the isScopedToUri is no longer dependent on scope enforcement
and scope enforcement is now taken into account as part of whether
a group is selectable or not, disableOosGroupSelection is no longer
necessary. Group selection can be determined purely from isScopedToUri
and scopes.enforced.

Previously, if the scope was not enforced, isScopedToUri would always
return true. Now, isScopedToUri is true only when the scopes.uri_patterns
match.

getInScopeGroups returns the list of groups for the old groups
menu. Aka: only groups that are in scope are returned.

Revert "Remove feature flag from group service"

Remove feature flag from group service

Convert `<svg-icon>` component to React

Merge pull request #1055 from hypothesis/dependabot/npm_and_yarn/babel-plugin-transform-async-to-promises-0.8.9

Bump babel-plugin-transform-async-to-promises from 0.8.8 to 0.8.9

Bump mocha from 6.1.1 to 6.1.2

Bump enzyme-adapter-preact-pure from 1.10.3 to 1.10.4

Bumps [babel-plugin-transform-async-to-promises](https://github.com/rpetrich/babel-plugin-transform-async-to-promises) from 0.8.8 to 0.8.9.
- [Release notes](https://github.com/rpetrich/babel-plugin-transform-async-to-promises/releases)
- [Commits](https://github.com/rpetrich/babel-plugin-transform-async-to-promises/compare/v0.8.8...v0.8.9)
Signed-off-by: dependabot[bot] <support@dependabot.com>

Bumps [mocha](https://github.com/mochajs/mocha) from 6.1.1 to 6.1.2.
- [Release notes](https://github.com/mochajs/mocha/releases)
- [Changelog](https://github.com/mochajs/mocha/blob/master/CHANGELOG.md)
- [Commits](https://github.com/mochajs/mocha/compare/v6.1.1...v6.1.2)
Signed-off-by: dependabot[bot] <support@dependabot.com>

Bumps [enzyme-adapter-preact-pure](https://github.com/robertknight/enzyme-adapter-preact-pure) from 1.10.3 to 1.10.4.
- [Release notes](https://github.com/robertknight/enzyme-adapter-preact-pure/releases)
- [Changelog](https://github.com/robertknight/enzyme-adapter-preact-pure/blob/master/CHANGELOG.md)
- [Commits](https://github.com/robertknight/enzyme-adapter-preact-pure/commits)
Signed-off-by: dependabot[bot] <support@dependabot.com>

Previously, isScopedToUri took into account enforcement but now
it does not. Thus, isSelectable logic based on scopes.enforced
and isScopedToUri was added to the group-list-section controller
to determine whether a group should be out of scope/not selectable.

Since the feature flag isn't available until the profile api request
completes, and the profile api request is run asynchronously with the
group api requests, it doesn't make sense to load conditionaly. Instead,
call all the group api requests and filter the groups for the old groups
menu later. In this case, filtering is done in the groups.all
conditionally based on the feature flag.

Previously, if the scope was not enforced, isScopedToUri would always
return true. Now, isScopedToUri is true only when the scopes.uri_patterns
match.

getInScopeGroups returns the list of groups for the old groups
menu. Aka: only groups that are in scope are returned.