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.

Sign up for GitHub

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

Jump to bottom

Site Theme Loads at Bottom of "Site Currently Offline" Page #3087

Closed
shefik opened this issue Sep 26, 2016 · 7 comments
Closed

Site Theme Loads at Bottom of "Site Currently Offline" Page #3087

shefik opened this issue Sep 26, 2016 · 7 comments
Labels
Bug
Milestone
1.4.4

Comments

@shefik
Copy link
Contributor

shefik commented Sep 26, 2016 •
edited
Loading

Q A
Zikula Version 1.4.3
PHP Version 5.x

Expected behavior

The site theme is loading at the bottom of the "Site Currently Offline" page.

Actual behavior

Only the "Site Currently Offline" message should display.

Steps to reproduce
@shefik
Copy link
Contributor Author

shefik commented Sep 26, 2016

offline

@shefik shefik changed the title Unable to Upgrade to Zikula 1.4.3 Site Theme Loads at Bottom of Upgrade Page Sep 26, 2016
@shefik shefik changed the title Site Theme Loads at Bottom of Upgrade Page Site Theme Loads at Bottom of "Site Currently Offline" Page Sep 26, 2016
@shefik
Copy link
Contributor Author

shefik commented Sep 26, 2016

upgrade-theme

@craigh
Copy link
Member

craigh commented Oct 5, 2016

Steps to reproduce

@Guite Guite modified the milestone: 1.4.4 Oct 21, 2016
@Guite
Copy link
Member

Guite commented Oct 21, 2016

ping @shefik please explain how to reproduce

@Guite
Copy link
Member

Guite commented Oct 23, 2016

Not able to reproduce, postponing.

@Guite Guite modified the milestones: 1.4.6, 1.4.4 Oct 23, 2016
@Guite Guite modified the milestones: 1.4.6, 1.4.5, 1.4.4 Nov 12, 2016
@Guite
Copy link
Member

Guite commented Nov 12, 2016

Closed to lack of response.

@Guite Guite closed this as completed Nov 12, 2016
@craigh
Copy link
Member

craigh commented Nov 17, 2016

this was fixed by f171d66. injecting a PlainResponse() was required.

@craigh craigh mentioned this issue Jan 7, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug
Projects
None yet
Milestone
1.4.4
Development

No branches or pull requests
3 participants
@shefik @Guite @craigh