Skip to content
This repository has been archived by the owner on Oct 3, 2021. It is now read-only.

Fix health page not working #27

Merged
merged 5 commits into from
Aug 4, 2017
Merged

Conversation

MrPetovan
Copy link
Collaborator

Fixes issue #24

  • Fix formatting
  • Add references to App
  • Fix parse error
  • Fix a bunch of notices
  • Remove inexplicable killme() for *_content pages preventing them from
    displaying

Hypolite Petovan added 5 commits June 25, 2017 10:17
- Add maintenance HTML file
- Add maintenance HTML file
… (user)" errors

- Changed returning true when there's a session ID but no session data
in ref_session_write()
- Moved session.php requires around
- Fix formatting
- Add references to App
- Fix parse error
- Fix a bunch of notices
- Remove inexplicable killme() for *_content pages preventing them from
displaying
Copy link
Collaborator

@annando annando left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just some questions.

return replace_macros($tpl, array(
'$versions' => $versions,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is "versions" vanished?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have no idea, the variable didn't exist in the controller nor in the template.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay. When you had checked that, then It's fine for me.

</h2>

<p class="about">
The Friendica Directory is unavailable at this time, please connect with <a href="https://friendica.mrpetovan.com/profile/hypolite">hypolite@friendica.mrpetovan.com</a> for updates about its status.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm confused: Hadn't you added this here with the other PR as well?

@MrPetovan MrPetovan merged commit 109633d into friendica:master Aug 4, 2017
@MrPetovan MrPetovan deleted the issue/#24 branch August 4, 2017 01:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants