Resolve TMSS-682
All threads resolved!
All threads resolved!
Closes TMSS-682
Merge request reports
Activity
- Resolved by Roy de Goei
Note: This merge request / branch should remain open until frontend has also been addressed. So please don't merge to master yet.
- Resolved by Roy de Goei
- Resolved by Roy de Goei
- Resolved by Roy de Goei
- Resolved by Roy de Goei
- Resolved by Roy de Goei
- Resolved by Roy de Goei
- Resolved by Roy de Goei
- Resolved by Roy de Goei
Alright, backend code looks overall quite nicely done.
Again, we need to wait for the frontend code, but concerning the backend this is almost good to merge. Maybe sort out that variable name in the populate, but I think that's all that should be addressed.
I made some further general remarks, because the whole structure is relatively complex and naming (which was established before this ticket) does not help to intuitively grasp this easily, at least in my opinion. Maybe we should review that in a contingency sprint or so.
mentioned in commit 118dfd65
Please register or sign in to reply