Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Incorrect image url with some extensions (p.e. cloudinary extension) #1288

Closed
gonzaloebiz opened this issue Sep 7, 2021 · 0 comments · Fixed by #1289, #1290, #1291, #1292 or #1293
Closed

Incorrect image url with some extensions (p.e. cloudinary extension) #1288

gonzaloebiz opened this issue Sep 7, 2021 · 0 comments · Fixed by #1289, #1290, #1291, #1292 or #1293
Assignees
@gonzaloebiz
Labels
bug done Magento 2.1 Magento 2.2 Magento 2.3 Magento 2.4
Milestone
10x.x.44

Comments

@gonzaloebiz
Copy link
Collaborator

Some image extensions don't fit the Magento way and return the image path without the preceding /, this cause that when we fill the product image to sync the URL is wrong
@gonzaloebiz gonzaloebiz added this to the 10x.x.44 milestone Sep 7, 2021
@gonzaloebiz gonzaloebiz self-assigned this Sep 7, 2021
gonzaloebiz added a commit that referenced this issue Sep 7, 2021
@gonzaloebiz
closes #1288 for magento 2.1
691f6c9
gonzaloebiz added a commit that referenced this issue Sep 7, 2021
@gonzaloebiz
closes #1288 for magento 2.2
ff654e2
gonzaloebiz added a commit that referenced this issue Sep 7, 2021
@gonzaloebiz
closes #1288 for magento 2.3
9445f98
gonzaloebiz added a commit that referenced this issue Sep 7, 2021
@gonzaloebiz
closes #1288 for magento 2.3.6-p1
f2e48d2
gonzaloebiz added a commit that referenced this issue Sep 7, 2021
@gonzaloebiz
closes #1288 for magento 2.4
a192b3f
This was linked to pull requests Sep 7, 2021
closes #1288 for magento 2.1 #1289
Merged
closes #1288 for magento 2.2 #1290
Merged
closes #1288 for magento 2.3 #1291
Merged
closes #1288 for magento 2.3.6-p1 #1292
Merged
closes #1288 for magento 2.4 #1293
Merged
gonzaloebiz added a commit that referenced this issue Sep 7, 2021
@gonzaloebiz
Merge pull request #1289 from mailchimp/Issue1288-2.1
2bdbf1f 
closes #1288 for magento 2.1
gonzaloebiz added a commit that referenced this issue Sep 7, 2021
@gonzaloebiz
Merge pull request #1290 from mailchimp/Issue1288-2.2
efcae5c 
closes #1288 for magento 2.2
gonzaloebiz added a commit that referenced this issue Sep 7, 2021
@gonzaloebiz
Merge pull request #1292 from mailchimp/Issue1288-2.3.6-p1
aaa1480 
closes #1288 for magento 2.3.6-p1
gonzaloebiz added a commit that referenced this issue Sep 7, 2021
@gonzaloebiz
Merge pull request #1293 from mailchimp/Issue1288-2.4
e23e1e9 
closes #1288 for magento 2.4
gonzaloebiz added a commit that referenced this issue Sep 7, 2021
@gonzaloebiz
Merge pull request #1291 from mailchimp/Issue1288-2.3
f259255 
closes #1288 for magento 2.3
gonzaloebiz added a commit that referenced this issue Sep 7, 2021
@gonzaloebiz
closes #1288 for magento 2.1
eeb53c2
gonzaloebiz added a commit that referenced this issue Sep 7, 2021
@gonzaloebiz
closes #1288 for magento 2.2
40bf017
gonzaloebiz added a commit that referenced this issue Sep 7, 2021
@gonzaloebiz
closes #1288 for magento 2.3
6e761d7
gonzaloebiz added a commit that referenced this issue Sep 7, 2021
@gonzaloebiz
closes #1288 for magento 2.3.6-p1
d156611
This was linked to pull requests Sep 7, 2021
closes #1288 for magento 2.1 #1294
Merged
closes #1288 for magento 2.2 #1295
Merged
This was linked to pull requests Sep 7, 2021
closes #1288 for magento 2.3 #1296
Merged
closes #1288 for magento 2.3.6-p1 #1297
Merged
gonzaloebiz added a commit that referenced this issue Sep 7, 2021
@gonzaloebiz
Merge pull request #1294 from mailchimp/Issue1288-2.1
46a3b14 
closes #1288 for magento 2.1
gonzaloebiz added a commit that referenced this issue Sep 7, 2021
@gonzaloebiz
Merge pull request #1295 from mailchimp/Issue1288-2.2
7e5c682 
closes #1288 for magento 2.2
gonzaloebiz added a commit that referenced this issue Sep 7, 2021
@gonzaloebiz
Merge pull request #1296 from mailchimp/Issue1288-2.3
09ad414 
closes #1288 for magento 2.3
gonzaloebiz added a commit that referenced this issue Sep 7, 2021
@gonzaloebiz
Merge pull request #1297 from mailchimp/Issue1288-2.3.6-p1
04c6a8d 
closes #1288 for magento 2.3.6-p1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gonzaloebiz gonzaloebiz

Labels
bug done Magento 2.1 Magento 2.2 Magento 2.3 Magento 2.4
Projects
None yet
Milestone
10x.x.44
1 participant
@gonzaloebiz