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

Update setup_page.php #19

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

javaftper
Copy link

Just a small syntax error that was causing the clock synchronisation code to fail. isSyncTZ instead of $isSyncTZ.

Just a small syntax error that was causing the clock synchronisation code to fail.  isSyncTZ instead of $isSyncTZ.
@sultanoswing
Copy link

Great to see this is still being updated for we stalwart S20 users, thanks javaftper!

Any chance you have the know-how to implement an auto servertime sync. function, since fernando silva, the original maestro hasn't picked that issue up over the last year.

@javaftper
Copy link
Author

There was an update in January 2021 by Fernando that addressed the auto sync- I forgot to close that issue- It's not done exactly the same way I suggested but it achieves the same thing pretty much. Have you updated your local code base since then? You should also update your setup_page.php With this version.

If it doesn't work, you might need to update your local PHP version because I think the code was updated to be compatible with PHP 8 and I think there were some deprecated functions/keywords that had to be changed.

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

Successfully merging this pull request may close these issues.

2 participants