Giter Site home page Giter Site logo

Comments (20)

celm1990 avatar celm1990 commented on July 2, 2024 3

It would be better to have them available for OCA days, as there a lot of module migrations happen, although it depends if Odoo have a 17.0 branch already available at odoo/odoo.

Branch is here now https://github.com/odoo/odoo/tree/17.0 ;)

from maintainer-tools.

pedrobaeza avatar pedrobaeza commented on July 2, 2024 2

I will do the OCB and OpenUpgrade stuff.

from maintainer-tools.

sbidoul avatar sbidoul commented on July 2, 2024 1

how much of a difference is there in the file changes when using pre-commit with ruff?

Normally not much or none at all as it is possible to configure ruff to do the same as isort+flake8.
I'll see if I can find the time to setup an example on some repo.

from maintainer-tools.

sbidoul avatar sbidoul commented on July 2, 2024 1

Great! Anyone planning to work on some the above tasks, please announce yourself to avoid duplicate work.

from maintainer-tools.

gurneyalex avatar gurneyalex commented on July 2, 2024 1

Do we have something about the XML declaration in static/description/index.html?

see OCA/partner-contact#1625 and possibly many other.

from maintainer-tools.

sbidoul avatar sbidoul commented on July 2, 2024 1

It's a TODO, now tracked in #595. It's unlikely that I will have time for that in the coming weeks, so PR welcome :)

from maintainer-tools.

celm1990 avatar celm1990 commented on July 2, 2024

For click-odoo-contrib is need adapt code according this commit odoo/odoo#68491

https://github.com/acsone/click-odoo-contrib/blob/1f2c6323b85520007f8cfbec16d6147a25d5d349/click_odoo_contrib/_dbutils.py#L20

When branch for 17.0 are available? I ask this for Doodba adaptations so

from maintainer-tools.

sbidoul avatar sbidoul commented on July 2, 2024

@celm1990 thanks for the hint about click-odoo. Would you like to do a PR there?

When branch for 17.0 are available? I ask this for Doodba adaptations so

We'll likely create the OCA 17.0 branches in the next couple of week after Odoo has released opened the 17.0 branch, depending on volunteers availability.

from maintainer-tools.

pedrobaeza avatar pedrobaeza commented on July 2, 2024

It would be better to have them available for OCA days, as there a lot of module migrations happen, although it depends if Odoo have a 17.0 branch already available at odoo/odoo.

from maintainer-tools.

celm1990 avatar celm1990 commented on July 2, 2024

Use ruff

If the community agrees, we could use ruff to improve pre-commit run performance:

  • Convert our pre-commit config to replace flake8, isort and pyupgrade with ruff. A
    .ruff.toml at the repo root can replace .flake8 and .isort.cfg.

Just out of curiosity, how much of a difference is there in the file changes when using pre-commit with ruff?

from maintainer-tools.

pedrobaeza avatar pedrobaeza commented on July 2, 2024

OCA branch 17.0 is now pushed (it needs to be done in several push batches, as GitHub limits the index of one push to 2 GB, and this exceeds such size). @sbidoul please put it as default branch and after merging the ocb sync change, deploy it in the server.

from maintainer-tools.

sbidoul avatar sbidoul commented on July 2, 2024

Ok, good progress so far.

Main next step now is to update the addon repo template, and particularly review the hook versions and pylint-odoo config. Help welcome.

from maintainer-tools.

bosd avatar bosd commented on July 2, 2024

The addon icon menu generator is ready for merge.
However, only icons for versions pre 17 are supported.
Where is a good place to start the discussion about generating v17 community icons?

Can be quite a challenge to switch something which is compatible with the new odoo *milk theme.
Sticking to the old pre V17 icons is not an good option either because of the huge style mismatch.

Is there already some discussion about this in the community?

from maintainer-tools.

pedrobaeza avatar pedrobaeza commented on July 2, 2024

Not yet. About icons, we are using https://ivantodorovich.github.io/odoo-icon/ as a versatile one. I have told @ivantodorovich about transferring it to OCA.

from maintainer-tools.

ivantodorovich avatar ivantodorovich commented on July 2, 2024

Not yet. About icons, we are using https://ivantodorovich.github.io/odoo-icon/ as a versatile one. I have told @ivantodorovich about transferring it to OCA.

Hey! I'm happy to transfer it to the OCA. I might need some assistance with the process, though 😓


About v17 icons, I don't think it'll be possible to have a tool for this. It's not like you can pick a fontawesome glyph and add a background color.. the new style, for better and for worse, requires a little bit of graphics design skills.

The good news is that after you get that done, there's no tedious repetitive work to do. Unlike the previous versions, the icon card and background is handled by the web client with css, so the app icon really just is a picture with transparent background, like this:

account

That means tools like https://ivantodorovich.github.io/odoo-icon/ won't really be needed. There's no gradient background, no shadow.. nothing to handle, really

from maintainer-tools.

bosd avatar bosd commented on July 2, 2024

It's not like you can pick a fontawesome glyph and add a background color.. the new style, for better and for worse, requires a little bit of graphics design skills.

Considered the options IMO this is still the way to go. Pick an multi-color icon font, use an icon generator tool to adjust it to the colors used in the new menu.

Other options:

  1. Keep using pre-17 Icons. --> Huge style mismatch 😞
  2. Expect a programmer to have time and skills to create an icon which matches the new Milk theme. 🤔
  3. Do nothing, (Almost the same as point 2 🙊 )

There are some critics of the new Milk design. That the icons are to hard to interpret.
While switching to an multicolor icon font. We have the posibilitie to still keep the clarity of the functions. Making the colors (programmically) match the Milk theme, will prevent a huge style clash.
As example. an candidate could be core flat icon pack.

from maintainer-tools.

sbidoul avatar sbidoul commented on July 2, 2024

I'm adding the conversion of fragments to markdown in OCA/oca-addons-repo-template#228

from maintainer-tools.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.