Skip to content

resources: procurement: doc how to make a procurement request#12

Merged
NicolasCARPi merged 2 commits into
masterfrom
mouss/260521-procurementRequest
May 21, 2026
Merged

resources: procurement: doc how to make a procurement request#12
NicolasCARPi merged 2 commits into
masterfrom
mouss/260521-procurementRequest

Conversation

@MoustaphaCamara
Copy link
Copy Markdown
Contributor

@MoustaphaCamara MoustaphaCamara commented May 21, 2026

Summary by CodeRabbit

  • Documentation
    • Added a “Making a Resource procurable” section: explains enabling procurement parameters, placing orders via the procurement modal, and managing procurement requests from the Team page (including confirmation messaging).
    • Simplified and clarified instructions for making resources bookable, with a concise directive to use the Modify booking parameters action.
    • Minor spacing/formatting adjustment before booking content.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 21, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: d8b3950b-1b74-43a1-8afe-094f80ca5498

📥 Commits

Reviewing files that changed from the base of the PR and between a4b1bf2 and 72efd68.

📒 Files selected for processing (1)
  • docs/usage/user-guide/resources.md

Walkthrough

This PR edits the Resources user guide: small spacing change, replaces the "Making a Resource bookable" instruction with a concise pointer to Modify booking parameters, and adds a new "Making a Resource procurable" section describing procurement enablement, ordering, confirmation, and Team page request management.

Changes

Resource Documentation Update

Layer / File(s) Summary
Procurement and booking documentation
docs/usage/user-guide/resources.md
Minor spacing change before booking content; shortens "Making a Resource bookable" to direct users to Modify booking parameters; adds "Making a Resource procurable" section covering procurement enablement, placing orders via the procurement modal, confirmation messaging, and managing procurement requests from the Team page.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

  • elabftw/documentation#8: Updates the same resource guide section covering the "Modify booking parameters" toolbar workflow and procurement-related procedures.

Suggested reviewers

  • NicolasCARPi

Poem

As First Officer, I note with care,
A booking hint trimmed to be fair,
Procurement steps now plainly told,
Team page traces, confirmations bold,
Files updated—log this repair. 🖖

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main change: documentation additions explaining how to make a procurement request, with related updates to procurement procedures.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch mouss/260521-procurementRequest

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

Copy link
Copy Markdown

@eltouma eltouma left a comment

Choose a reason for hiding this comment

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

👌

@NicolasCARPi NicolasCARPi merged commit 4f36ba0 into master May 21, 2026
1 of 2 checks passed
@NicolasCARPi NicolasCARPi deleted the mouss/260521-procurementRequest branch May 21, 2026 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants