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

[14.0][IMP] l10n_br_fiscal: imp document supplement #3446

Open
wants to merge 3 commits into
base: 14.0
Choose a base branch
from

Conversation

marcelsavegnago
Copy link
Member

@marcelsavegnago marcelsavegnago commented Oct 16, 2024

This pull request introduces a new model for document supplements in the l10n_br_fiscal module and integrates it with the existing NFe supplement model. The most important changes include adding a new import in the __init__.py file, creating the DocumentSupplement model, and updating the NFeSupplement model to inherit from the new model.

New model introduction:

Integration with existing models:

@OCA-git-bot
Copy link
Contributor

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

Copy link
Member

@mileo mileo left a comment

Choose a reason for hiding this comment

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

    nfe40_qrCode = fields.Char(
        string="Texto com o QR-Code impresso",
        xsd_required=True,
        help="Texto com o QR-Code impresso no DANFE NFC-e",
    )

    nfe40_urlChave = fields.Char(
        string="URL da 'Consulta por chave de acesso",
        xsd_required=True,
        help=(
            "URL da 'Consulta por chave de acesso da NFC-e'. A mesma URL que "
            "deve estar informada no DANFE NFC-e para consulta por chave de "
            "acesso."
        ),
    )

    def _prepare_nfce_send(self):
        self.ensure_one()
        self._prepare_payments_for_nfce()
        self.nfe40_infNFeSupl = self.env["l10n_br_fiscal.document.supplement"].create(
            {
                "nfe40_qrCode": self.get_nfce_qrcode(),
                "nfe40_urlChave": self.get_nfce_qrcode_url(),
            }
        )

Pode transformar esses campos em related?

@marcelsavegnago
Copy link
Member Author

@mileo done.. dá uma confirmada por favor

@mileo
Copy link
Member

mileo commented Oct 16, 2024

Mas precisa de um migration para não perder esses dados, pois o related vai remover, pode providenciar?

@mileo
Copy link
Member

mileo commented Oct 16, 2024

Mas precisa de um migration para não perder esses dados, pois o related vai remover, pode providenciar?

Pode ser via SQL

@mileo
Copy link
Member

mileo commented Oct 16, 2024

@marcelsavegnago isso é por conta do CT-e e MDF-e né?

@marcelsavegnago
Copy link
Member Author

Mas precisa de um migration para não perder esses dados, pois o related vai remover, pode providenciar?

Se puder assumir este script eu agradeço :D

@mileo
Copy link
Member

mileo commented Oct 16, 2024

Mas precisa de um migration para não perder esses dados, pois o related vai remover, pode providenciar?

Se puder assumir este script eu agradeço :D

Não vejo como o seu PR entrar sem o script, vocês não podem incluir já no PR para evitar perda de dados.

segue uma ideia: https://chatgpt.com/share/670ff194-4578-800b-8db3-f1c58a015b7e

@marcelsavegnago
Copy link
Member Author

marcelsavegnago commented Oct 16, 2024

Mas precisa de um migration para não perder esses dados, pois o related vai remover, pode providenciar?

Se puder assumir este script eu agradeço :D

Não vejo como o seu PR entrar sem o script, vocês não podem incluir já no PR para evitar perda de dados.

segue uma ideia: https://chatgpt.com/share/670ff194-4578-800b-8db3-f1c58a015b7e

Soh perguntei :D.... como vc ofereceu ajuda, um commit ai no meu fork já resolvia.. mas faço aqui.. vlwww

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.

4 participants