Skip to content

Conversation

@rhoerr
Copy link
Contributor

@rhoerr rhoerr commented Oct 2, 2025

I propose adding mage-os/module-inventory-reservations-grid as a bundled module. https://github.com/mage-os-lab/module-inventory-reservations-grid

  • It provides a clear admin grid for Magento inventory reservations, improving visibility into reservation data.
  • It is slim, purpose-built, and compatible with the latest Magento versions.
  • It is maintained and focused on a single, important feature for inventory transparency.

Implications

  • Admins gain direct access to view and manage inventory reservations via a dedicated grid in the admin panel.
  • May reveal underlying issues with reservations or stock management that require process adjustments.
  • No impact on frontend performance or customer experience, as all changes are on the backend.

Risks

  • Potential for minor admin confusion if reservation data is unfamiliar, but documentation is available.
  • Risk of unsafe inventory data operations, if admin enables the delete mass action and uses it unwisely. There are warnings against this.

Benefits

  • Enhanced transparency for inventory reservations, aiding troubleshooting and operational oversight.

PR

This PR results in the module being added as a pinned require of mage-os/product-community-edition like:

"mage-os/module-inventory-reservations-grid": "1.0.0",

which composer will then require via Packagist, like any other third party package. The latest published version will be pinned at the time of each release.

@rhoerr
Copy link
Contributor Author

rhoerr commented Oct 2, 2025

Created as a draft for now; there are PRs on https://github.com/mage-os-lab/module-inventory-reservations-grid/issues that need to be processed before we move forward.

@rhoerr
Copy link
Contributor Author

rhoerr commented Oct 2, 2025

CC @thomas-kl1

@fballiano
Copy link
Contributor

I think the bundled modules should be in the mage-os namespace because of branch protection rules

@rhoerr
Copy link
Contributor Author

rhoerr commented Oct 2, 2025

I think the bundled modules should be in the mage-os namespace because of branch protection rules

I agree. I don't want to move anything though until

  1. Quality issues are resolved;
  2. We've decided definitely that they're ready and should be bundled

@rhoerr rhoerr added this to the Mage-OS 2.0 milestone Oct 2, 2025
@rhoerr rhoerr changed the base branch from main to release/mage-os-2.0 October 2, 2025 12:20
@rhoerr rhoerr added the enhancement New feature or request label Oct 2, 2025
@rhoerr rhoerr marked this pull request as ready for review October 7, 2025 16:22
@rhoerr rhoerr requested a review from a team as a code owner October 7, 2025 16:22
@rhoerr
Copy link
Contributor Author

rhoerr commented Oct 7, 2025

Marked ready for review/consideration/merge if appropriate.

All issues I reported from review are resolved and merged. I tagged and released version 1.0.0. We will need to move it to the Mage-OS org still, but IMO the module is ready.

https://github.com/mage-os-lab/module-inventory-reservations-grid/releases/tag/1.0.0

@rhoerr
Copy link
Contributor Author

rhoerr commented Oct 8, 2025

  • Code was reviewed, patched, and released.
  • It adds a report and mass action that covers a gap in default functionality.
  • I haven't heard any voices of concern.
  • I think it's simple enough and adds value enough to justify bundling it.

Merging.

I opened #246 for moving lab modules to the proper org before release. IMO the PRs and decision for whether something should be bundled or not is separate from moving the modules we do decide to bundle to the proper GH org.

@rhoerr rhoerr merged commit ca3fd48 into mage-os:release/mage-os-2.0 Oct 8, 2025
1 check passed
@rhoerr rhoerr deleted the bundle/mage-os-module-inventory-reservations-grid branch October 8, 2025 01:41
rhoerr added a commit that referenced this pull request Oct 16, 2025
* Bundle aligent/magento2-pci-4-compatibility (#239)

* Bundle mage-os/module-meta-robots-tag (#244)

* Bundle mage-os/meta-robots-tag

* Fixed composer package name based on latest changes

* Bundle mage-os/module-inventory-reservations-grid (#241)

* Bundle mage-os/module-pagebuilder-template-import-export (#230)

* Update dependencies-template.json with pagebuilder enhancements

* Add mage-os pagebuilder import/export and cms widget component modules to dependencies-template.json

* Update dependencies-template.json

* Fix wrong dependency git repos using https

* Changed PR to just mage-os/module-pagebuilder-template-import-export

---------

Co-authored-by: Ryan Hoerr <rhoerr@users.noreply.github.com>

* Bundle mage-os/module-page-builder-widget (#243)

* Fix magento2-base build error for new releases (#247)

* Bundle mage-os/module-theme-optimization (#242)

* Bundle mage-os/module-automatic-translation (#231)

* Add SEO meta robots module "mage-os/meta-robots-tag"

* Add automatica translation with AI module

* Changed PR to just mage-os/module-automatic-translation

---------

Co-authored-by: Ryan Hoerr <rhoerr@users.noreply.github.com>

* Changed mage-os-lab references to mage-os

* Fix magento2-base composer map of tiny_mce_6 to hugerte (#258)

---------

Co-authored-by: dadolun95 <dadolun@gmail.com>
Co-authored-by: SamueleMartini <40766441+SamueleMartini@users.noreply.github.com>
Co-authored-by: Mage-OS <100189073+mage-os-ci@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants