- Issue created by @tedbow
- π¦πΊAustralia larowlan π¦πΊπ.au GMT+10
wim leers β credited larowlan β .
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
Looks like the right direction, but still a bit rough around the edges. With minimal effort we can avoid the need for follow-up issues. π
- π¦πΊAustralia larowlan π¦πΊπ.au GMT+10
This is very close, just a couple of minor adjustments needed
- π¦πΊAustralia larowlan π¦πΊπ.au GMT+10
Addressed Wim and my review comments whilst its a public holiday in the US, this looks ready to go
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
π Add a Publish Experience Builder Content permission Postponed will need to update this to use the appropriate permissions.
β¨ Implement OAuth2 authentication for API endpoints related to code components Active made it impossible to run this test locally without also installing
simple_oauth
, butCONTRIBUTING.md
had not yet been updated. Missed that in review there π Fixing that here. -
wim leers β
committed 23f5d9f4 on 0.x authored by
tedbow β
Issue #3529207 by tedbow, wim leers, larowlan: For selective reverting...
-
wim leers β
committed 23f5d9f4 on 0.x authored by
tedbow β
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
- #3530497-4: [PP-1] Create UI for selective discarding of changes β is now less blocked.
- #3529892-11: Add a Publish Experience Builder Content permission β now needs updating.
- Created follow-up: π [later phase] Generalize GET/PATCH/DELETE for individual auto-save entries: refactor `ApiConfigAutoSaveControllers` and `ApiAutoSaveController::delete()` Postponed .