Skip to content
This repository was archived by the owner on Jul 15, 2026. It is now read-only.

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

@MoustaphaCamara MoustaphaCamara commented May 21, 2026

Copy link
Copy Markdown
Contributor

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

coderabbitai Bot commented May 21, 2026

Copy link
Copy Markdown

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.

@eltouma eltouma 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.

👌

@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 subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants