Skip to content

[18.0][FIX] website_altcha: Add sitemap value for /altcha route#1204

Merged
OCA-git-bot merged 1 commit into
OCA:18.0from
tarteo:18-altcha-sitemap
Jul 8, 2026
Merged

[18.0][FIX] website_altcha: Add sitemap value for /altcha route#1204
OCA-git-bot merged 1 commit into
OCA:18.0from
tarteo:18-altcha-sitemap

Conversation

@tarteo

@tarteo tarteo commented Jul 3, 2026

Copy link
Copy Markdown
Member

Small change to remove this warning:

2026-07-03 11:09:30,020 351 WARNING odoo odoo.addons.website.models.website: No Sitemap value provided for controller <function AltchaController.generate_altcha_challenge at 0x7f7a28b05870> (/altcha) 

@tarteo tarteo changed the title [FIX] website_altcha: Add sitemap value for /altcha route [18.0][FIX] website_altcha: Add sitemap value for /altcha route Jul 3, 2026

@ByteMeAsap ByteMeAsap 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.

LGTM!!

@hbrunn hbrunn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

your review on #1203 will be welcome

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@hbrunn

hbrunn commented Jul 8, 2026

Copy link
Copy Markdown
Member

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-1204-by-hbrunn-bump-patch, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit bf9c05e into OCA:18.0 Jul 8, 2026
7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 9f96cd4. Thanks a lot for contributing to OCA. ❤️

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants