Skip to content

Use carrierwave remote url seeding instead of our own#23198

Open
oliverguenther wants to merge 1 commit into
release/17.4from
chore/remote-url-seeding
Open

Use carrierwave remote url seeding instead of our own#23198
oliverguenther wants to merge 1 commit into
release/17.4from
chore/remote-url-seeding

Conversation

@oliverguenther
Copy link
Copy Markdown
Member

Copy link
Copy Markdown
Contributor

@dfriquet dfriquet left a comment

Choose a reason for hiding this comment

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

I think a configuration is missing, and maybe a test that stubs a failure and retry if that’s an important expected improvement from this WP.

And somehow, CustomDesignSeeder tests are now failing despite the existing request stubs.

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.

The doc states that one must enable retries and it seems it’s not configured yet for this project.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants