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

Staging favicon not being shown #2301

Open
irilyth opened this issue Dec 5, 2017 · 5 comments
Open

Staging favicon not being shown #2301

irilyth opened this issue Dec 5, 2017 · 5 comments
Labels

Comments

@irilyth
Copy link

irilyth commented Dec 5, 2017

The staging site is not showing the staging favicon.

@blackshadowshade
Copy link
Contributor

I've just checked, and the staging config table looks like this:

mysql> select * from config;
+-----------+-------------+
| conf_key | conf_value |
+-----------+-------------+
| site_type | development |
+-----------+-------------+
1 row in set (0.00 sec)

Should this should have been set to 'staging' at some stage?

@irilyth
Copy link
Author

irilyth commented Dec 5, 2017 via email

@blackshadowshade
Copy link
Contributor

Linking #2180, which was the pull request that implemented the changes needed to show the staging favicon.

@blackshadowshade
Copy link
Contributor

So, I discovered today that the selection of the favicon is entirely dependent on the value of Config.siteType (set in Config.js). That should help us track down the incorrect favicon being loaded on staging.

@blackshadowshade
Copy link
Contributor

I've manually changed Config.1586785864.js to have the siteType set to 'staging'. After clearing my JS cache, this fixes this problem, so staging is now showing with the correct favicon and the yellow-backgrounded header.

@cgolubi1 has said that she'll check how her deploy script is setting the Config.siteType, since this is apparently not being set correctly for the dev and staging sites.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants