Fix addTapListener both on initial load and on navigation.

Initial page load attempted to access a stale actionsApi that was set
at construction time. This wasn't valid until a SHOW_REVISION_ACTIONS

Snapshots of actionsApi and restApi were used across page navigation.
These stale references would prevent proper addTapListener registration.

Change-Id: I12ffd0433badccb5370505f3ac391b8d458d9480
2 files changed
tree: 160331626913c4380cda5b3e7f9e3d07017500c1
  1. src/
  2. web/
  3. .gitignore
  4. BUILD
  5. external_plugin_deps.bzl
  7. LICENSE-re2j

Gerrit Code Review Automerger Plugin

See src/main/resources/documentation.

Web Plugin Development

For running unit tests execute:

bazel test --test_output=all //plugins/automerger/web:karma_test

For checking or fixing eslint formatter problems run:

bazel test //plugins/automerger/web:lint_test
bazel run //plugins/automerger/web:lint_bin -- --fix "$(pwd)/plugins/automerger/web"

For testing the plugin with Gerrit FE Dev Helper build the JavaScript bundle and copy it to the plugins/ folder:

bazel build //plugins/automerger/web:automerger
cp -f bazel-bin/plugins/automerger/web/automerger.js plugins/

and let the Dev Helper redirect from .+/plugins/automerger/static/automerger.js to http://localhost:8081/plugins_/automerger.js.