Skip to content

[18.0][ADD] l10n_fr_einvoicing: neutralize script - #57

Open
remi-filament wants to merge 2 commits into
akretion:18.0from
lefilament:18.0-l10n_fr_einvocing-neutralize
Open

[18.0][ADD] l10n_fr_einvoicing: neutralize script#57
remi-filament wants to merge 2 commits into
akretion:18.0from
lefilament:18.0-l10n_fr_einvocing-neutralize

Conversation

@remi-filament

Copy link
Copy Markdown
Contributor

Fixes #54

@ak-git-bot

Copy link
Copy Markdown

Hi @alexis-via,
some modules you are maintaining are being modified, check this out!

@ak-git-bot ak-git-bot added mod:l10n_fr_einvoicing Module l10n_fr_einvoicing series:18.0 labels Aug 19, 2026
@remi-filament
remi-filament force-pushed the 18.0-l10n_fr_einvocing-neutralize branch from 4857720 to db3616f Compare August 20, 2026 07:40
@risto42

risto42 commented Aug 20, 2026

Copy link
Copy Markdown

Thanks for implementing this! Two issues in neutralize.sql, both verified on a real database:

  1. fr_ctc_secret_id does not exist — the column is fr_ctc_client_secret, so the script fails outright.
  2. SET … = False on varchar columns: PostgreSQL casts the boolean to the string 'false', which is truthy on the ORM side — fr_ctc_accredited_platform would still look configured after neutralization. Use = NULL instead.

@remi-filament

Copy link
Copy Markdown
Contributor Author

Hi @risto42 thanks for double checking, it was too late yesterday I mixed things up !
Both points should be fixed.
Best Regards !

@remi-filament remi-filament changed the title [ADD] l10n_fr_einvoicing: neutralize script [18.0][ADD] l10n_fr_einvoicing: neutralize script Aug 20, 2026
@remi-filament
remi-filament force-pushed the 18.0-l10n_fr_einvocing-neutralize branch from 9228676 to fe21f0e Compare August 21, 2026 05:21
@remi-filament
remi-filament force-pushed the 18.0-l10n_fr_einvocing-neutralize branch 2 times, most recently from 82421c5 to 0388426 Compare September 3, 2026 07:21
@remi-filament
remi-filament force-pushed the 18.0-l10n_fr_einvocing-neutralize branch from 0388426 to b93175d Compare September 8, 2026 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mod:l10n_fr_einvoicing Module l10n_fr_einvoicing series:18.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

l10n_fr_einvoicing: add data/neutralize.sql — a production dump restored on a test database keeps live credentials and active crons

3 participants