Skip to content

[19.0][IMP] base_tier_validation_correction: UI polish#27

Open
bosd wants to merge 2 commits into
OCA:19.0from
bosd:19.0-imp-base_tier_validation_correction-ui
Open

[19.0][IMP] base_tier_validation_correction: UI polish#27
bosd wants to merge 2 commits into
OCA:19.0from
bosd:19.0-imp-base_tier_validation_correction-ui

Conversation

@bosd
Copy link
Copy Markdown
Contributor

@bosd bosd commented May 13, 2026

Summary

UI improvements for the tier.correction views.

  • Apply many2many_avatar_user to old_reviewer_ids / new_reviewer_ids (form view, items inline list and form).
  • Apply many2one_avatar_user to create_uid (form view and list view).
  • Add the Model column to the list view, plus create_uid (avatar) and create_date.
  • Search view:
    • Add model_id searchable field.
    • Add My Corrections filter (create_uid = uid).
    • Add per-state filters: Draft / Preparing / Corrected / Cancelled / Reverted.
    • Add Status groupby and set it as the default via the action context.
  • Fix bug: the search view declared two <filter name="correction_type">. The duplicate name caused the Model groupby to shadow the Correction Type one. Renamed to group_state / group_correction_type / group_model.

Test plan

  • Open Tier Review Correction: list view groups by Status by default and shows Model, Created by (avatar) and Creation Date.
  • Open a record: reviewer fields show user avatars in both the main form groups and the Correction Details inline list.
  • In the search view, both "Correction Type" and "Model" groupby filters work independently.
  • The "My Corrections" filter restricts to records created by the current user.
  • The per-state filters each show the matching records.

…iews

- Apply many2many_avatar_user / many2one_avatar_user on user fields
  (old/new reviewers in form + items, create_uid in form and list).
- Add Model and creation columns to the list view.
- Search view: add model_id field, "My Corrections" filter, per-state
  filters, and a Status groupby; default the action to group by Status.
- Fix duplicate <filter name="correction_type"> in the search view that
  silently shadowed the Correction Type groupby with the Model one.
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Hi @kittiu,
some modules you are maintaining are being modified, check this out!

@OCA-git-bot OCA-git-bot added series:19.0 mod:base_tier_validation_correction Module base_tier_validation_correction labels May 13, 2026
… classes

- Fix button label typo: "Make Correcton" -> "Make Correction".
- Fix the Cancelled badge: decoration-danger="state in 'cancel'" was a
  substring match against the literal "cancel" -- it accidentally
  matched 'c', 'a', 'n', 'e', 'l' if state were ever a single char.
  Use decoration-danger="state in ['cancel', 'revert']" so both
  terminal states get the danger badge (instead of the Reverted state
  having no decoration at all).
- Replace legacy classes:
    - oe_grey  -> text-muted   (Bootstrap 5 muted-text equivalent;
      oe_grey is no longer documented as a public Odoo helper).
    - oe_edit_only "Name" hint above the title: drop entirely; the
      <field placeholder="..."> already covers the same UX in edit
      mode, and the span left a stray "Name" label visible in v19.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mod:base_tier_validation_correction Module base_tier_validation_correction series:19.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants