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 zpush.sh to version 2.7.0 #2236

Merged
merged 4 commits into from
May 13, 2023
Merged

Update zpush.sh to version 2.7.0 #2236

merged 4 commits into from
May 13, 2023

Conversation

matidau
Copy link
Contributor

@matidau matidau commented Feb 21, 2023

Updated Z-Push repo that works with PHP8 and Mail-in-a-box

Updated Z-Push repo that works with PHP8 and Mail-in-a-box
@jvolkenant
Copy link
Contributor

I still find that my Inbox (which has 20k+ emails), disappears and needs to be resynced with a few different z-push php 8 branches I've tried, this branch included.

@matidau
Copy link
Contributor Author

matidau commented Feb 21, 2023

I thought we were almost home free. Do you mind grabbing your Z-Push log files and dropping them in this issue matidau/Z-Push#3
/var/log/z-push/z-push.log
/var/log/z-push/z-push-error.log

and just in case the autodiscover logs
/var/log/z-push/autodiscover.log
/var/log/z-push/autodiscover-error.log

Also what device/s are you testing with?

Appreciate you raising this and any testing you can help with.

Edit: and just to check did you run the last line from zpush.sh?
# Fix states after upgrade
hide_output php$PHP_VER /usr/local/lib/z-push/z-push-admin.php -a fixstates

@jvolkenant
Copy link
Contributor

To report an update; Things have worked much better with matidau's changes 👍

Update to new community release of 2.7.0
Change back to main Z-Hub repo for latest release
@matidau matidau changed the title Update zpush.sh Update zpush.sh to version 2.7.0 Apr 9, 2023
@jvolkenant
Copy link
Contributor

Your branch was working great for the last few weeks. Trying 2.7.0. I appreciate the work!

@JoshData
Copy link
Member

So is everything OK with 2.7.0? Is this PR ok to merge?

@matidau
Copy link
Contributor Author

matidau commented May 13, 2023

I believe it is :)

@JoshData JoshData merged commit 93380b2 into mail-in-a-box:main May 13, 2023
@JoshData
Copy link
Member

Thanks!

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.

3 participants