Skip to content

[17.0][MIG] auth_oauth_environment: Migration to 17.0#254

Open
vvrossem wants to merge 5 commits into
OCA:17.0from
camptocamp:17.0-mig-auth_oauth_environment
Open

[17.0][MIG] auth_oauth_environment: Migration to 17.0#254
vvrossem wants to merge 5 commits into
OCA:17.0from
camptocamp:17.0-mig-auth_oauth_environment

Conversation

@vvrossem

@vvrossem vvrossem commented Nov 19, 2025

Copy link
Copy Markdown
Contributor

supersedes #253

@vvrossem vvrossem mentioned this pull request Nov 19, 2025
Closed
@vvrossem vvrossem force-pushed the 17.0-mig-auth_oauth_environment branch from 295836a to dd7e861 Compare November 19, 2025 13:59
yankinmax
yankinmax reviewed Nov 21, 2025
View reviewed changes
Comment thread auth_oauth_environment/README.rst
To configure this module, you need to add a section
``[auth_oauth.provider_simple_name]`` to you server_environment_files
configurations, where 'provider_simple_name' match the simplified name field on
``[auth_oauth.provider_name]`` to you server_environment_files

@yankinmax yankinmax Nov 21, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

to your server_environment_files

yankinmax
yankinmax approved these changes Nov 21, 2025
View reviewed changes

@yankinmax yankinmax left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for adding the test. As we're migrationg form v11 probably it worth it to rename openerp to odoo?

@vvrossem

vvrossem commented Nov 21, 2025

Copy link
Copy Markdown
Contributor Author

Thanks for adding the test. As we're migrationg form v11 probably it worth it to rename openerp to odoo?

I used for the tech_name the same value as the XML ID auth_oauth.provider_openerp, mainly for consistency

@gurneyalex

gurneyalex commented Jun 26, 2026

Copy link
Copy Markdown
Member

/ocabot merge nobump

@OCA-git-bot

OCA-git-bot commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 17.0-ocabot-merge-pr-254-by-gurneyalex-bump-nobump, awaiting test results.

@OCA-git-bot

OCA-git-bot commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

@gurneyalex The merge process could not be finalized, because command twine upload --disable-progress-bar --non-interactive --repository-url https://upload.pypi.org/legacy/ -u __token__ odoo_addon_auth_oauth_environment-17.0.1.0.0.2-py3-none-any.whl failed with output:

Uploading distributions to https://upload.pypi.org/legacy/
Uploading odoo_addon_auth_oauth_environment-17.0.1.0.0.2-py3-none-any.whl
�[33mWARNING �[0m Error during upload. Retry with the --verbose option for more details. 
�[31mERROR   �[0m HTTPError: 429 Too Many Requests from https://upload.pypi.org/legacy/  
         Too Many Requests

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gurneyalex gurneyalex gurneyalex approved these changes
+2 more reviewers
@ivantodorovich ivantodorovich ivantodorovich approved these changes
@yankinmax yankinmax yankinmax approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@vvrossem @gurneyalex @OCA-git-bot @ivantodorovich @yankinmax @grindtildeath @yvaucher