@coveord/jsadmin-ua

jsadmin-usage-analytics

The project's main objective is to handle every feature that lives under the Analytics section of Coveo Cloud Administration Console.

Prerequisites

Set up a developer machine.

Setup

All the commands in the instructions must be run at the root of the jsadmin-usage-analytics folder, unless otherwise specified.

Install

First you need to install the project's dependencies.

npm install

Once the operation is complete, you should see a node_modules folder at the root of the jsadmin-usage-analytics folder.

Build

npm start

This command builds the source code, and automatically triggers a rebuild when a source file is saved.

Test

Run the complete test suite to ensure your local jsadmin-usage-analytics build is healthy and does not introduce any unwanted regressions.

npm test

Contributing

1. hg update on the tip of default branch (hg pull && hg update default).
2. Create a branch to work on following our branching model.
3. Work work work!
4. Create a pull request, our team will automatically be notified.
5. We will review your awesome fix/feature as soon as we manage to.
6. Once your PR gets enough approvals (at least 2 different members of our team), you can merge (or ask us to do so if you don't have the required credentials).
7. Our automated build system will take care of the rest.

Guidelines

• A pull request that increases or maintain the code coverage ratio has higher chances of being merged quickly.
• Make sure your pull requests follow the project's code style. To do so, you can simply run npm run lintfix and it'll fix all the code style errors by itself. Don't forget to commit those changes.
• We're pretty flexible on the commit messages structure and pull requests descriptions, but still, make sure your commit messages and pull request descriptions provide enough details for fellow reviewers.
• We tend to avoid comments in our code base, we strongly prefer good naming and code structure.

Pull request demos

Our automated build system kindly provides a demo link for each pull request, given the build is healthy (all unit tests have run successfully). When available, the links are automatically posted in our #cloudadminbuild private Slack channel. We will gladly invite you to the channel if you're not already a member when you submit a pull request. If we forget to do so, just ping us in Slack and we'll do it right away.

Front-end developpers are visual creatures!

We always like to see your changes in action (when it is relevant) before merging pull requests. Therefore, please include a comment in your pull requests that provides the demo link once it is available. Also, it makes things easier for everone when the link points towards the exact section where lies the bulk of your changes.

Tips and tricks

• npm test command creates a local coverage report at the end of its execution. You can browse this report to see the hits and misses of your unit tests. To do so, open the coverage/lcov-report/index.html file in your favorite web browser.
• npm run test:watch command will run tests on file save.
• npm run reconstruct command will reinstall and rebuild your local branch from scratch without discarding any pending change. Run this if you think your current state is corrupted.
• You can target specific unit tests or test suites by putting an f character in front of any it or describe block (thus writing fit and fdescribe). Only tests in the targeted blocks will be run on the next execution. Don't forget to remove the f before committing.

Running your jsadmin-usage-analytics changes locally in the administration console

If you want to test out your changes locally directly in the administration console, you can do so using a local build of jsadmin-v2 and the npm link feature.

Step	Terminal window 1	Terminal window 2
1	cd path/to/jsadmin-usage-analytics	cd path/to/jsadmin-v2
2		npm link ../relative/path/to/jsadmin-usage-analytics
3	npm start
4		npm start

You can leave both terminal windows running. Changes saved in either repositories will trigger a rebuild of the local demo.

WARNING: running npm install in either of the linked repos might break the npm link. Running again the steps above will fix the situation.

Sometimes you might also need to use your local changes from jsadmin-common. If that is the case:

1. link jsadmin-common to jsadmin-v2;
2. link jsdamin-common to jsadmin-usage-analytics, this step is only to provide the proper typings to your IDE, it would still work at runtime without it;
3. link jsadmin-usage-analytics to jsadmin-v2 using the commands described above.