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

PageTexts: Find out why hero and about us sections not showing PageText data #682

Closed
kasugaijin opened this issue May 2, 2024 · 2 comments · Fixed by #687
Closed

PageTexts: Find out why hero and about us sections not showing PageText data #682

kasugaijin opened this issue May 2, 2024 · 2 comments · Fixed by #687
Labels
bug 🐞 Something isn't working

Comments

@kasugaijin
Copy link
Collaborator

This PR #618 enabled orgs to add Hero images and About Us images that would be displayed on the org root page if added. On the production site https://pet-rescue-f6df7ce0e850.herokuapp.com this works for Alta, but not Baja. Why?

@kasugaijin kasugaijin added the bug 🐞 Something isn't working label May 2, 2024
@jmilljr24
Copy link
Collaborator

This might have something to do with the seed files. 02_baja has PageText.create!(hero: nil, about: nil) and 01_alta does not. They both create the PageText(with values) when the org is created so baja ends up have two PageTexts.

I can't seem to produce the issue locally but it would make sense that the conditionals in home/index view are false if the Current.organization.page_text being checked has the nil values.

Could you try deleting the nil pagetext from the rails console on heroku and see if that fixes it? Or would it be easier to push up a change and reset the db?

@kasugaijin
Copy link
Collaborator Author

Good catch! That looks like a likely culprit @jmilljr24
I think we should push an update of the seed file. I think every successful deploy to heroku will rerun bin/setup so it should re-seed. If not, we can do that after. Did you want to make a PR with that seed file change?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐞 Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants