diff --git a/.github/workflows/scala.yml b/.github/workflows/scala.yml index 07a2cc43e..d4b20044e 100644 --- a/.github/workflows/scala.yml +++ b/.github/workflows/scala.yml @@ -7,7 +7,7 @@ on: branches: [ master ] jobs: - build: + build_and_test: runs-on: ubuntu-latest diff --git a/app/femr/data/daos/system/UserRepository.java b/app/femr/data/daos/system/UserRepository.java index b68c489cf..14028907a 100644 --- a/app/femr/data/daos/system/UserRepository.java +++ b/app/femr/data/daos/system/UserRepository.java @@ -237,9 +237,12 @@ public List retrieveUsersByTripId(Integer tripId){ @Override public List retrieveAllRoles(){ - ExpressionList roleQuery = QueryProvider.getRoleQuery() - .where() + ExpressionList roleQuery = QueryProvider.getRoleQuery().where(); + + /* .ne("name", "SuperUser"); + Why remove SuperUser from role list? Caused bug where updating SuperUser removed its role... + */ List allRoles; try{ diff --git a/app/femr/ui/controllers/ReferenceController.java b/app/femr/ui/controllers/ReferenceController.java index 6b48e0ce1..3e679980c 100644 --- a/app/femr/ui/controllers/ReferenceController.java +++ b/app/femr/ui/controllers/ReferenceController.java @@ -11,7 +11,7 @@ import femr.ui.views.html.reference.index; @Security.Authenticated(FEMRAuthenticated.class) -@AllowedRoles({Roles.PHYSICIAN, Roles.PHARMACIST, Roles.NURSE, Roles.SUPERUSER}) +@AllowedRoles({Roles.PHYSICIAN, Roles.PHARMACIST, Roles.NURSE, Roles.SUPERUSER, Roles.ADMINISTRATOR}) public class ReferenceController extends Controller { private final AssetsFinder assetsFinder; diff --git a/app/femr/ui/views/admin/trips/cities.scala.html b/app/femr/ui/views/admin/trips/cities.scala.html index c52863b37..56f349994 100644 --- a/app/femr/ui/views/admin/trips/cities.scala.html +++ b/app/femr/ui/views/admin/trips/cities.scala.html @@ -12,6 +12,38 @@ @additionalScripts = { + } @admin("Trips - Manage Cities", currentUser, scripts = additionalScripts, styles = additionalStyles, assets = assets) { @@ -24,15 +56,18 @@ @helper.form(action = TripController.citiesPost()) {
-

Add City:

+

Add City

-
-
-
-
-
- +
} @@ -77,12 +120,12 @@

Add Trip:

- - - - - - + + + + + + diff --git a/app/femr/ui/views/admin/trips/teams.scala.html b/app/femr/ui/views/admin/trips/teams.scala.html index 25ca8ef40..4bcfa7c91 100644 --- a/app/femr/ui/views/admin/trips/teams.scala.html +++ b/app/femr/ui/views/admin/trips/teams.scala.html @@ -12,6 +12,41 @@ @additionalScripts = { + } @admin("Trips - Manage Teams", currentUser, styles = additionalStyles, scripts = additionalScripts, assets = assets) { @@ -25,21 +60,24 @@ @helper.form(action = TripController.teamsPost()) {
-

Add Team:

+

Add Team:

-
-
-
@@ -49,7 +87,7 @@

Add Team:

- +
} @@ -58,9 +96,9 @@

Add Team:

EditTeam NameCountryCityStart DateEnd DateEditTeam NameCountryCityStart DateEnd Date
- - - + + + diff --git a/app/femr/ui/views/admin/updates/manage.scala.html b/app/femr/ui/views/admin/updates/manage.scala.html index f0ac84229..adf8eec08 100644 --- a/app/femr/ui/views/admin/updates/manage.scala.html +++ b/app/femr/ui/views/admin/updates/manage.scala.html @@ -26,6 +26,7 @@

Network Status

+
Team NameDescriptionOriginTeam NameDescriptionOrigin
diff --git a/app/femr/ui/views/home/index.scala.html b/app/femr/ui/views/home/index.scala.html index 504fc0220..300d65095 100644 --- a/app/femr/ui/views/home/index.scala.html +++ b/app/femr/ui/views/home/index.scala.html @@ -56,7 +56,7 @@ } - + @*Sample texts. Content is pulled from the languages json in the public directory.*@ @top = {
diff --git a/app/femr/ui/views/layouts/main.scala.html b/app/femr/ui/views/layouts/main.scala.html index 69edd3d9a..5a8e94d84 100644 --- a/app/femr/ui/views/layouts/main.scala.html +++ b/app/femr/ui/views/layouts/main.scala.html @@ -15,7 +15,7 @@ - + diff --git a/app/femr/ui/views/partials/footer.scala.html b/app/femr/ui/views/partials/footer.scala.html index 4a1790bad..1543c673d 100644 --- a/app/femr/ui/views/partials/footer.scala.html +++ b/app/femr/ui/views/partials/footer.scala.html @@ -2,15 +2,12 @@