Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ADD] registroamostras: add new module #52

Closed
wants to merge 1 commit into from

Conversation

DigoRameh
Copy link

No description provided.

@@ -9,10 +9,11 @@

/*
:Author: David Goodger ([email protected])
:Id: $Id: html4css1.css 8954 2022-01-20 10:10:25Z milde $
:Id: $Id: html4css1.css 9511 2024-01-13 09:50:07Z milde $
Copy link
Contributor

@kaynnan kaynnan Apr 26, 2024

Choose a reason for hiding this comment

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

Foi incluído outros módulos que não deveriam estar ali, sempre quando for fazer uma PR, roda os seguintes comandos para manter seu epa-addons atualizado:

git checkout 14.0
git fetch origin
git rebase origin/14.0

E depois para nova branch:

git checkout -b nomebranchnova origin/14.0

E na hora de realizar o git add, inclua somente o módulo em que está sendo desenvolvido, nesse caso foi o registroamostrar_custom, então inclua somente ele e a pasta setup/registroamostrar_custom

Copy link
Contributor

@kaynnan kaynnan Apr 26, 2024

Choose a reason for hiding this comment

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

Nomenclatura do módulo, se ele é um módulo novo ele deve conter o prefixo do epa_nomemodulo, nesse caso ficou a entender que você estava customizando algum módulo existente para atender uma necessidade, caso for um módulo novo mantenha o prefixo epa_nomemodulo somente.

Copy link
Contributor

Choose a reason for hiding this comment

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

Pelo o que compreendi o módulo é um novo desenvolvimento mesmo, sendo assim, manter sempre o padrão do módulo em ingles, exemplo de como poderia ser:

from odoo import fields, models

class Laboratories(models.Model):
    _name = "laboratories.laboratory"

    name = fields.Char("Laboratory Name", required=True)

    def name_get(self, context=None):
        if context is None:
            context = {}
        result = []
        if context.get("custom_display_name", "labnames"):
            for rec in self:
                result.append((rec.id, rec.name))
            return result
        return result

Fazer essa correção para os demais arquivos, tanto views quanto models.

Observação: nome do arquivo deve ser também em ingles

Copy link
Contributor

@kaynnan kaynnan left a comment

Choose a reason for hiding this comment

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

O módulo deve conter um padrão a ser seguido, ao desenvolver o módulo utilize esse comando do bobtemplates, você pode fazer a instalaçao dele seguindo o guia https://github.com/acsone/bobtemplates.odoo

Nesse link você tem um guia mais detalhado de como criar um novo módulo ou quando for criar algum custom de um módulo já existente, nessa situação ele já monta um padrão a ser seguido pelo Odoo.

Quando for rodar o comando:

mrbob bobtemplates.odoo:addon

Ele vai te dar algumas informações como qual vai ser o nome do módulo, já para o comando com o model, ele te oferece a possibilidade de especificar se é um modelo novo ou uma herança "inherit" e também já mantém um padrão para criar menus, views, regras de segurança entre outros pontos.

Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale label Aug 25, 2024
@kaynnan
Copy link
Contributor

kaynnan commented Aug 26, 2024

Closing this PR, as its continuity is in PR65

@kaynnan kaynnan closed this Aug 26, 2024
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