[18.0][IMP] auth_oidc_environment: extra fields

[qgroulard]

Needs OCA/server-env#264

I believe more fields make sense to be configurable by environment.

Typically when you use different SSO for test and production.

[sbidoul]

Does this need a migration or will it preserve existing values when this is deployed without setting the new env options?

[qgroulard]

Does this need a migration or will it preserve existing values when this is deployed without setting the new env options?

AFAIK no migration script is needed, this does nothing while no env var is set.

[sbidoul]

I had the impression that server_environment removes the colums from the database and gets the data from a json field, but I don't know how it works when you add already-used fields to the server env mixin.

[qgroulard]

I'll try to have a look but if this is the case we might have problems here as well: OCA/server-env@3574e86

[qgroulard]

Good catch, the previous values are indeed ignored.

I have opened this PR: OCA/server-env#264

[qgroulard]

@sbidoul pre-commit doesn't like that. What would be the right syntax ?

[sbidoul]

Ah yes, Odoo would not support that either.

So the trick is to do that with a new section in pyproject.toml:

[tool.whool]
depends_override = { "server_environment": "odoo-addon-server_environment ==18.0.*, >18.0.1.0.6" }

[github-actions]

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.