- Issue created by @wim leers
- First commit to issue fork.
- @penyaskito opened merge request.
- πͺπΈSpain penyaskito Seville π, Spain πͺπΈ, UTC+2 πͺπΊ
> Update all of our API responses (both GET individual and GET list) to provide the entity operations' link relationship for those operations that are permitted
There's no individual GET for ApiContentControllers. I don't think this should be also on auto-saved data, so this should be good now?
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
What a fantastic start! π€©π
- πͺπΈSpain penyaskito Seville π, Spain πͺπΈ, UTC+2 πͺπΊ
The only failing test is because of π Cypress test publish-validation is flaky Active
- Status changed to Needs review
16 days ago 12:35pm 8 June 2025 - πͺπΈSpain penyaskito Seville π, Spain πͺπΈ, UTC+2 πͺπΊ
Surprisingly the rebase was automatic.
- πͺπΈSpain penyaskito Seville π, Spain πͺπΈ, UTC+2 πͺπΊ
Assigning Lee for reviews
- π¦πΊAustralia larowlan π¦πΊπ.au GMT+10
Couple of minor nits on the MR and one concern regarding test stability because of the use of the state api. See π Race conditions/bad lock logic in CacheCollector/State Active
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
Agreed with most of @larowlan's remarks, plus 1 cacheability question and 2 bits of logic simplification.
- πͺπΈSpain penyaskito Seville π, Spain πͺπΈ, UTC+2 πͺπΊ
Handled all feedback.
-
wim leers β
committed 9ff74cbb on 0.x authored by
penyaskito β
Issue #3516657 by penyaskito, wim leers, mglaman, larowlan: Update `...
-
wim leers β
committed 9ff74cbb on 0.x authored by
penyaskito β
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
Related: #3452581-44: [META] XB Permissions β .