Skip to content

docs(App): drop/update outdated {OC, OCP}\AppFramework\App docblocks #60902

Open
joshtrichards wants to merge 3 commits into
masterfrom
jtr/docs-AppFramework-App
Open

docs(App): drop/update outdated {OC, OCP}\AppFramework\App docblocks #60902
joshtrichards wants to merge 3 commits into
masterfrom
jtr/docs-AppFramework-App

Conversation

@joshtrichards
Copy link
Copy Markdown
Member

@joshtrichards joshtrichards commented Jun 1, 2026

Summary

  • Updates OCP class and method-level docs for clarity and accuracy
  • Updates OC class and method-level docs for clarity and accuracy
  • Removes outdated and no longer applicable explanations throughout both classes
  • Removes outdated examples (which better belong in Dev Manual anyhow).

TODO

  • I'm not sure, but I think maybe OCP\AppFramework\App::dispatch() could/should be deprecated and/or better documented (from what I can tell it was part of the old approach to adding controller routes in routes.php unless I'm thinking about this entirely wrong).

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Signed-off-by: Josh <josh.t.richards@gmail.com>
Signed-off-by: Josh <josh.t.richards@gmail.com>
Signed-off-by: Josh <josh.t.richards@gmail.com>
@joshtrichards joshtrichards added this to the Nextcloud 35 milestone Jun 1, 2026
@joshtrichards joshtrichards added 2. developing Work in progress feature: apps management technical debt 🧱 🤔🚀 developer experience 3. to review Waiting for reviews and removed 2. developing Work in progress labels Jun 1, 2026
@joshtrichards joshtrichards marked this pull request as ready for review June 1, 2026 14:02
@joshtrichards joshtrichards requested a review from a team as a code owner June 1, 2026 14:02
@joshtrichards joshtrichards requested review from ArtificialOwl, artonge, leftybournes and salmart-dev and removed request for a team June 1, 2026 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant