16: Strip PHP from administration - #3396
Merged
Merged
Conversation
Removes the "PHP API" section from the recent activity page, keeping its Configuration, Permission and security, User privacy and REST API sections; removes the ConfigResolver service-injection section from dynamic configuration while keeping the SiteAccess scope and namespace semantics the spec preserves; and drops the container parameter sample from the configuration page. Clears residue R2 (a GraphQL console reference orphaned by ticket 07) and R17 (the back office described as produced by the ibexa/admin-ui bundle, which a tenant does not install). Repoints three anchor links orphaned by the removed PHP API section at the surviving REST API section. Removes 20 administration/ entries from the invariant baseline. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Ticket 16 of the Cohesivo SaaS conversion. Base:
saas-batch-3.Scope note: the ticket estimated ~20 pages; only 3 remained. Ticket 11
deleted the rest as code-level extension points before this ran. Re-measuring
rather than working from the estimate is why this PR is small.
Changes
recent_activity.mdloses its whole## PHP APIsection (212 lines,including a Twig template include). Its Configuration, Permission and
security, User privacy and REST API sections all survive — the capability
is still documented, over REST.
dynamic_configuration.mdloses## Inject ConfigResolver into services,which is pure PHP service wiring. The
ConfigResolversection is kept: itcarries the SiteAccess scope, namespace and parameter-format semantics that the
spec explicitly preserves.
configuration.mdloses its container-parameter PHP sample; the YAMLexample stays.
Residue
integrated_help.mdadvertised "the GraphQL console" to developers.GraphQL was deleted by ticket 07, so the reference is gone.
back_office.mddescribed the interface as "produced by theibexa/admin-uibundle" — a tenant installs no bundles. Restated as what theback office is. The
/adminURL is kept: it is the real path a tenantvisits, not an installation detail.
Three anchors the strip orphaned
Removing the PHP API section broke three inbound anchor links that no
page-level link inventory contained, caught only by
--strict:recent_activity.mdlinked to its own#disable-logging-activities, whichlived inside the removed section. The sentence offered a PHP way to disable
logging per action, so it is gone.
#searching-in-the-activity-log-groups. Both now point at the surviving#rest-apisection, which is how a tenant actually queries the log.Verification
mkdocs build --strict→ exit 0.pytest -q→ 90 passed.administration/still match``` phpor aninclude_code/include_filemacro naming a.phpfile.administration/entries removed from the invariant baseline.🤖 Generated with Claude Code