...
# | Criteria | Comments/action items |
| Evidence | |||||
---|---|---|---|---|---|---|---|---|---|
1 | Uses Apache 2.0 license |
| ui-reading-room/LICENSE at master · folio-org/ui-reading-room (github.com) | ||||||
2 | Module build MUST produce a valid module descriptor |
| |||||||
3 | Module descriptor MUST include interface requirements for all consumed APIs |
| |||||||
4 | Third party dependencies use an Apache 2.0 compatible license |
| |||||||
5 | In order to ensure reproducible builds, snapshot versions of build-time dependencies should not be referenced. |
| |||||||
6 | Installation documentation is included | Nothing specific for UI module |
| ui-reading-room/README.md at master · folio-org/ui-reading-room (github.com) | |||||
7 | Personal data form is completed, accurate, and provided as PERSONAL_DATA_DISCLOSURE.md file | ||||||||
8 | Sensitive and environment-specific information is not checked into git repository |
| |||||||
9 | Module is written in a language and framework from the officially approved technologies page |
| |||||||
10 | Module only uses FOLIO interfaces already provided by previously accepted modules e.g. a UI module cannot be accepted that relies on an interface only provided by a back end module that hasn't been accepted yet | This UI module comes together with mod-reading-room that is new module as well, can't mark this is completed, but as soon as mod-consortia reading-room is accepted this point can be marked as done | |||||||
11 | Module gracefully handles the absence of third party systems or related configuration |
| |||||||
12 | Sonarqube hasn't identified any security issues, major code smells or excessive (>3%) duplication |
| |||||||
13 | Uses officially supported build tools |
| |||||||
14 | Unit tests have 80% coverage or greater, and are based on officially approved technologies |
| |||||||
15 | If provided, End-to-end tests must be written in an officially approved technology | while it's strongly recommended that modules implement integration tests, it's not a requirement |
| ||||||
16 | Have i18n support via react-intl and an |
| |||||||
17 | Have WCAG 2.1 AA compliance as measured by a current major version of axe DevTools Chrome Extension | master latest has been pulled and accessibility verified from localhost. 1. Lighthouse score is 100.
|
| ||||||
18 | Use the latest release of Stripes at the time of evaluation |
| |||||||
19 | Follow relevant existing UI layouts, patterns and norms |
| |||||||
20 | Must work in the latest version of Chrome (the supported runtime environment) at the time of evaluation |
|
...