Skip to content

Add additional api menu entries - #255

Merged
andreslucena merged 4 commits into
developfrom
feature/additional-api
Jul 20, 2026
Merged

Add additional api menu entries#255
andreslucena merged 4 commits into
developfrom
feature/additional-api

Conversation

@alecslupu

@alecslupu alecslupu commented Feb 9, 2026

Copy link
Copy Markdown
Contributor

@coderabbitai

coderabbitai Bot commented Feb 9, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 59328f7f-dd4b-41ab-8b0f-8354aae592fe

📥 Commits

Reviewing files that changed from the base of the PR and between dffcd3b and f48755a.

📒 Files selected for processing (1)
  • docs/en/modules/ROOT/nav.adoc

📝 Walkthrough

Summary by CodeRabbit

  • Documentation
    • Expanded the API reference navigation with new API reference error documentation for unauthorized_object.
    • Added new accountability component documentation (create/update/delete) plus milestone create/update/delete guides.
    • Added new budgets component documentation (create/update/delete) with nested projects create/update/delete pages.

Walkthrough

API documentation navigation adds an unauthorized_object error entry and xref entries for the accountability and budgets components, including their CRUD, milestone, and nested project operation pages.

Changes

API Reference Navigation

Layer / File(s) Summary
API reference navigation entries
docs/en/modules/ROOT/nav.adoc
Adds the unauthorized_object error entry and component navigation for accountability and budgets, including milestone and nested projects operation pages.

Estimated code review effort: 1 (Trivial) | ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly reflects the added API menu entries in the documentation navigation.
Description check ✅ Passed The description is directly related, linking the documentation updates to the upstream work they document.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@alecslupu
alecslupu marked this pull request as ready for review May 23, 2026 02:32
Copilot AI review requested due to automatic review settings May 23, 2026 02:32

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds new API reference navigation entries so the documentation site exposes the newly documented Budgets and Accountability API operations (as referenced in the linked decidim/decidim PRs).

Changes:

  • Add Budgets API reference menu group with Budget and Project CRUD operation links.
  • Add Accountability API reference menu group with Result and Milestone CRUD operation links.
Comments suppressed due to low confidence (1)

docs/en/modules/ROOT/nav.adoc:270

  • The action links under Accountability are ordered Create → Delete → Update. For consistency with other action lists in this same section, consider reordering to Create → Update → Delete.
****** xref:develop:api/reference/components/accountability/create.adoc[Create Result]
****** xref:develop:api/reference/components/accountability/delete.adoc[Delete Result]
****** xref:develop:api/reference/components/accountability/update.adoc[Update Result]
****** xref:develop:api/reference/components/accountability/milestone/create.adoc[Create Milestone]
****** xref:develop:api/reference/components/accountability/milestone/update.adoc[Update Milestone]
****** xref:develop:api/reference/components/accountability/milestone/delete.adoc[Delete Milestone]

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/en/modules/ROOT/nav.adoc
Comment thread docs/en/modules/ROOT/nav.adoc Outdated

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@docs/en/modules/ROOT/nav.adoc`:
- Around line 257-270: The new xrefs in docs/en/modules/ROOT/nav.adoc reference
non-existent pages (e.g.,
develop:api/reference/components/accountability/create.adoc,
develop:api/reference/components/accountability/milestone/create.adoc,
develop:api/reference/components/budgets/create.adoc,
develop:api/reference/components/budgets/projects/create.adoc); either change
those xref targets to the actual existing docs under admin (for example point to
admin pages like
docs/en/modules/admin/pages/components/accountability/results.adoc or
docs/en/modules/admin/pages/components/budgets/projects.adoc) or add the missing
develop:api/reference/... .adoc files so the xrefs resolve; update the lines
that list the xrefs in docs/en/modules/ROOT/nav.adoc to use the correct target
strings or ensure the matching files are created with the expected names.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 7855dd46-52ca-4f82-93ac-97841eea417e

📥 Commits

Reviewing files that changed from the base of the PR and between 873816d and dffcd3b.

📒 Files selected for processing (1)
  • docs/en/modules/ROOT/nav.adoc

Comment thread docs/en/modules/ROOT/nav.adoc

@andreslucena andreslucena left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one missing error and we're done

Comment thread docs/en/modules/ROOT/nav.adoc
Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
@alecslupu
alecslupu requested a review from andreslucena July 20, 2026 09:42
@andreslucena
andreslucena merged commit 6a50fdf into develop Jul 20, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants