Conversation
Em-849 externalise JUI PDF annotator
5eccd3c
to
f081b81
Compare
f081b81
to
2d3955a
Compare
42cf1e5
to
e0d8382
Compare
…oad-npm-in-pipeline
/node_modules | ||
node_modules/ | ||
projects/annotation-ui-lib/node_modules/ | ||
**/**/node_modules/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
question: why is this double -> **/ **/ , would **/node_modules/ work
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
supposed to exclude node_modules for the library, under projects/annotation-ui-lib/node_modules
- go to http://localhost:3000 and the viewer should load the document. | ||
|
||
### Building annotation library | ||
- npm run package |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
as mentioned before, maybe best to have this before running
api/index.ts
Outdated
|
||
if (config.configEnv !== 'prod') { | ||
// Uncomment to enable direct access to Microservices | ||
dmStoreApiRoutes(router) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reason for the new line?
package.json
Outdated
@@ -35,6 +40,7 @@ | |||
"@angular/platform-browser-dynamic": "6.1.0", | |||
"@angular/platform-server": "6.1.0", | |||
"@angular/router": "6.1.0", | |||
"@hmcts/annotation-ui-lib": "1.0.7-RC", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this need to be updated for the new version
Codecov Report
@@ Coverage Diff @@
## master #6 +/- ##
=========================================
Coverage ? 90.62%
=========================================
Files ? 6
Lines ? 64
Branches ? 2
=========================================
Hits ? 58
Misses ? 4
Partials ? 2
Continue to review full report at Codecov.
|
No description provided.