connector-interfaces icon indicating copy to clipboard operation
connector-interfaces copied to clipboard

[18.0][MIG] connector_importer_source_sftp: Migration to 18.0

Open yankinmax opened this issue 1 month ago • 3 comments

yankinmax avatar Oct 24 '25 10:10 yankinmax

Hello @simahawk @sebalix @SilvioC2C can you pls help with any suggestions? I'm not familiar with the module usage. My debug session shows that filelist = self.list(relative_path) is empty, so no files are really found. https://github.com/OCA/storage/blob/5bbfe12c01e68e562922838aa5fd0eb28dc235a0/storage_backend/components/base_adapter.py#L32

yankinmax avatar Oct 29 '25 17:10 yankinmax

Hello @simahawk @sebalix @SilvioC2C can you pls help with any suggestions? I'm not familiar with the module usage. My debug session shows that filelist = self.list(relative_path) is empty, so no files are really found. https://github.com/OCA/storage/blob/5bbfe12c01e68e562922838aa5fd0eb28dc235a0/storage_backend/components/base_adapter.py#L32

So, the issue is definitely with the _selection_source_ref_id override to include import.source.csv.sftp source. It's not called somehow :cry:

https://github.com/OCA/connector-interfaces/pull/172/files#r2476825551

yankinmax avatar Oct 30 '25 07:10 yankinmax

Thanks for help @ivantodorovich :pray:

yankinmax avatar Nov 06 '25 15:11 yankinmax

@simahawk this one's ready too

ivantodorovich avatar Dec 02 '25 13:12 ivantodorovich

/ocabot merge nobump

simahawk avatar Dec 02 '25 14:12 simahawk

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 18.0-ocabot-merge-pr-172-by-simahawk-bump-nobump, awaiting test results.

OCA-git-bot avatar Dec 02 '25 14:12 OCA-git-bot

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

OCA-git-bot avatar Dec 02 '25 14:12 OCA-git-bot