Skip to content

Comments

[18.0] [MIG] website_oca_integrator#7

Open
Kiplangatdan wants to merge 30 commits into18.0from
18.0-mig-website-oca-integrator
Open

[18.0] [MIG] website_oca_integrator#7
Kiplangatdan wants to merge 30 commits into18.0from
18.0-mig-website-oca-integrator

Conversation

@Kiplangatdan
Copy link

No description provided.

st-aprajapati and others added 25 commits February 11, 2026 12:24
Remove unneeded, failing, step, move tours to demo data and lint the file.
They should be date Python objects
…tains non ascii characters

we are getting a lot of ugly searches that create some kind of deny of service.

We should improve that later but this give an easy solution for the time being.
@Kiplangatdan Kiplangatdan force-pushed the 18.0-mig-website-oca-integrator branch from 3ac4eb1 to f2790fd Compare February 12, 2026 14:45
@KKamaa KKamaa force-pushed the 18.0-mig-website-oca-integrator branch 4 times, most recently from 0cd5717 to 0cfa6df Compare February 17, 2026 08:10
@KKamaa KKamaa force-pushed the 18.0-mig-website-oca-integrator branch 2 times, most recently from 2f85458 to db98835 Compare February 18, 2026 01:34
@KKamaa KKamaa force-pushed the 18.0-mig-website-oca-integrator branch 3 times, most recently from 57d52f1 to 6b071b7 Compare February 19, 2026 03:10
@KKamaa
Copy link

KKamaa commented Feb 19, 2026

@Kiplangatdan @thomaspaulb please note for this PR the error related to detect unreleased dependencies is due to dependencies that are not coming from a released wheel (wheelhouse/PyPI style) and instead are coming from git URLs / paths set on test-requirements and pyprojecttoml file. The culprits are:

  • account_statement_import_online_wise
  • membership_delegated_partner_line
  • apps_product_creator

So the fix is: keep requirements*.txt “release-only” (normal pip requirements with no URLs). We can fix this officially once we are done with the needed modifications just in case these modules are needed for the mig db testing. The tests are ok, it would have lead to "all green" if it wasn't for the test to detect unreleased dependencies.

@thomaspaulb
Copy link

@KKamaa Right yes, the "Detect unreleased dependencies" test is not a blocking test, it's just meant (I think) as a reminder to maintainers that there are PR's that need review and merge

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.

10 participants