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.

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

GitHub is mis-cased in the edit image #1421

Closed
jsoref opened this issue Dec 20, 2020 · 7 comments · May be fixed by #1425
Closed

GitHub is mis-cased in the edit image #1421

jsoref opened this issue Dec 20, 2020 · 7 comments · May be fixed by #1425

Comments

@jsoref
Copy link

jsoref commented Dec 20, 2020

Github

See GitHub

@spotz
Copy link
Contributor

spotz commented Mar 19, 2021

This image has been removed so this issue is closed. @amoralej can you close this one?

@jsoref
Copy link
Author

jsoref commented Mar 19, 2021

I'm not sure what you mean. The image is still very much in https://github.com/redhat-openstack/website/blob/master/source/images/edit.png and https://github.com/redhat-openstack/website/tree/master/source/images

If you either fix the content of the image or delete the image from the tip of master, then it's removed.

Until then, the file itself is an attractive nuisance -- someone may choose to use it thinking it's good, when it's in fact bad.

If you're no longer using the file, then deleting it should be a pretty simple task.

@spotz
Copy link
Contributor

spotz commented Mar 19, 2021 via email

@jsoref
Copy link
Author

jsoref commented Mar 19, 2021

Great, so you can delete the file from the repository, would it help if I created the PR for it?

@spotz
Copy link
Contributor

spotz commented Mar 19, 2021 via email

@jsoref
Copy link
Author

jsoref commented Mar 19, 2021

Please do it for me, thanks.

rdoproject pushed a commit that referenced this issue Mar 31, 2021
As requested in GitHub is mis-cased in the edit image (#1421) the
edit.png file has been removed the repo. It was no longer in use
due to switch to Gerrit

Change-Id: I2e4369bd2ad127fa7559e62466a8ba9edbb73fac
@apevec
Copy link
Member

apevec commented Apr 7, 2023

https://review.rdoproject.org/r/c/rdo-website/+/32620

@jsoref jsoref closed this as completed Apr 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants