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.

Sign up for GitHub

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

Jump to bottom

[IMP] l10n_ar_txt_tucuman: new module #461

Closed
wants to merge 1 commit into from
Closed

[IMP] l10n_ar_txt_tucuman: new module #461

wants to merge 1 commit into from

Conversation

pablohmontenegro
Copy link
Contributor

Tarea: 38200

@roboadhoc
Copy link
Contributor

Pull request status dashboard.

@pablohmontenegro pablohmontenegro force-pushed the 16.0-t-38200-pam branch 7 times, most recently from 4981ea6 to 59246bf Compare October 21, 2024 20:14
@pablohmontenegro pablohmontenegro force-pushed the 16.0-t-38200-pam branch 12 times, most recently from d06310a to e28b14f Compare November 4, 2024 12:24
pablohmontenegro
pablohmontenegro commented Nov 11, 2024
View reviewed changes
l10n_ar_txt_tucuman/README.rst Outdated Show resolved Hide resolved
l10n_ar_txt_tucuman/__manifest__.py Outdated Show resolved Hide resolved
l10n_ar_txt_tucuman/__manifest__.py Outdated Show resolved Hide resolved
l10n_ar_txt_tucuman/models/account_journal.py Outdated Show resolved Hide resolved
l10n_ar_txt_tucuman/models/account_journal.py Outdated Show resolved Hide resolved
l10n_ar_txt_tucuman/models/account_journal.py Outdated Show resolved Hide resolved
l10n_ar_txt_tucuman/models/account_journal.py Outdated Show resolved Hide resolved
l10n_ar_txt_tucuman/models/account_journal.py Outdated Show resolved Hide resolved
l10n_ar_txt_tucuman/models/account_journal.py Show resolved Hide resolved
l10n_ar_txt_tucuman/models/account_journal.py Outdated Show resolved Hide resolved
@pablohmontenegro pablohmontenegro force-pushed the 16.0-t-38200-pam branch 7 times, most recently from a2f3676 to 97ffcae Compare November 12, 2024 11:52
zaoral
zaoral suggested changes Nov 12, 2024
View reviewed changes
l10n_ar_txt_tucuman/models/account_journal.py Outdated Show resolved Hide resolved
l10n_ar_txt_tucuman/models/account_journal.py Outdated Show resolved Hide resolved
l10n_ar_txt_tucuman/models/account_journal.py Show resolved Hide resolved
l10n_ar_txt_tucuman/models/account_journal.py Outdated Show resolved Hide resolved
l10n_ar_txt_tucuman/models/account_journal.py Outdated Show resolved Hide resolved
l10n_ar_txt_tucuman/models/account_journal.py Outdated Show resolved Hide resolved
l10n_ar_txt_tucuman/models/account_journal.py Outdated Show resolved Hide resolved
l10n_ar_txt_tucuman/models/account_journal.py Show resolved Hide resolved
zaoral
zaoral suggested changes Nov 12, 2024
View reviewed changes
l10n_ar_txt_tucuman/models/account_journal.py Outdated
content_datos, content_retper, content_ncfact = '', '', ''

# VALIDACIONES
percepciones = self._iibb_tucuman_validations(move_lines)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
percepciones = self._iibb_tucuman_validations(move_lines)
self._iibb_tucuman_validations(move_lines)

l10n_ar_txt_tucuman/models/account_journal.py Outdated
""" Implementado segun especificación indicada en tarea 38200.
Ver especificación también en l10n_ar_txt_tucuman/doc/MRETPER6R2.pdf a partir de la página 12. """
self.ensure_one()
content_datos, content_retper, content_ncfact = '', '', ''
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
content_datos, content_retper, content_ncfact = '', '', ''

@zaoral
Copy link
Contributor

zaoral commented Nov 12, 2024

@roboadhoc r+

roboadhoc pushed a commit that referenced this pull request Nov 12, 2024
@pablohmontenegro
[IMP] l10n_ar_txt_tucuman: new module
85d4845 
closes #461

Tarea: 38200
Signed-off-by: Katherine Zaoral <kz@adhoc.com.ar>
@roboadhoc roboadhoc closed this Nov 12, 2024
@fw-bot-adhoc fw-bot-adhoc mentioned this pull request Nov 12, 2024
Closed
@fw-bot-adhoc
Copy link
Contributor

@pablohmontenegro @zaoral this pull request has forward-port PRs awaiting action (not merged or closed):

7 similar comments
@fw-bot-adhoc
Copy link
Contributor

@pablohmontenegro @zaoral this pull request has forward-port PRs awaiting action (not merged or closed):

@fw-bot-adhoc
Copy link
Contributor

@pablohmontenegro @zaoral this pull request has forward-port PRs awaiting action (not merged or closed):

@fw-bot-adhoc
Copy link
Contributor

@pablohmontenegro @zaoral this pull request has forward-port PRs awaiting action (not merged or closed):

@fw-bot-adhoc
Copy link
Contributor

@pablohmontenegro @zaoral this pull request has forward-port PRs awaiting action (not merged or closed):

@fw-bot-adhoc
Copy link
Contributor

@pablohmontenegro @zaoral this pull request has forward-port PRs awaiting action (not merged or closed):

@fw-bot-adhoc
Copy link
Contributor

@pablohmontenegro @zaoral this pull request has forward-port PRs awaiting action (not merged or closed):

@fw-bot-adhoc
Copy link
Contributor

@pablohmontenegro @zaoral this pull request has forward-port PRs awaiting action (not merged or closed):

@zaoral zaoral deleted the 16.0-t-38200-pam branch November 25, 2024 15:42
@fw-bot-adhoc
Copy link
Contributor

@pablohmontenegro @zaoral this pull request has forward-port PRs awaiting action (not merged or closed):

2 similar comments
@fw-bot-adhoc
Copy link
Contributor

@pablohmontenegro @zaoral this pull request has forward-port PRs awaiting action (not merged or closed):

@fw-bot-adhoc
Copy link
Contributor

@pablohmontenegro @zaoral this pull request has forward-port PRs awaiting action (not merged or closed):

@fw-bot-adhoc fw-bot-adhoc mentioned this pull request Jan 23, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zaoral zaoral zaoral approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@pablohmontenegro @roboadhoc @zaoral @fw-bot-adhoc