Skip to content

[18.0] [IMP] spreadsheet_oca: named input parameters with domain substitution - #105

Open
dnplkndll wants to merge 4 commits into
OCA:18.0from
ledoent:18.0-feat-input-params
Open

[18.0] [IMP] spreadsheet_oca: named input parameters with domain substitution#105
dnplkndll wants to merge 4 commits into
OCA:18.0from
ledoent:18.0-feat-input-params

Conversation

@dnplkndll

@dnplkndll dnplkndll commented Mar 13, 2026

Copy link
Copy Markdown

Adds named input parameters: a spreadsheet cell drives the domain of the Odoo
pivots inside it, so changing the cell re-filters the next refresh.

Depends on #99. This PR carries three of its commits — the
shared pair plus the scheduled refresh — because input parameters extend
spreadsheet.refresh.schedule. All three drop out once #99 merges.

Non-mechanical points worth flagging

  • Substitution touches only the value position of a domain leaf — never a
    field name, never an operator, and it recurses into nested domains. A
    parameter can therefore filter a domain but cannot restructure it.
  • An unknown token is left literal and logged, rather than resolving to
    something that silently matches everything.

AI-assisted (Claude Code); every change reviewed, tested, and owned by the author.

@dnplkndll dnplkndll changed the title [IMP] spreadsheet_oca: named input parameters with domain substitution [18.0] [IMP] spreadsheet_oca: named input parameters with domain substitution Mar 13, 2026
@dnplkndll
dnplkndll force-pushed the 18.0-feat-input-params branch from b243888 to 9cf5c3e Compare March 13, 2026 02:05
@dnplkndll
dnplkndll force-pushed the 18.0-feat-input-params branch from 9cf5c3e to f97f920 Compare May 13, 2026 20:39
@dnplkndll
dnplkndll force-pushed the 18.0-feat-input-params branch from f97f920 to e6d5e6c Compare July 27, 2026 16:25
@dnplkndll
dnplkndll force-pushed the 18.0-feat-input-params branch 2 times, most recently from fa4211c to 4c73512 Compare July 27, 2026 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants