OCA / account-analytic

Odoo Account Analytic Related Addons
GNU Affero General Public License v3.0
101 stars 358 forks source link

[15.0][FIX] account_move_update_analytic: Switch button text to icon #593

Closed Shide closed 11 months ago

Shide commented 12 months ago

Button text is super large and doesn't fit well on any screen. Also is pretty ugly

Replace the button text with a pretty icon fa-tags and add a proper title to know what is going to happen if you click the button. Also added maintainers

I pretend to FWP this commit into 16.0 once it's merged.

MT-3838 @moduon @yajo @rafaelbn @EmilioPascual @remi-filament plz review if you want 😄

EmilioPascual commented 12 months ago

LGTM

remi-filament commented 11 months ago

I pretend to FWP this commit into 16.0 once it's merged.

I do not think this module is necessary in v16.0 since you can already update analytic distribution !

OCA-git-bot commented 11 months ago

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). 🤖

pedrobaeza commented 11 months ago

/ocabot merge minor

OCA-git-bot commented 11 months ago

This PR looks fantastic, let's merge it! Prepared branch 15.0-ocabot-merge-pr-593-by-pedrobaeza-bump-minor, awaiting test results.

OCA-git-bot commented 11 months ago

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