-
Notifications
You must be signed in to change notification settings - Fork 2
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
Permissions/rows #267
Permissions/rows #267
Conversation
a1a7330
to
f33442d
Compare
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.
Was fehlt noch?
- Tests für history annotation changed
- Anpassung openApi docs (patch row, was noch?)
- Anpassung auth permission doc
- Runtime error, wenn mehr als eine row permission übergeben wird
- Runtime error, wenn der Endpoint
rows/annotations
verwendet wird - Rebase auf master fehlt noch
- Anpassungen Frontend
src/main/scala/com/campudus/tableaux/router/TableauxRouter.scala
Outdated
Show resolved
Hide resolved
src/main/scala/com/campudus/tableaux/database/model/tableaux/RowModel.scala
Outdated
Show resolved
Hide resolved
src/test/scala/com/campudus/tableaux/api/content/LinkTest.scala
Outdated
Show resolved
Hide resolved
src/main/scala/com/campudus/tableaux/database/model/TableauxModel.scala
Outdated
Show resolved
Hide resolved
0d31e2e
to
9c7a164
Compare
Minimum allowed coverage is Generated by 🐒 cobertura-action against 0baa7a0 |
9c7a164
to
985e578
Compare
7de11c4
to
6ff3a5b
Compare
e670e72
to
3c4a823
Compare
API/parameters should be equal over http verbs
on create row
c53ca82
to
41c303a
Compare
src/main/scala/com/campudus/tableaux/controller/TableauxController.scala
Outdated
Show resolved
Hide resolved
src/main/scala/com/campudus/tableaux/database/domain/annotation.scala
Outdated
Show resolved
Hide resolved
src/main/scala/com/campudus/tableaux/database/domain/annotation.scala
Outdated
Show resolved
Hide resolved
src/main/scala/com/campudus/tableaux/database/model/TableauxModel.scala
Outdated
Show resolved
Hide resolved
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.
Vom Code her sieht es gut aus. Bin über alles mal drüber geflogen.
AC-Task: https://app.activecollab.com/116706/projects/2?modal=Task-43888-2