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

Weird precedence issues between config.yml and ActiveAdmin #465

Open
camdendotlol opened this issue Jun 23, 2023 · 1 comment
Open

Weird precedence issues between config.yml and ActiveAdmin #465

camdendotlol opened this issue Jun 23, 2023 · 1 comment

Comments

@camdendotlol
Copy link

camdendotlol commented Jun 23, 2023

On https://umich.covecollective.org/, the "University of Michigan" text in the banner is given the banner_color attribute instead of the site_color attribute, causing it to be indistinguishable from the banner.

This issue is really strange.

Update:

The issue is that the colors are pulled from the config/domain_specific/config.yml file, and the values set in ActiveAdmin are ignored. https://colorado.covecollective.org/ is also affected, where the text color is supposed to be gold but the config file sets it to light gray. I've definitely successfully changed a tenant's colors from ActiveAdmin before, so I'm wondering if there's a precedence thing where whatever is in config.yml takes priority over what's set in ActiveAdmin.

This is a really weird system, and I would propose fixing it by removing the site_color and banner_color lines from config.yml, and making them only updateable via ActiveAdmin. Being able to update the values in the admin interface is by far preferable because it doesn't require code changes or deployments.

@camdendotlol camdendotlol changed the title Incorrect config value being read for UMich Weird precedence issues between config.yml and ActiveAdmin Jun 23, 2023
@camdendotlol
Copy link
Author

@jamiefolsom Dino reported a text color bug in the UMich site and it sent me down this rabbit hole. I could fix the text color in the short term by committing an updated config.yml, but I'd really like to deal with the source of the issue, because having randomly non-functional fields in the admin interface isn't good.

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