-
Notifications
You must be signed in to change notification settings - Fork 110
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
Replace organization logo URLs with uploaded images #1850
Comments
This issue is |
I tested this on catalog-next and I'm still seeing a broken image. |
@adborden this is a PR so it does not display a broken link for either upload or link url - once this is merged I can upload the images GSA/ckanext-datagovtheme#81 |
@thejuliekramer when you upload an organization image, where is it stored? |
Not complete. Catalog is displaying the uploaded image. But image is stored at local instance: Inventory is showing a broken image after upload. Image is stored at local instance |
I'm going to remove this from the Epic since it is a standalone problem no resolved by the 2.8 rollout. |
User Story
In order to control the delivery of organization logos in CKAN and prevent broken images, data.gov operators want to replace any image URLs from CKAN2.3 with image uploads (in CKAN 2.8).
Acceptance Criteria
[ACs should be clearly demoable/verifiable whenever possible. Try specifying them using BDD.]
AND I inspect the page source
THEN I see the logo is served from a data.gov domain or a data.gov CDN
Background
Catalog never supported organization image uploads, you had to supply a URL. Some logos point to image in GSA/logo and some point to third-party resources (Wikipedia).
This causes problems when third-parties remove or change the image. Suddenly we'll start seeing broken links on catalog.data.gov organization pages.
In CKAN2.8, we can upload images directly to be served from CKAN.
Security Considerations (required)
None
Sketch
[Notes or a checklist reflecting our understanding of the selected approach]
The text was updated successfully, but these errors were encountered: