Merge pull request #122 from Financial-Times/ACC-2296_docCoverageUpdate

ACC-2296: Doc and coverage script update
Financial-Times · Apr 17, 2023 · 74bca88 · 74bca88
2 parents e90e996 + 0c57f04
commit 74bca88
Show file tree

Hide file tree

Showing 4 changed files with 1,399 additions and 78 deletions.
diff --git a/Makefile b/Makefile
@@ -7,6 +7,9 @@ node_modules/@financial-times/n-gage/index.mk:
 unit-test:
 	mocha 'test/**/*.spec.js' --inline-diffs
 
+unit-test-coverage:
+	nyc mocha 'test/**/*.spec.js' --inline-diffs
+
 test:
 	make verify
 	make unit-test

diff --git a/README.md b/README.md
@@ -1,5 +1,6 @@
 # n-feedback
 Displays a feedback button when clicked, allows you to fill in a form with feedback questions.
+This questions come from "next-feedback-api" that get it from "qualtrics configured survey"
 
 ## Demo
 ```sh
@@ -41,4 +42,9 @@ If you are wanting to deploy to FT.com, after creating the tag you will need to
 ```sh
 make test        
 ```
+# Monitoring
+To monitoring we can check the grafana panel of "next-feedback-api" to check if the requests still working or if we have an increment of errors on it
+[next-feedback-api](https://grafana.ft.com/d/1eU9QR3mz/next-feedback-api?orgId=1)
+
+