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

No partial for directory image setting #1967

Open
ShanaLMoore opened this issue Jul 18, 2023 · 0 comments
Open

No partial for directory image setting #1967

ShanaLMoore opened this issue Jul 18, 2023 · 0 comments

Comments

@ShanaLMoore
Copy link
Collaborator

ShanaLMoore commented Jul 18, 2023

Descriptive summary

Hyku v4.1.0

An admin can set a directory image and an alt text for it, but it is not being utilized.

We noticed that Pals has a partial that uses it. It must not have been contributed back to hyku.

ref Pals staging. I set a tenant's directory image to a picture of baby diego:
image (23)

image

Rationale

It's misleading to allow a user to set something that they cannot see or use. Either pull in the partial that uses it or delete the ability to set it.

Expected behavior

Actual behavior

Steps to reproduce the behavior

  1. Go to admin dashboard, settings, appearance and click on the directory image tag
  2. Upload a photo and set the alt text
  3. go to the proprietor home page (ie: hyku.test). The directory image should be used there like it is in PALS.

Related work

scientist-softserv/utk-hyku#471

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

No branches or pull requests

1 participant