Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AC#1102/custom annotations #291

Open
wants to merge 22 commits into
base: master
Choose a base branch
from
Open

AC#1102/custom annotations #291

wants to merge 22 commits into from

Conversation

smnhgn
Copy link
Member

@smnhgn smnhgn commented Jan 15, 2025

Beinhaltet folgende Anpassungen:

  • Neue Tabelle für die Konfiguration von Annotations -> system_annotations
  • DB-Migration für die 4 Standard Annotations important, check-me, postpone, needs_translation
  • Endpunkte zum Abrufen, Erstellen und Löschen von Annotations inkl. Tests und Swagger-Doku -> /system/annotations

Copy link

github-actions bot commented Jan 15, 2025

Test Results

1 047 tests  +17   1 039 ✅ +17   3m 24s ⏱️ -34s
  126 suites + 1       8 💤 ± 0 
  126 files   + 1       0 ❌ ± 0 

Results for commit 137fbcb. ± Comparison against base commit 27e37cd.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jan 15, 2025

File Coverage
All files 88%
com/campudus/tableaux/exceptions.scala 0%
com/campudus/tableaux/exceptions.scala 0%
com/campudus/tableaux/arguments.scala 97%
com/campudus/tableaux/Starter.scala 78%
com/campudus/tableaux/cache/CacheVerticle.scala 77%
com/campudus/tableaux/cache/CacheClient.scala 63%
com/campudus/tableaux/controller/SystemController.scala 99%
com/campudus/tableaux/controller/TableauxController.scala 98%
com/campudus/tableaux/controller/StructureController.scala 91%
com/campudus/tableaux/controller/MediaController.scala 80%
com/campudus/tableaux/database/historyType.scala 7%
com/campudus/tableaux/database/languageType.scala 86%
com/campudus/tableaux/database/historyType.scala 66%
com/campudus/tableaux/database/database.scala 96%
com/campudus/tableaux/database/database.scala 81%
com/campudus/tableaux/database/database.scala 97%
com/campudus/tableaux/database/domain/column.scala 71%
com/campudus/tableaux/database/domain/column.scala 95%
com/campudus/tableaux/database/domain/table.scala 96%
com/campudus/tableaux/database/domain/service.scala 81%
com/campudus/tableaux/database/domain/column.scala 77%
com/campudus/tableaux/database/domain/annotation.scala 14%
com/campudus/tableaux/database/domain/column.scala 48%
com/campudus/tableaux/database/domain/annotation.scala 60%
com/campudus/tableaux/database/domain/history.scala 84%
com/campudus/tableaux/database/domain/column.scala 0%
com/campudus/tableaux/database/domain/annotation.scala 83%
com/campudus/tableaux/database/domain/annotation.scala 80%
com/campudus/tableaux/database/domain/table.scala 99%
com/campudus/tableaux/database/domain/column.scala 95%
com/campudus/tableaux/database/domain/file.scala 72%
com/campudus/tableaux/database/domain/domainobject.scala 99%
com/campudus/tableaux/database/domain/column.scala 92%
com/campudus/tableaux/database/domain/column.scala 78%
com/campudus/tableaux/database/domain/domainobject.scala 93%
com/campudus/tableaux/database/model/HistoryModel.scala 96%
com/campudus/tableaux/database/model/AttachmentModel.scala 82%
com/campudus/tableaux/database/model/TableauxModel.scala 91%
com/campudus/tableaux/database/model/StructureModel.scala 95%
com/campudus/tableaux/database/model/SystemModel.scala 85%
com/campudus/tableaux/database/model/FileModel.scala 94%
com/campudus/tableaux/database/model/structure/ColumnModel.scala 88%
com/campudus/tableaux/database/model/structure/TableModel.scala 84%
com/campudus/tableaux/database/model/structure/TableGroupModel.scala 79%
com/campudus/tableaux/database/model/structure/ColumnModel.scala 88%
com/campudus/tableaux/database/model/tableaux/RowModel.scala 91%
com/campudus/tableaux/database/model/tableaux/RowModel.scala 92%
com/campudus/tableaux/database/model/tableaux/RowModel.scala 93%
com/campudus/tableaux/helper/FileUtils.scala 74%
com/campudus/tableaux/helper/JsonUtils.scala 94%
com/campudus/tableaux/helper/ResultChecker.scala 98%
com/campudus/tableaux/helper/IdentifierFlattener.scala 80%
com/campudus/tableaux/router/DocumentationRouter.scala 21%
com/campudus/tableaux/router/BaseRouter.scala 63%
com/campudus/tableaux/router/SystemRouter.scala 90%
com/campudus/tableaux/router/MediaRouter.scala 60%
com/campudus/tableaux/router/TableauxRouter.scala 82%
com/campudus/tableaux/router/StructureRouter.scala 99%
com/campudus/tableaux/router/RouterRegistry.scala 52%
com/campudus/tableaux/router/auth/KeycloakAuthHandler.scala 72%
com/campudus/tableaux/router/auth/permission/Condition.scala 89%
com/campudus/tableaux/router/auth/permission/Condition.scala 74%
com/campudus/tableaux/router/auth/permission/Condition.scala 87%
com/campudus/tableaux/router/auth/permission/RoleModel.scala 87%
com/campudus/tableaux/router/auth/permission/RoleModel.scala 0%
com/campudus/tableaux/router/auth/permission/Condition.scala 90%
com/campudus/tableaux/router/auth/permission/Condition.scala 79%
com/campudus/tableaux/router/auth/permission/Action.scala 97%
com/campudus/tableaux/router/auth/permission/Condition.scala 85%
com/campudus/tableaux/router/auth/permission/TableauxUser.scala 63%
com/campudus/tableaux/verticles/JsonSchemaValidator/JsonSchemaValidator.scala 86%
com/campudus/tableaux/verticles/MessagingVerticle/MessagingVerticle.scala 84%

Minimum allowed coverage is 80%

Generated by 🐒 cobertura-action against 137fbcb

@smnhgn smnhgn requested a review from zingmane January 22, 2025 10:18
Copy link
Member

@zingmane zingmane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Die cascade trigger brauchen wir noch an den beiden Stellen ansonsten nur Kleinigkeiten und Kosmetik

Copy link
Member

@zingmane zingmane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Für mich siehts gut aus 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants