fix: image URL in README.md

[iamdarshshah]

Updated logo image URL

Screenshot:

[codecov]

Codecov Report

Merging #6988 into master will decrease coverage by 6.12%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #6988      +/-   ##
==========================================
- Coverage   26.18%   20.05%   -6.13%     
==========================================
  Files        1130      858     -272     
  Lines       15404    12050    -3354     
  Branches     2435     1951     -484     
==========================================
- Hits         4033     2417    -1616     
+ Misses       9556     8062    -1494     
+ Partials     1815     1571     -244     

Flag	Coverage Δ
#front	14.68% <ø> (-3.61%)	⬇️
#unit	42.45% <ø> (-10.50%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...t-manager/admin/src/containers/EditView/reducer.js	0.00% <0.00%> (-100.00%)	⬇️
...manager/admin/src/containers/ListView/constants.js	0.00% <0.00%> (-100.00%)	⬇️
...t-manager/admin/src/containers/ListView/reducer.js	0.00% <0.00%> (-93.34%)	⬇️
...t-manager/admin/src/containers/ListView/actions.js	0.00% <0.00%> (-90.91%)	⬇️
...api-admin/admin/src/containers/AuthPage/reducer.js	0.00% <0.00%> (-85.72%)	⬇️
packages/strapi-utils/lib/sanitize-entity.js	5.55% <0.00%> (-55.74%)	⬇️
packages/strapi-utils/lib/config.js	13.51% <0.00%> (-51.36%)	⬇️
...strapi-admin/admin/src/containers/Admin/actions.js	50.00% <0.00%> (-50.00%)	⬇️
...strapi-admin/admin/src/containers/Admin/reducer.js	80.00% <0.00%> (-20.00%)	⬇️
packages/strapi-utils/lib/validators.js	88.88% <0.00%> (-6.95%)	⬇️
... and 539 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 35bae7d...d45f6f7. Read the comment docs.

[alexandrebodin]

Thanks @iamdarshshah, closing as the image is already up to date

[iamdarshshah]

Hello @alexandrebodin

It looks like, the issue still exists. Can you please have a look at it again?

Screenshot

[lauriejim]

Hum looks good on my side... The image is hosted on GitHub so that's weird.
Cache issue?

[iamdarshshah]

@lauriejim It seems the issue is common across the different browsers. It could not be related to Cache. Maybe something else. Can you please share the screenshot?

Thanks 👍

[lauriejim]

Just tried on my mobile - looks good.

[iamdarshshah]

Okay, cool! No problem... If it works fine then there is no need for updating it. Still don't know why it's not working for me 😕

Will try to look for the solution to this issue. Anyways! Thank you 👍

[alexandrebodin]

Let us know if this comes from a specific browser so we can fix it :D

[iamdarshshah]

@alexandrebodin
I have tried with a different system and with different browsers. Here are the results:

I don't know what might be the reason behind this. The issue still exists across different browsers.

Screenshots:

1. Safari

2. Chrome

3. Opera

(P.S - Updated the PR info with a screenshot that includes changes that has been updated through this PR.)

[iamdarshshah]

This might happen due to the fact that Amazon AWS serves content to specific locations or rather I would say, It will work for certain locations only. It restricts data access to certain locations, this seems to be the reason for this issue. Due to which I can't access this particular file. Also, tried visiting current image URL directly and got this error as a response:

We can update the URL by https://strapi.io/assets/strapi-logo.svg, which will directly point to the static files associated with the Strapi website. This will resolve this particular issue.

would appreciate your thoughts/ideas on this @lauriejim & @alexandrebodin

[alexandrebodin]

@iamdarshshah You can actually use this one instead so it is the right one :D https://strapi.io/assets/strapi-logo-dark.svg

[iamdarshshah]

@alexandrebodin Sure, Updating that! But, need to reopen this PR to update the commit. Thanks 👍

[iamdarshshah]

@alexandrebodin Should I open new PR for updating image URL?

[alexandrebodin]

@iamdarshshah I'll reopen this one so you can update 👍

[iamdarshshah]

Thanks @alexandrebodin 👍 , I have updated the image URL. You can review it now :)

[alexandrebodin]

LGTM THanks !