Skip to content

Conversation

@peluko00
Copy link

@peluko00 peluko00 commented Dec 24, 2025

This module adds a configurable sequence for dms file records.

cc https://github.com/APSL 30754
@miquelalzanillas @javierobcn @mpascuall @BernatObrador @ppyczko @lbarry-apsl please review

@peluko00 peluko00 force-pushed the 17.0-add-dms_file_sequence branch 5 times, most recently from a89789d to 68e2626 Compare December 24, 2025 10:35
@peluko00 peluko00 force-pushed the 17.0-add-dms_file_sequence branch from 68e2626 to 06b079c Compare December 24, 2025 10:55
Copy link

@BernatObrador BernatObrador left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@lbarry-apsl lbarry-apsl left a comment

Choose a reason for hiding this comment

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

Revised code and runboat. LGTM

@peluko00
Copy link
Author

Sorry for ping directly @pedrobaeza, seems it's ready for merge

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

Copy link
Member

@etobella etobella left a comment

Choose a reason for hiding this comment

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

Technical review

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 17.0-ocabot-merge-pr-453-by-etobella-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 22a3f44 into OCA:17.0 Dec 31, 2025
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 94ec427. Thanks a lot for contributing to OCA. ❤️

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.

5 participants