Fix import wizard

[stsewd]

I think this fixes it (at least makes .com tests green), but we don't have tests for this, so not sure if it doesn't break anything else. @humitos you may want to test it locally.

ref https://read-the-docs.sentry.io/issues/6127455073/?project=161479&query=is%3Aunresolved&referrer=issue-stream&sort=trends&statsPeriod=90d&stream_index=3

[humitos]

It works fine on community, but on commercial it fails with

RecursionError at /dashboard/import/manual/
maximum recursion depth exceeded

It seems that wizard.get_step_index() calls self.steps.current, which calls again show_config_step 😢

[stsewd]

It works fine on community, but on commercial it fails with

RecursionError at /dashboard/import/manual/
maximum recursion depth exceeded

It seems that wizard.get_step_index() calls self.steps.current, which calls again show_config_step 😢

I wasn't able to replicate that in tests, did you rebuild your docker image with to use the latest version of the dependency?

[humitos]

I wasn't able to replicate that in tests

Are you able to reproduce this by importing a project?

did you rebuild your docker image with to use the latest version of the dependency?

Yes.

[humitos]

I pushed a commit here that combined with https://github.com/readthedocs/readthedocs-corporate/pull/1920 make the wizard import to work locally.

I feel that the condition dict option is completely broken, tho and we should probably consider to go back to my original hack 😄

[stsewd]

Are you able to reproduce this by importing a project?

I was able to manually import a project before my computer's battery died, but didn't get to test the import from a remote repo part.

[humitos]

Let's try this solution and see how it goes.