Skip to content

[IMP] spreadsheet: global filters - two new types + general update#16382

Open
mcsi-odoo wants to merge 1 commit into19.0from
19.0-spreadsheet-global-filters-mcsi
Open

[IMP] spreadsheet: global filters - two new types + general update#16382
mcsi-odoo wants to merge 1 commit into19.0from
19.0-spreadsheet-global-filters-mcsi

Conversation

@mcsi-odoo
Copy link
Contributor

task-5881742

@robodoo
Copy link
Collaborator

robodoo commented Feb 12, 2026

Pull request status dashboard

@C3POdoo C3POdoo requested review from a team February 12, 2026 15:08
@mcsi-odoo mcsi-odoo requested review from a team and removed request for a team February 13, 2026 09:37
@xpl-odoo xpl-odoo requested review from xpl-odoo and removed request for a team February 17, 2026 11:10
Copy link
Contributor

@xpl-odoo xpl-odoo left a comment

Choose a reason for hiding this comment

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

Thanks for the update to add the new filter types and the clear explanation about a not-so-clear mechanism (since they don't work like your typical filter) :)

from scratch or add suggested global filters <spreadsheet/global-filters/configure>`.

#. For the relevant global filter, select a conditional operator from the first dropdown, e.g.,
*is in*, then, as relevant, enter or select the appropriate value.
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe explain what are the different operators in a tip?
I think the "is in" is especially confusing 😅

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Indeed, is in and is not in are not really self-explanatory!

I understand your point about explaining the different operators, but I have some questions:

  • would it make more sense to have such explanations on this page about custom filters?
  • from what I can tell, these are all the operators you can have when applying global filters (different ones available depending on field type)

Is in
Is not in
Contains
Does not contain
Is set
Is not set
Starts with
Is equal to
Is not equal to
Greater than
Lower than
Between

... and then in custom filters, these are available for dates:

Before
After

Many of these are self-explanatory ... so I'm not sure how I'd tackle explaning them all.

Would be good to discuss this with you.

Copy link
Contributor Author

@mcsi-odoo mcsi-odoo left a comment

Choose a reason for hiding this comment

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

Thanks for your review, Xavier!

I've replied to some comments here, and started to update the PR ... but I'd appreciate discussing some points in more detail (like using a gif, how to illustrate the various types of global filter, explaining conditional operators, etc.) before going further. I'll check in with you after your time off to set up a short meeting.

from scratch or add suggested global filters <spreadsheet/global-filters/configure>`.

#. For the relevant global filter, select a conditional operator from the first dropdown, e.g.,
*is in*, then, as relevant, enter or select the appropriate value.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Indeed, is in and is not in are not really self-explanatory!

I understand your point about explaining the different operators, but I have some questions:

  • would it make more sense to have such explanations on this page about custom filters?
  • from what I can tell, these are all the operators you can have when applying global filters (different ones available depending on field type)

Is in
Is not in
Contains
Does not contain
Is set
Is not set
Starts with
Is equal to
Is not equal to
Greater than
Lower than
Between

... and then in custom filters, these are available for dates:

Before
After

Many of these are self-explanatory ... so I'm not sure how I'd tackle explaning them all.

Would be good to discuss this with you.

@mcsi-odoo mcsi-odoo force-pushed the 19.0-spreadsheet-global-filters-mcsi branch from a46d31c to f4c67f7 Compare March 16, 2026 10:30
@mcsi-odoo
Copy link
Contributor Author

Hi @xpl-odoo, the main changes in this version are:

  • use of a gif in the intro section
  • 'matches with' changed to 'matches against', which I feel is more commonly used
  • some wording changes in the examples of each type of global filter
  • better consistency in use of literal, italics, guilabel, etc. ... I hope!

@mcsi-odoo mcsi-odoo requested a review from xpl-odoo March 16, 2026 13:18
Copy link
Contributor

@xpl-odoo xpl-odoo left a comment

Choose a reason for hiding this comment

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

Thanks for addressing my suggestions! The GIF works really well, it's worth having it. You can r+ whenever :)

@robodoo delegate+

@mcsi-odoo mcsi-odoo force-pushed the 19.0-spreadsheet-global-filters-mcsi branch from f4c67f7 to cb009ba Compare March 17, 2026 14:50
@mcsi-odoo
Copy link
Contributor Author

@robodoo r+

robodoo pushed a commit that referenced this pull request Mar 18, 2026
task-5881742

closes #16382

Signed-off-by: Siobhan McGonigle (mcsi) <mcsi@odoo.com>
robodoo pushed a commit that referenced this pull request Mar 18, 2026
task-5881742

closes #16382

Signed-off-by: Siobhan McGonigle (mcsi) <mcsi@odoo.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants