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

[FIX] point_of_sale,retentions_basic,perceptions_basic #10

Merged
merged 1 commit into from
Aug 19, 2015

Conversation

ssaid
Copy link

@ssaid ssaid commented Aug 18, 2015

En los xml de data, agrego el forcecreate="0" para todos los registros.
Es necesario para que en la migracion cuando tiro -u all no me tire constraint error.

En los xml de data, agrego el forcecreate="0" para todos los registros.
Es necesario para que en la migracion cuando tiro -u all no me tire constraint error.

Conflicts:
@sebastiken
Copy link

Ok, @ssaid . Sería buena idea tal vez, que estos datos de configuración estén en otro módulo, ya que, si uno no utiliza el Plan de Cuentas de este repo, este módulo falla al querer insertar los datos de Percepciones. Esto es porque busca cuentas particulares de ese Plan de Cuentas.
Ese xml con datos preconfigurados se hizo para tener ya varias percepciones cargadas de ejemplo que funcionan y se pueden utilizar sin tener que hacer todo un trabajo manual de configuración.
El cambio que proponés acá, corrige el problema que ocurre al borrar una de estas configuraciones de percepción y luego actualizar el módulo.

sebastiken added a commit that referenced this pull request Aug 19, 2015
[FIX] point_of_sale,retentions_basic,perceptions_basic
@sebastiken sebastiken merged commit 4a61ec7 into OCA:8.0 Aug 19, 2015
@ssaid
Copy link
Author

ssaid commented Aug 20, 2015

@sebastiken me parece correcta tu propuesta, te parece organizarlo asi?

  1. l10n_ar_perceptions_basic_data: requires l10n_ar_perceptions_basic, l10n_ar_chart_of_account
  2. l10n_ar_retentions_basic_data: requires l10n_ar_retentions_basic, l10n_ar_chart_of_account
    En l10n_ar_point_of_sale dejarlo como esta.

@sebastiken
Copy link

@ssaid me parece bien lo que proponés, ya que de esta manera, si tenés instalado el módulo l10n_ar_chart_of_account, ese módulo de datos va a funcionar correctamente.
Perfecto, si lo hacés y completás este PR lo mergeamos.
Gracias.

gabrielo77 referenced this pull request in gabrielo77/l10n-argentina Apr 19, 2018
[RM] Se elimina funcion que modificaba el tax_amount
gabrielo77 referenced this pull request in gabrielo77/l10n-argentina Jan 12, 2019
aisola2001 pushed a commit to aisola2001/l10n-argentina that referenced this pull request Sep 15, 2020
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.

2 participants