Merge branch 'develop' of https://github.com/GoogleCloudPlatform/fda-…

…mystudies into urwebapp-sites-tab

# Conflicts:
#	UR-web-app/src/app/site-coordinator/location/shared/location.service.spec.ts
#	UR-web-app/src/app/site-coordinator/studies/shared/studies.service.spec.ts

.github/workflows/build-participant-manager-module.yml

@@ -0,0 +1,19 @@
+on:
+  pull_request:
+    paths:
+    - 'participant-manager-module/**'
+    - '.github/workflows/build-participant-manager-module.yml'
+name: Build-participant-manager-module
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v2
+    - name: Set up JDK 1.8
+      uses: actions/setup-java@v1
+      with:
+        java-version: 1.8
+    - name: Build with Maven
+      run: |
+          cd participant-manager-module
+          mvn -B package --file pom.xml        

.gitignore

@@ -9,3 +9,5 @@
 **/.factorypath
 **/*.log
 user-registration-server-ws/consent-mgmt/ConsentMgmtServicesWs.iml
+UR-web-app/coverage/
+UR-web-app/node_modules/

UR-web-app/.eslintrc.js

@@ -11,7 +11,9 @@ module.exports = {
     'google',
   ],
   globals: {
+    // eslint-disable-next-line @typescript-eslint/naming-convention
     Atomics: 'readonly',
+    // eslint-disable-next-line @typescript-eslint/naming-convention
     SharedArrayBuffer: 'readonly',
     localStorage: true,
   },
@@ -22,7 +24,7 @@ module.exports = {
     tsconfigRootDir: __dirname,
     project: 'tsconfig.eslint.json',
   },
-  plugins: ['@typescript-eslint'],
+  plugins: ['@typescript-eslint', 'unicorn'],
   rules: {
     'require-jsdoc': 'off',
     'no-unused-vars': 'off',
@@ -35,6 +37,33 @@ module.exports = {
     '@typescript-eslint/prefer-readonly': 'error',
     '@typescript-eslint/promise-function-async': 'error',
     '@typescript-eslint/switch-exhaustiveness-check': 'error',
+    '@typescript-eslint/naming-convention': [
+      'error',
+      {
+        selector: 'default',
+        format: ['camelCase'],
+      },
+
+      {
+        selector: 'variable',
+        format: ['camelCase', 'UPPER_CASE'],
+      },
+      {
+        selector: 'parameter',
+        format: ['camelCase'],
+        leadingUnderscore: 'allow',
+      },
+      {
+        selector: 'typeLike',
+        format: ['PascalCase'],
+      },
+    ],
+    'unicorn/filename-case': [
+      'error',
+      {
+        case: 'kebabCase',
+      },
+    ],
 
     // Handled by prettier.
     'semi': 'off',

UR-web-app/.prettierrc

@@ -1,6 +1,6 @@
 {
-    "quoteProps": "consistent",
-    "trailingComma": "all",
-    "bracketSpacing": false,
-    "xmlWhitespaceSensitivity": "ignore"
+  "quoteProps": "consistent",
+  "trailingComma": "all",
+  "bracketSpacing": false,
+  "xmlWhitespaceSensitivity": "ignore"
 }

UR-web-app/e2e/protractor.conf.js

@@ -2,6 +2,7 @@
 // Protractor configuration file, see link for more information
 // https://github.com/angular/protractor/blob/master/lib/config.ts
 
+// eslint-disable-next-line @typescript-eslint/naming-convention
 const {SpecReporter} = require('jasmine-spec-reporter');
 
 exports.config = {